* [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
@ 2021-12-04 5:32 kernel test robot
2021-12-04 13:02 ` Marco Elver
0 siblings, 1 reply; 4+ messages in thread
From: kernel test robot @ 2021-12-04 5:32 UTC (permalink / raw)
To: Marco Elver; +Cc: kbuild-all, linux-kernel, Paul E. McKenney
tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2021.12.02a
head: d157b802f05bd12cf40bef7a73ca6914b85c865e
commit: 31a060491205cae6f710fa65d118580ee680619f [31/43] locking/barriers, kcsan: Add instrumentation for barriers
config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20211204/202112041334.X44uWZXf-lkp@intel.com/config)
compiler: alpha-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git/commit/?id=31a060491205cae6f710fa65d118580ee680619f
git remote add paulmck-rcu https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
git fetch --no-tags paulmck-rcu dev.2021.12.02a
git checkout 31a060491205cae6f710fa65d118580ee680619f
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=alpha prepare
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
arch/alpha/kernel/asm-offsets.c:15:6: warning: no previous prototype for 'foo' [-Wmissing-prototypes]
15 | void foo(void)
| ^~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/asm-generic/bug.h:5,
from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/linux/thread_info.h:13,
from include/asm-generic/current.h:5,
from ./arch/alpha/include/generated/asm/current.h:1,
from include/linux/sched.h:12,
from arch/alpha/kernel/asm-offsets.c:10:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/init.h:5,
from drivers/gpu/drm/drm_dp_aux_bus.c:15:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/asm-generic/bug.h:5,
from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/linux/mmdebug.h:5,
from include/linux/gfp.h:5,
from include/linux/slab.h:15,
from drivers/gpu/drm/nouveau/include/nvif/os.h:6,
from drivers/gpu/drm/nouveau/include/nvif/object.h:4,
from drivers/gpu/drm/nouveau/include/nvif/client.h:5,
from drivers/gpu/drm/nouveau/nvif/client.c:25:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
drivers/gpu/drm/nouveau/nvif/client.c: In function 'nvif_client_ctor':
drivers/gpu/drm/nouveau/nvif/client.c:72:9: warning: 'strncpy' specified bound 32 equals destination size [-Wstringop-truncation]
72 | strncpy(args.name, name, sizeof(args.name));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/asm-generic/bug.h:5,
from arch/alpha/include/asm/bug.h:23,
from include/linux/bug.h:5,
from include/linux/mmdebug.h:5,
from include/linux/gfp.h:5,
from include/linux/slab.h:15,
from drivers/gpu/drm/nouveau/include/nvif/os.h:6,
from drivers/gpu/drm/nouveau/include/nvkm/core/os.h:4,
from drivers/gpu/drm/nouveau/include/nvkm/core/oclass.h:3,
from drivers/gpu/drm/nouveau/include/nvkm/core/device.h:4,
from drivers/gpu/drm/nouveau/nvkm/engine/device/priv.h:4,
from drivers/gpu/drm/nouveau/nvkm/engine/device/user.c:25:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
drivers/gpu/drm/nouveau/nvkm/engine/device/user.c: In function 'nvkm_udevice_info':
drivers/gpu/drm/nouveau/nvkm/engine/device/user.c:164:9: warning: 'strncpy' specified bound 16 equals destination size [-Wstringop-truncation]
164 | strncpy(args->v0.chip, device->chip->name, sizeof(args->v0.chip));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/nouveau/nvkm/engine/device/user.c:165:9: warning: 'strncpy' specified bound 64 equals destination size [-Wstringop-truncation]
165 | strncpy(args->v0.name, device->name, sizeof(args->v0.name));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
drivers/gpu/drm/nouveau/nouveau_display.c: In function 'nouveau_framebuffer_new':
drivers/gpu/drm/nouveau/nouveau_display.c:309:22: warning: variable 'width' set but not used [-Wunused-but-set-variable]
309 | unsigned int width, height, i;
| ^~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/build_bug.h:5,
from include/linux/bits.h:22,
from include/linux/ioport.h:13,
from include/linux/acpi.h:12,
from include/linux/i2c.h:13,
from include/drm/drm_crtc.h:28,
from include/drm/drm_atomic.h:31,
from drivers/gpu/drm/nouveau/nouveau_display.c:29:
drivers/gpu/drm/nouveau/nouveau_display.c: At top level:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
drivers/gpu/drm/nouveau/dispnv50/disp.c: In function 'nv50_mstm_cleanup':
drivers/gpu/drm/nouveau/dispnv50/disp.c:1390:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
1390 | int ret;
| ^~~
drivers/gpu/drm/nouveau/dispnv50/disp.c: In function 'nv50_mstm_prepare':
drivers/gpu/drm/nouveau/dispnv50/disp.c:1414:13: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
1414 | int ret;
| ^~~
drivers/gpu/drm/nouveau/dispnv50/disp.c: At top level:
drivers/gpu/drm/nouveau/dispnv50/disp.c:2692:1: warning: no previous prototype for 'nv50_display_create' [-Wmissing-prototypes]
2692 | nv50_display_create(struct drm_device *dev)
| ^~~~~~~~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/build_bug.h:5,
from include/linux/container_of.h:5,
from include/linux/list.h:5,
from include/linux/timer.h:5,
from include/linux/workqueue.h:9,
from drivers/gpu/drm/nouveau/dispnv50/disp.h:3,
from drivers/gpu/drm/nouveau/dispnv50/disp.c:24:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
drivers/gpu/drm/nouveau/dispnv50/headc57d.c:173:1: warning: no previous prototype for 'headc57d_olut' [-Wmissing-prototypes]
173 | headc57d_olut(struct nv50_head *head, struct nv50_head_atom *asyh, int size)
| ^~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/build_bug.h:5,
from include/linux/container_of.h:5,
from include/linux/list.h:5,
from include/linux/timer.h:5,
from include/linux/workqueue.h:9,
from drivers/gpu/drm/nouveau/dispnv50/head.h:4,
from drivers/gpu/drm/nouveau/dispnv50/headc57d.c:22:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:870:6: warning: no previous prototype for 'amdgpu_ras_mca_query_error_status' [-Wmissing-prototypes]
870 | void amdgpu_ras_mca_query_error_status(struct amdgpu_device *adev,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/build_bug.h:5,
from include/linux/container_of.h:5,
from include/linux/list.h:5,
from include/linux/wait.h:7,
from include/linux/wait_bit.h:8,
from include/linux/fs.h:6,
from include/linux/debugfs.h:15,
from drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:24:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c:521:6: warning: no previous prototype for 'release_psp_cmd_buf' [-Wmissing-prototypes]
521 | void release_psp_cmd_buf(struct psp_context *psp)
| ^~~~~~~~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/firmware.h:6,
from drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c:26:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c:399:5: warning: no previous prototype for 'amdgpu_vkms_output_init' [-Wmissing-prototypes]
399 | int amdgpu_vkms_output_init(struct drm_device *dev,
| ^~~~~~~~~~~~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/build_bug.h:5,
from include/linux/bits.h:22,
from include/linux/ioport.h:13,
from include/linux/acpi.h:12,
from include/linux/i2c.h:13,
from include/drm/drm_crtc.h:28,
from include/drm/drm_atomic_helper.h:31,
from drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c:3:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
--
In file included from drivers/gpu/drm/amd/amdgpu/../display/dmub/dmub_srv.h:67,
from drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:35:
drivers/gpu/drm/amd/amdgpu/../display/dmub/inc/dmub_cmd.h: In function 'dmub_rb_flush_pending':
drivers/gpu/drm/amd/amdgpu/../display/dmub/inc/dmub_cmd.h:2893:26: warning: variable 'temp' set but not used [-Wunused-but-set-variable]
2893 | uint64_t temp;
| ^~~~
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: At top level:
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:11459:5: warning: no previous prototype for 'amdgpu_dm_set_dmub_async_sync_status' [-Wmissing-prototypes]
11459 | int amdgpu_dm_set_dmub_async_sync_status(bool is_cmd_aux, struct dc_context *ctx,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from arch/alpha/include/asm/barrier.h:21,
from arch/alpha/include/asm/rwonce.h:10,
from include/linux/compiler.h:266,
from include/linux/init.h:5,
from include/linux/kgdb.h:17,
from drivers/gpu/drm/amd/amdgpu/../display/dc/os_types.h:30,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services_types.h:29,
from drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:29:
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
303 | smp_rmb(); /* see above */
| ^~~~~~~
>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
| ^~~~~~~~~
arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
293 | smp_rmb(); /* see above */
| ^~~~~~~
..
vim +70 include/asm-generic/barrier.h
68
69 #ifndef smp_rmb
> 70 #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
71 #endif
72
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
2021-12-04 5:32 [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static kernel test robot
@ 2021-12-04 13:02 ` Marco Elver
2021-12-07 0:32 ` [kbuild-all] " Rong Chen
0 siblings, 1 reply; 4+ messages in thread
From: Marco Elver @ 2021-12-04 13:02 UTC (permalink / raw)
To: kernel test robot; +Cc: kbuild-all, linux-kernel, Paul E. McKenney
On Sat, 4 Dec 2021 at 06:33, kernel test robot <lkp@intel.com> wrote:
[...]
> >> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
> 70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
> | ^~~~~~~~~
> arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
> 303 | smp_rmb(); /* see above */
> | ^~~~~~~
> >> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
> 70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
> | ^~~~~~~~~
> arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
> 293 | smp_rmb(); /* see above */
> | ^~~~~~~
I had been wondering if barriers could be static __always_inline, like
we do for atomic_t operations. Apparently this answers that questions.
This should fix it:
https://lkml.kernel.org/r/20211204125703.3344454-1-elver@google.com
Unrelatedly, I wonder if I can get the test robot to also check some
of my branches:
https://git.kernel.org/pub/scm/linux/kernel/git/melver/linux.git/
That'd be very useful, to catch things like this early, and avoid
disturbing Paul too much. :-)
Thanks,
-- Marco
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [kbuild-all] Re: [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
2021-12-04 13:02 ` Marco Elver
@ 2021-12-07 0:32 ` Rong Chen
2021-12-07 6:11 ` Marco Elver
0 siblings, 1 reply; 4+ messages in thread
From: Rong Chen @ 2021-12-07 0:32 UTC (permalink / raw)
To: Marco Elver, kernel test robot; +Cc: kbuild-all, linux-kernel, Paul E. McKenney
On 12/4/21 21:02, Marco Elver wrote:
> On Sat, 4 Dec 2021 at 06:33, kernel test robot <lkp@intel.com> wrote:
> [...]
>>>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
>> 70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
>> | ^~~~~~~~~
>> arch/alpha/include/asm/pgtable.h:303:9: note: in expansion of macro 'smp_rmb'
>> 303 | smp_rmb(); /* see above */
>> | ^~~~~~~
>>>> include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pmd_offset' which is not static
>> 70 | #define smp_rmb() do { kcsan_rmb(); __smp_rmb(); } while (0)
>> | ^~~~~~~~~
>> arch/alpha/include/asm/pgtable.h:293:9: note: in expansion of macro 'smp_rmb'
>> 293 | smp_rmb(); /* see above */
>> | ^~~~~~~
> I had been wondering if barriers could be static __always_inline, like
> we do for atomic_t operations. Apparently this answers that questions.
>
> This should fix it:
> https://lkml.kernel.org/r/20211204125703.3344454-1-elver@google.com
>
> Unrelatedly, I wonder if I can get the test robot to also check some
> of my branches:
Hi Marco,
We have added it into our monitor list.
Best Regards,
Rong Chen
> https://git.kernel.org/pub/scm/linux/kernel/git/melver/linux.git/
> That'd be very useful, to catch things like this early, and avoid
> disturbing Paul too much. :-)
>
> Thanks,
> -- Marco
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [kbuild-all] Re: [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static
2021-12-07 0:32 ` [kbuild-all] " Rong Chen
@ 2021-12-07 6:11 ` Marco Elver
0 siblings, 0 replies; 4+ messages in thread
From: Marco Elver @ 2021-12-07 6:11 UTC (permalink / raw)
To: Rong Chen; +Cc: kernel test robot, kbuild-all, linux-kernel, Paul E. McKenney
On Tue, 7 Dec 2021 at 01:32, Rong Chen <rong.a.chen@intel.com> wrote:
[...]
> Hi Marco,
>
> We have added it into our monitor list.
>
> Best Regards,
> Rong Chen
Great, thank you very much!
-- Marco
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-12-07 6:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-04 5:32 [paulmck-rcu:dev.2021.12.02a 31/43] include/asm-generic/barrier.h:70:30: warning: 'kcsan_rmb' is static but used in inline function 'pte_offset_kernel' which is not static kernel test robot
2021-12-04 13:02 ` Marco Elver
2021-12-07 0:32 ` [kbuild-all] " Rong Chen
2021-12-07 6:11 ` Marco Elver
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).