From: Daniel Lezcano <daniel.lezcano@linaro.org>
In the context of the time keeping and the timers, some platforms have
timers which need to be initialized very early. It is the case of the
ARM platform which do not have the architected timers.
The macro TIMER_OF_DECLARE adds an entry in the timer init functions
array at compile time and the function timer_probe is called from the
timer_init() function in kernel/time.c
This array contains a tuple with the init function and the compatible
string.
The init function has a device node pointer parameter.
The timer_probe() function browses the of nodes and find the ones
matching the compatible string given when using the TIMER_OF_DECLARE
macro. It then calls the init function with the device node as a
pointer.
But there are some platforms where there are multiple timers like the
ARM64 with the architected timers. Those are always initialized very
early and the other timers can be initialized later.
For this reason we find timer drivers with the platform_driver
incarnation. Consequently their init functions are different, they
have a platform_device pointer parameter and rely on the devm_
function for rollbacking.
To summarize, we have:
- TIMER_OF_DECLARE with init function prototype:
int (*init)(struct device_node *np);
- module_platform_driver (and variant) with the probe function
prototype:
int (*init)(struct platform_device *pdev);
The current situation with the timers is the following:
- Two platforms can have the same timer hardware, hence the same
driver but one without alternate timers and the other with multiple
timers. For example, the Exynos platform has only the Exynos MCT on
ARM but has the architeched timers in addition on the ARM64.
- The timer drivers can be modules now which was not the case until
recently. TIMER_OF_DECLARE do not allow the build as a module.
It results in duplicate init functions (one with rollback and one with
devm_) and different way to declare the driver (TIMER_OF_DECLARE and
module_platform_driver).
This proposed change is to unify the prototyping of the init functions
to receive a platform_device pointer as parameter. Consequently, it
will allow a smoother and nicer module conversion and a huge cleanup
of the init functions by removing all the rollback code from all the
timer drivers. It introduces a TIMER_PDEV_DECLARE() macro.
If the macro is used a platform_device is manually allocated and
initialized with the needed information for the probe
function. Otherwise module_platform_driver can be use instead with the
same probe function without the timer_probe() initialization.
The plan is to have all timers to use TIMER_PDEV_DECLARE with all the
init functions optimized and then remove the TIMER_OF_DECLARE macro.
Signed-off-by: Daniel Lezcano <daniel.lezcano@kernel.org>
Cc: Hans de Goede <hansg@kernel.org>
Cc: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Cc: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Cc: Rob Herring <robh@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
drivers/clocksource/timer-probe.c | 66 +++++++++++++++++++++++++++++--
include/asm-generic/vmlinux.lds.h | 10 +++++
include/linux/clocksource.h | 21 ++++++++++
3 files changed, 93 insertions(+), 4 deletions(-)
diff --git a/drivers/clocksource/timer-probe.c b/drivers/clocksource/timer-probe.c
index b7860bc0db4b..cdaceb68d356 100644
--- a/drivers/clocksource/timer-probe.c
+++ b/drivers/clocksource/timer-probe.c
@@ -7,13 +7,11 @@
#include <linux/init.h>
#include <linux/of.h>
#include <linux/clocksource.h>
+#include <linux/platform_device.h>
extern struct of_device_id __timer_of_table[];
-static const struct of_device_id __timer_of_table_sentinel
- __used __section("__timer_of_table_end");
-
-void __init timer_probe(void)
+static int __init timer_of_probe(void)
{
struct device_node *np;
const struct of_device_id *match;
@@ -38,6 +36,66 @@ void __init timer_probe(void)
timers++;
}
+ return timers;
+}
+
+static int __init __timer_pdev_probe(struct platform_driver *drv)
+{
+ struct device_node *np;
+ struct platform_device *pdev;
+ const struct of_device_id *match;
+ unsigned int timers = 0;
+ int ret;
+
+ for_each_matching_node_and_match(np, drv->driver.of_match_table, &match) {
+ if (!of_device_is_available(np))
+ continue;
+
+ pdev = platform_device_alloc(of_node_full_name(np), -1);
+ if (!pdev)
+ continue;
+
+ ret = device_add_of_node(&pdev->dev, np);
+ if (ret) {
+ platform_device_put(pdev);
+ continue;
+ }
+
+ dev_set_name(&pdev->dev, pdev->name);
+
+ ret = drv->probe(pdev);
+ if (!ret) {
+ timers++;
+ continue;
+ }
+
+ if (ret != -EPROBE_DEFER)
+ pr_err("Failed to initialize '%pOF': %d\n", np, ret);
+
+ device_remove_of_node(&pdev->dev);
+
+ platform_device_put(pdev);
+ }
+
+ return timers;
+}
+
+static int __init timer_pdev_probe(void)
+{
+ struct platform_driver **drv;
+
+ for_each_pdev_timer_table(drv)
+ __timer_pdev_probe(*drv);
+
+ return 0;
+}
+
+void __init timer_probe(void)
+{
+ unsigned timers = 0;
+
+ timers += timer_of_probe();
+ timers += timer_pdev_probe();
timers += acpi_probe_device_table(timer);
if (!timers)
diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index eeb070f330bd..5d619e831dce 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -359,6 +359,15 @@
#define THERMAL_TABLE(name)
#endif
+#ifdef CONFIG_TIMER_OF
+#define TIMER_TABLE(name) \
+ . = ALIGN(8); \
+ BOUNDED_SECTION_POST_LABEL(__##name##_timer_table, \
+ __##name##_timer_table,, _end)
+#else
+#define TIMER_TABLE(name)
+#endif
+
#define KERNEL_DTB() \
STRUCT_ALIGN(); \
__dtb_start = .; \
@@ -738,6 +747,7 @@
ACPI_PROBE_TABLE(irqchip) \
ACPI_PROBE_TABLE(timer) \
THERMAL_TABLE(governor) \
+ TIMER_TABLE(pdev) \
EARLYCON_TABLE() \
LSM_TABLE() \
EARLY_LSM_TABLE() \
diff --git a/include/linux/clocksource.h b/include/linux/clocksource.h
index 65b7c41471c3..6e05b78e64b8 100644
--- a/include/linux/clocksource.h
+++ b/include/linux/clocksource.h
@@ -18,6 +18,7 @@
#include <linux/init.h>
#include <linux/of.h>
#include <linux/clocksource_ids.h>
+#include <linux/platform_device.h>
#include <asm/div64.h>
#include <asm/io.h>
@@ -295,6 +296,26 @@ extern void timer_probe(void);
static inline void timer_probe(void) {}
#endif
+extern struct platform_driver *__pdev_timer_table[];
+extern struct platform_driver *__pdev_timer_table_end[];
+
+#define TIMER_PDEV_DECLARE(__name, __probe, __remove, __match) \
+ static struct platform_driver __pdev_timer_table_entry_##__name = { \
+ .probe = __probe, \
+ .remove = __remove, \
+ .driver = { \
+ .name = #__name, \
+ .of_match_table = __match \
+ }, \
+ }; \
+ static struct platform_driver *___pdev_timer_table_entry_##__name \
+ __used __section("__pdev_timer_table") = &__pdev_timer_table_entry_##__name
+
+#define for_each_pdev_timer_table(__pdev) \
+ for (__pdev = __pdev_timer_table; \
+ __pdev < __pdev_timer_table_end; \
+ __pdev++)
+
#define TIMER_ACPI_DECLARE(name, table_id, fn) \
ACPI_DECLARE_PROBE_ENTRY(timer, name, table_id, 0, NULL, 0, fn)
--
2.43.0
Hi Daniel,
kernel test robot noticed the following build errors:
[auto build test ERROR on tip/timers/core]
[also build test ERROR on rockchip/for-next arnd-asm-generic/master linus/master v7.0-rc5 next-20260327]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Daniel-Lezcano/clocksource-drivers-timer-probe-Create-a-platform_device-before-the-framework-is-initialized/20260328-080154
base: tip/timers/core
patch link: https://lore.kernel.org/r/20260327180600.8150-2-daniel.lezcano%40kernel.org
patch subject: [PATCH resend v1 1/7] clocksource/drivers/timer-probe: Create a platform_device before the framework is initialized
config: x86_64-randconfig-013-20260328 (https://download.01.org/0day-ci/archive/20260328/202603282049.OhKWsoYR-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260328/202603282049.OhKWsoYR-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603282049.OhKWsoYR-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from arch/x86/include/asm/current.h:11,
from arch/x86/include/asm/processor.h:17,
from arch/x86/include/asm/timex.h:5,
from include/linux/timex.h:67,
from include/linux/time32.h:13,
from include/linux/time.h:60,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:11,
from arch/x86/entry/vdso/vdso32/vclock_gettime.c:1:
arch/x86/include/asm/tlbflush.h: In function 'cpu_tlbstate_update_lam':
arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:369:25: note: in expansion of macro 'this_cpu_write_1'
369 | case 1: stem##1(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:370:25: note: in expansion of macro 'this_cpu_write_2'
370 | case 2: stem##2(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:371:25: note: in expansion of macro 'this_cpu_write_4'
371 | case 4: stem##4(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
In file included from arch/x86/include/asm/percpu.h:598:
arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
include/asm-generic/percpu.h:87:33: note: in definition of macro 'raw_cpu_generic_to_op'
87 | *raw_cpu_ptr(&(pcp)) op val; \
| ^~~
include/asm-generic/percpu.h:412:41: note: in expansion of macro 'this_cpu_generic_to_op'
412 | #define this_cpu_write_8(pcp, val) this_cpu_generic_to_op(pcp, val, =)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/percpu-defs.h:372:25: note: in expansion of macro 'this_cpu_write_8'
372 | case 8: stem##8(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
In file included from include/linux/kasan.h:38,
from include/linux/slab.h:264,
from include/linux/fs.h:45,
from include/linux/compat.h:17,
from arch/x86/include/asm/ia32.h:7,
from arch/x86/include/asm/elf.h:10,
from include/linux/elf.h:6,
from include/linux/module.h:20,
from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/platform_device.h:13,
from include/linux/clocksource.h:21,
from include/clocksource/hyperv_timer.h:16,
from arch/x86/include/asm/vdso/gettimeofday.h:20,
from include/vdso/datapage.h:196,
from arch/x86/include/uapi/../../../../lib/vdso/gettimeofday.c:6,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:16:
include/linux/pgtable.h: At top level:
>> include/linux/pgtable.h:2204:2: error: #error Missing MAX_POSSIBLE_PHYSMEM_BITS definition
2204 | #error Missing MAX_POSSIBLE_PHYSMEM_BITS definition
| ^~~~~
arch/x86/include/asm/ia32.h:28:8: error: redefinition of 'struct stat64'
28 | struct stat64 {
| ^~~~~~
In file included from include/linux/stat.h:6,
from include/linux/sysfs.h:22,
from include/linux/kobject.h:20,
from include/linux/of.h:18,
from include/linux/clocksource.h:19:
arch/x86/include/uapi/asm/stat.h:42:8: note: originally defined here
42 | struct stat64 {
| ^~~~~~
vim +2204 include/linux/pgtable.h
^1da177e4c3f41 include/asm-generic/pgtable.h Linus Torvalds 2005-04-16 2196
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2197 #if !defined(MAX_POSSIBLE_PHYSMEM_BITS) && !defined(CONFIG_64BIT)
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2198 #ifdef CONFIG_PHYS_ADDR_T_64BIT
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2199 /*
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2200 * ZSMALLOC needs to know the highest PFN on 32-bit architectures
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2201 * with physical address space extension, but falls back to
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2202 * BITS_PER_LONG otherwise.
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2203 */
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 @2204 #error Missing MAX_POSSIBLE_PHYSMEM_BITS definition
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2205 #else
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2206 #define MAX_POSSIBLE_PHYSMEM_BITS 32
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2207 #endif
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2208 #endif
cef397038167ac include/linux/pgtable.h Arnd Bergmann 2020-11-11 2209
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Hi Daniel,
kernel test robot noticed the following build errors:
[auto build test ERROR on tip/timers/core]
[also build test ERROR on rockchip/for-next arnd-asm-generic/master linus/master v7.0-rc5 next-20260327]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Daniel-Lezcano/clocksource-drivers-timer-probe-Create-a-platform_device-before-the-framework-is-initialized/20260328-080154
base: tip/timers/core
patch link: https://lore.kernel.org/r/20260327180600.8150-2-daniel.lezcano%40kernel.org
patch subject: [PATCH resend v1 1/7] clocksource/drivers/timer-probe: Create a platform_device before the framework is initialized
config: x86_64-defconfig (https://download.01.org/0day-ci/archive/20260328/202603281507.BshDOBFT-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260328/202603281507.BshDOBFT-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603281507.BshDOBFT-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
In file included from arch/x86/include/asm/processor.h:21,
from arch/x86/include/asm/timex.h:5,
from include/linux/timex.h:67,
from include/linux/time32.h:13,
from include/linux/time.h:60,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:11,
from arch/x86/entry/vdso/vdso32/vclock_gettime.c:1:
arch/x86/include/asm/pgtable.h: In function 'pte_flags_pkey':
>> arch/x86/include/asm/pgtable_types.h:69:43: warning: left shift count >= width of type [-Wshift-count-overflow]
69 | #define _PAGE_PKEY_BIT0 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT0)
| ^~
arch/x86/include/asm/pgtable_types.h:80:26: note: in expansion of macro '_PAGE_PKEY_BIT0'
80 | #define _PAGE_PKEY_MASK (_PAGE_PKEY_BIT0 | \
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable.h:1598:29: note: in expansion of macro '_PAGE_PKEY_MASK'
1598 | return (pte_flags & _PAGE_PKEY_MASK) >> _PAGE_BIT_PKEY_BIT0;
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:70:43: warning: left shift count >= width of type [-Wshift-count-overflow]
70 | #define _PAGE_PKEY_BIT1 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT1)
| ^~
arch/x86/include/asm/pgtable_types.h:81:26: note: in expansion of macro '_PAGE_PKEY_BIT1'
81 | _PAGE_PKEY_BIT1 | \
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable.h:1598:29: note: in expansion of macro '_PAGE_PKEY_MASK'
1598 | return (pte_flags & _PAGE_PKEY_MASK) >> _PAGE_BIT_PKEY_BIT0;
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:71:43: warning: left shift count >= width of type [-Wshift-count-overflow]
71 | #define _PAGE_PKEY_BIT2 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT2)
| ^~
arch/x86/include/asm/pgtable_types.h:82:26: note: in expansion of macro '_PAGE_PKEY_BIT2'
82 | _PAGE_PKEY_BIT2 | \
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable.h:1598:29: note: in expansion of macro '_PAGE_PKEY_MASK'
1598 | return (pte_flags & _PAGE_PKEY_MASK) >> _PAGE_BIT_PKEY_BIT0;
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:72:43: warning: left shift count >= width of type [-Wshift-count-overflow]
72 | #define _PAGE_PKEY_BIT3 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT3)
| ^~
arch/x86/include/asm/pgtable_types.h:83:26: note: in expansion of macro '_PAGE_PKEY_BIT3'
83 | _PAGE_PKEY_BIT3)
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable.h:1598:29: note: in expansion of macro '_PAGE_PKEY_MASK'
1598 | return (pte_flags & _PAGE_PKEY_MASK) >> _PAGE_BIT_PKEY_BIT0;
| ^~~~~~~~~~~~~~~
In file included from arch/x86/include/asm/tlbflush.h:17,
from arch/x86/include/asm/uaccess.h:17,
from include/linux/uaccess.h:13,
from include/linux/sched/task.h:13,
from include/linux/sched/signal.h:9,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs/super_types.h:13,
from include/linux/fs/super.h:5,
from include/linux/fs.h:5,
from include/linux/compat.h:17,
from arch/x86/include/asm/ia32.h:7,
from arch/x86/include/asm/elf.h:10,
from include/linux/elf.h:6,
from include/linux/module.h:20,
from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/platform_device.h:13,
from include/linux/clocksource.h:21,
from include/clocksource/hyperv_timer.h:16,
from arch/x86/include/asm/vdso/gettimeofday.h:20,
from include/vdso/datapage.h:196,
from arch/x86/include/uapi/../../../../lib/vdso/gettimeofday.c:6,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:16:
>> arch/x86/include/asm/pgtable.h:1598:46: warning: right shift count >= width of type [-Wshift-count-overflow]
1598 | return (pte_flags & _PAGE_PKEY_MASK) >> _PAGE_BIT_PKEY_BIT0;
| ^~
arch/x86/include/asm/tlbflush.h: In function 'pte_flags_need_flush':
>> arch/x86/include/asm/pgtable_types.h:69:43: warning: left shift count >= width of type [-Wshift-count-overflow]
69 | #define _PAGE_PKEY_BIT0 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT0)
| ^~
arch/x86/include/asm/tlbflush.h:377:45: note: in expansion of macro '_PAGE_PKEY_BIT0'
377 | _PAGE_PAT_LARGE | _PAGE_PKEY_BIT0 | _PAGE_PKEY_BIT1 |
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:70:43: warning: left shift count >= width of type [-Wshift-count-overflow]
70 | #define _PAGE_PKEY_BIT1 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT1)
| ^~
arch/x86/include/asm/tlbflush.h:377:63: note: in expansion of macro '_PAGE_PKEY_BIT1'
377 | _PAGE_PAT_LARGE | _PAGE_PKEY_BIT0 | _PAGE_PKEY_BIT1 |
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:71:43: warning: left shift count >= width of type [-Wshift-count-overflow]
71 | #define _PAGE_PKEY_BIT2 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT2)
| ^~
arch/x86/include/asm/tlbflush.h:378:27: note: in expansion of macro '_PAGE_PKEY_BIT2'
378 | _PAGE_PKEY_BIT2 | _PAGE_PKEY_BIT3 | _PAGE_NX;
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/pgtable_types.h:72:43: warning: left shift count >= width of type [-Wshift-count-overflow]
72 | #define _PAGE_PKEY_BIT3 (_AT(pteval_t, 1) << _PAGE_BIT_PKEY_BIT3)
| ^~
arch/x86/include/asm/tlbflush.h:378:45: note: in expansion of macro '_PAGE_PKEY_BIT3'
378 | _PAGE_PKEY_BIT2 | _PAGE_PKEY_BIT3 | _PAGE_NX;
| ^~~~~~~~~~~~~~~
arch/x86/include/asm/ia32.h: At top level:
>> arch/x86/include/asm/ia32.h:28:8: error: redefinition of 'struct stat64'
28 | struct stat64 {
| ^~~~~~
In file included from include/linux/stat.h:6,
from include/linux/sysfs.h:22,
from include/linux/kobject.h:20,
from include/linux/of.h:18,
from include/linux/clocksource.h:19:
arch/x86/include/uapi/asm/stat.h:42:8: note: originally defined here
42 | struct stat64 {
| ^~~~~~
vim +28 arch/x86/include/asm/ia32.h
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 24
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 25 /* This matches struct stat64 in glibc2.2, hence the absolutely
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 26 * insane amounts of padding around dev_t's.
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 27 */
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 @28 struct stat64 {
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 29 unsigned long long st_dev;
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 30 unsigned char __pad0[4];
^1da177e4c3f415 include/asm-x86_64/ia32.h Linus Torvalds 2005-04-16 31
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Hi Daniel,
kernel test robot noticed the following build warnings:
[auto build test WARNING on tip/timers/core]
[also build test WARNING on rockchip/for-next arnd-asm-generic/master linus/master v7.0-rc5 next-20260327]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Daniel-Lezcano/clocksource-drivers-timer-probe-Create-a-platform_device-before-the-framework-is-initialized/20260328-080154
base: tip/timers/core
patch link: https://lore.kernel.org/r/20260327180600.8150-2-daniel.lezcano%40kernel.org
patch subject: [PATCH resend v1 1/7] clocksource/drivers/timer-probe: Create a platform_device before the framework is initialized
config: x86_64-randconfig-013-20260328 (https://download.01.org/0day-ci/archive/20260328/202603281427.LUamgyCn-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260328/202603281427.LUamgyCn-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603281427.LUamgyCn-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from arch/x86/include/asm/current.h:11,
from arch/x86/include/asm/processor.h:17,
from arch/x86/include/asm/timex.h:5,
from include/linux/timex.h:67,
from include/linux/time32.h:13,
from include/linux/time.h:60,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:11,
from arch/x86/entry/vdso/vdso32/vclock_gettime.c:1:
arch/x86/include/asm/tlbflush.h: In function 'cpu_tlbstate_update_lam':
>> arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:369:25: note: in expansion of macro 'this_cpu_write_1'
369 | case 1: stem##1(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
>> arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:370:25: note: in expansion of macro 'this_cpu_write_2'
370 | case 2: stem##2(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
>> arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
arch/x86/include/asm/percpu.h:145:69: note: in definition of macro '__raw_cpu_write'
145 | *(qual __my_cpu_type(pcp) * __force)__my_cpu_ptr(&(pcp)) = (val); \
| ^~~
include/linux/percpu-defs.h:371:25: note: in expansion of macro 'this_cpu_write_4'
371 | case 4: stem##4(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
In file included from arch/x86/include/asm/percpu.h:598:
>> arch/x86/include/asm/tlbflush.h:468:46: warning: right shift count >= width of type [-Wshift-count-overflow]
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~
include/asm-generic/percpu.h:87:33: note: in definition of macro 'raw_cpu_generic_to_op'
87 | *raw_cpu_ptr(&(pcp)) op val; \
| ^~~
include/asm-generic/percpu.h:412:41: note: in expansion of macro 'this_cpu_generic_to_op'
412 | #define this_cpu_write_8(pcp, val) this_cpu_generic_to_op(pcp, val, =)
| ^~~~~~~~~~~~~~~~~~~~~~
include/linux/percpu-defs.h:372:25: note: in expansion of macro 'this_cpu_write_8'
372 | case 8: stem##8(variable, __VA_ARGS__);break; \
| ^~~~
include/linux/percpu-defs.h:500:41: note: in expansion of macro '__pcpu_size_call'
500 | #define this_cpu_write(pcp, val) __pcpu_size_call(this_cpu_write_, pcp, val)
| ^~~~~~~~~~~~~~~~
arch/x86/include/asm/tlbflush.h:468:9: note: in expansion of macro 'this_cpu_write'
468 | this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
| ^~~~~~~~~~~~~~
In file included from include/linux/kasan.h:38,
from include/linux/slab.h:264,
from include/linux/fs.h:45,
from include/linux/compat.h:17,
from arch/x86/include/asm/ia32.h:7,
from arch/x86/include/asm/elf.h:10,
from include/linux/elf.h:6,
from include/linux/module.h:20,
from include/linux/device/driver.h:21,
from include/linux/device.h:32,
from include/linux/platform_device.h:13,
from include/linux/clocksource.h:21,
from include/clocksource/hyperv_timer.h:16,
from arch/x86/include/asm/vdso/gettimeofday.h:20,
from include/vdso/datapage.h:196,
from arch/x86/include/uapi/../../../../lib/vdso/gettimeofday.c:6,
from arch/x86/entry/vdso/vdso32/../common/vclock_gettime.c:16:
include/linux/pgtable.h: At top level:
include/linux/pgtable.h:2204:2: error: #error Missing MAX_POSSIBLE_PHYSMEM_BITS definition
2204 | #error Missing MAX_POSSIBLE_PHYSMEM_BITS definition
| ^~~~~
arch/x86/include/asm/ia32.h:28:8: error: redefinition of 'struct stat64'
28 | struct stat64 {
| ^~~~~~
In file included from include/linux/stat.h:6,
from include/linux/sysfs.h:22,
from include/linux/kobject.h:20,
from include/linux/of.h:18,
from include/linux/clocksource.h:19:
arch/x86/include/uapi/asm/stat.h:42:8: note: originally defined here
42 | struct stat64 {
| ^~~~~~
vim +468 arch/x86/include/asm/tlbflush.h
82721d8b25d76c Kiryl Shutsemau 2023-03-12 465
ec225f8c255fd0 Yosry Ahmed 2024-07-02 466 static inline void cpu_tlbstate_update_lam(unsigned long lam, u64 untag_mask)
82721d8b25d76c Kiryl Shutsemau 2023-03-12 467 {
ec225f8c255fd0 Yosry Ahmed 2024-07-02 @468 this_cpu_write(cpu_tlbstate.lam, lam >> X86_CR3_LAM_U57_BIT);
ec225f8c255fd0 Yosry Ahmed 2024-07-02 469 this_cpu_write(tlbstate_untag_mask, untag_mask);
82721d8b25d76c Kiryl Shutsemau 2023-03-12 470 }
82721d8b25d76c Kiryl Shutsemau 2023-03-12 471
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
© 2016 - 2026 Red Hat, Inc.