From: kernel test robot <lkp@intel.com>
To: wen.yang@linux.dev
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH v5 2/2] sysctl: convert kernel/sysctl-test.c to use CTLTBL_ENTRY_XXX()
Date: Tue, 31 Mar 2026 03:52:33 +0800 [thread overview]
Message-ID: <202603310301.Ogh6kR6N-lkp@intel.com> (raw)
In-Reply-To: <d2589e4a49044b0d857ee1b9da495ca73d19ae57.1774463505.git.wen.yang@linux.dev>
Hi,
[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:
[auto build test WARNING on sysctl/sysctl-next]
[also build test WARNING on kees/for-next/pstore kees/for-next/kspp linus/master v7.0-rc6 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/wen-yang-linux-dev/sysctl-introduce-CTLTBL_ENTRY_XXX-helper-macros/20260330-010040
base: https://git.kernel.org/pub/scm/linux/kernel/git/sysctl/sysctl.git sysctl-next
patch link: https://lore.kernel.org/r/d2589e4a49044b0d857ee1b9da495ca73d19ae57.1774463505.git.wen.yang%40linux.dev
patch subject: [RFC PATCH v5 2/2] sysctl: convert kernel/sysctl-test.c to use CTLTBL_ENTRY_XXX()
config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20260331/202603310301.Ogh6kR6N-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260331/202603310301.Ogh6kR6N-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/202603310301.Ogh6kR6N-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> kernel/sysctl-test.c:386:12: warning: implicit conversion from 'size_t' (aka 'unsigned long') to 'int' changes value from 18446744073709551615 to -1 [-Wconstant-conversion]
386 | .table = CTLTBL_ENTRY_VNMH(ctltbl_str, "foo", 0644,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
387 | proc_dostring),
| ~~~~~~~~~~~~~~
include/linux/sysctl.h:413:2: note: expanded from macro 'CTLTBL_ENTRY_VNMH'
413 | __CTLTBL_ENTRY(__var, __name, __mode, __handler, NULL, NULL, -1)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:316:18: note: expanded from macro '__CTLTBL_ENTRY'
312 | #define __CTLTBL_ENTRY(_var, _name, _mode, _handler, _min, _max, _maxlen) \
| ~
313 | { \
314 | .procname = __CTL_PROCNAME(_name), \
315 | .data = __CTL_DATA(_var), \
316 | .maxlen = __CTL_MAXLEN(_var, _maxlen), \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:277:5: note: expanded from macro '__CTL_MAXLEN'
277 | __SYSCTL_MAXLEN_VAR(__var, __maxlen) \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:264:21: note: expanded from macro '__SYSCTL_MAXLEN_VAR'
264 | ((__maxlen) >= 0 ? (size_t)(__maxlen) : (size_t)sizeof(__var))
| ^~~~~~~~~~~~~~~~~~
kernel/sysctl-test.c:376:12: warning: implicit conversion from 'size_t' (aka 'unsigned long') to 'int' changes value from 18446744073709551615 to -1 [-Wconstant-conversion]
376 | .table = CTLTBL_ENTRY_V(ctltbl_bool),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:347:2: note: expanded from macro 'CTLTBL_ENTRY_V'
347 | __CTLTBL_ENTRY(__var, #__var, 0444, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
348 | __CTL_AUTO_HANDLER(__var), NULL, NULL, -1)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:316:18: note: expanded from macro '__CTLTBL_ENTRY'
312 | #define __CTLTBL_ENTRY(_var, _name, _mode, _handler, _min, _max, _maxlen) \
| ~
313 | { \
314 | .procname = __CTL_PROCNAME(_name), \
315 | .data = __CTL_DATA(_var), \
316 | .maxlen = __CTL_MAXLEN(_var, _maxlen), \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:277:5: note: expanded from macro '__CTL_MAXLEN'
277 | __SYSCTL_MAXLEN_VAR(__var, __maxlen) \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:264:21: note: expanded from macro '__SYSCTL_MAXLEN_VAR'
264 | ((__maxlen) >= 0 ? (size_t)(__maxlen) : (size_t)sizeof(__var))
| ^~~~~~~~~~~~~~~~~~
kernel/sysctl-test.c:367:12: warning: implicit conversion from 'size_t' (aka 'unsigned long') to 'int' changes value from 18446744073709551615 to -1 [-Wconstant-conversion]
367 | .table = CTLTBL_ENTRY_VMR(ctltbl_u8, 0644,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
368 | &ctltbl_u8_min, &ctltbl_u8_max),
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:373:2: note: expanded from macro 'CTLTBL_ENTRY_VMR'
373 | __CTLTBL_ENTRY(__var, #__var, __mode, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
374 | __CTL_AUTO_HANDLER_RANGE(__var), __min, __max, -1)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:316:18: note: expanded from macro '__CTLTBL_ENTRY'
312 | #define __CTLTBL_ENTRY(_var, _name, _mode, _handler, _min, _max, _maxlen) \
| ~
313 | { \
314 | .procname = __CTL_PROCNAME(_name), \
315 | .data = __CTL_DATA(_var), \
316 | .maxlen = __CTL_MAXLEN(_var, _maxlen), \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:277:5: note: expanded from macro '__CTL_MAXLEN'
277 | __SYSCTL_MAXLEN_VAR(__var, __maxlen) \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:264:21: note: expanded from macro '__SYSCTL_MAXLEN_VAR'
264 | ((__maxlen) >= 0 ? (size_t)(__maxlen) : (size_t)sizeof(__var))
| ^~~~~~~~~~~~~~~~~~
kernel/sysctl-test.c:360:12: warning: implicit conversion from 'size_t' (aka 'unsigned long') to 'int' changes value from 18446744073709551615 to -1 [-Wconstant-conversion]
360 | .table = CTLTBL_ENTRY_V(ctltbl_u8),
| ^~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:347:2: note: expanded from macro 'CTLTBL_ENTRY_V'
347 | __CTLTBL_ENTRY(__var, #__var, 0444, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
348 | __CTL_AUTO_HANDLER(__var), NULL, NULL, -1)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:316:18: note: expanded from macro '__CTLTBL_ENTRY'
312 | #define __CTLTBL_ENTRY(_var, _name, _mode, _handler, _min, _max, _maxlen) \
| ~
313 | { \
314 | .procname = __CTL_PROCNAME(_name), \
315 | .data = __CTL_DATA(_var), \
316 | .maxlen = __CTL_MAXLEN(_var, _maxlen), \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:277:5: note: expanded from macro '__CTL_MAXLEN'
277 | __SYSCTL_MAXLEN_VAR(__var, __maxlen) \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:264:21: note: expanded from macro '__SYSCTL_MAXLEN_VAR'
264 | ((__maxlen) >= 0 ? (size_t)(__maxlen) : (size_t)sizeof(__var))
| ^~~~~~~~~~~~~~~~~~
kernel/sysctl-test.c:352:12: warning: implicit conversion from 'size_t' (aka 'unsigned long') to 'int' changes value from 18446744073709551615 to -1 [-Wconstant-conversion]
352 | .table = CTLTBL_ENTRY_VNM(ctltbl_int, "my-sysctl", 0644),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:398:2: note: expanded from macro 'CTLTBL_ENTRY_VNM'
398 | __CTLTBL_ENTRY(__var, __name, __mode, \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
399 | __CTL_AUTO_HANDLER(__var), NULL, NULL, -1)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/sysctl.h:316:18: note: expanded from macro '__CTLTBL_ENTRY'
312 | #define __CTLTBL_ENTRY(_var, _name, _mode, _handler, _min, _max, _maxlen) \
| ~
vim +386 kernel/sysctl-test.c
318
319 static const struct ctltbl_param ctltbl_cases[] = {
320 /* auto handler — int */
321 {
322 .desc = "sysctl_test_api_ctltbl_entry_V_int",
323 .table = CTLTBL_ENTRY_V(ctltbl_int),
324 .expected = { .procname = "ctltbl_int", .mode = 0444,
325 .data = &ctltbl_int, .maxlen = sizeof(int),
326 .proc_handler = proc_dointvec },
327 },
328 {
329 .desc = "sysctl_test_api_ctltbl_entry_VM_int",
330 .table = CTLTBL_ENTRY_VM(ctltbl_int, 0644),
331 .expected = { .procname = "ctltbl_int", .mode = 0644,
332 .data = &ctltbl_int, .maxlen = sizeof(int),
333 .proc_handler = proc_dointvec },
334 },
335 {
336 .desc = "sysctl_test_api_ctltbl_entry_VMR_int",
337 .table = CTLTBL_ENTRY_VMR(ctltbl_int, 0644,
338 SYSCTL_ZERO, SYSCTL_ONE_HUNDRED),
339 .expected = { .procname = "ctltbl_int", .mode = 0644,
340 .data = &ctltbl_int, .maxlen = sizeof(int),
341 .proc_handler = proc_dointvec_minmax },
342 },
343 {
344 .desc = "sysctl_test_api_ctltbl_entry_VN_int",
345 .table = CTLTBL_ENTRY_VN(ctltbl_int, "my-sysctl"),
346 .expected = { .procname = "my-sysctl", .mode = 0444,
347 .data = &ctltbl_int, .maxlen = sizeof(int),
348 .proc_handler = proc_dointvec },
349 },
350 {
351 .desc = "sysctl_test_api_ctltbl_entry_VNM_int",
352 .table = CTLTBL_ENTRY_VNM(ctltbl_int, "my-sysctl", 0644),
353 .expected = { .procname = "my-sysctl", .mode = 0644,
354 .data = &ctltbl_int, .maxlen = sizeof(int),
355 .proc_handler = proc_dointvec },
356 },
357 /* auto handler — u8 */
358 {
359 .desc = "sysctl_test_api_ctltbl_entry_V_u8",
360 .table = CTLTBL_ENTRY_V(ctltbl_u8),
361 .expected = { .procname = "ctltbl_u8", .mode = 0444,
362 .data = &ctltbl_u8, .maxlen = sizeof(u8),
363 .proc_handler = proc_dou8vec_minmax },
364 },
365 {
366 .desc = "sysctl_test_api_ctltbl_entry_VMR_u8",
367 .table = CTLTBL_ENTRY_VMR(ctltbl_u8, 0644,
368 &ctltbl_u8_min, &ctltbl_u8_max),
369 .expected = { .procname = "ctltbl_u8", .mode = 0644,
370 .data = &ctltbl_u8, .maxlen = sizeof(u8),
371 .proc_handler = proc_dou8vec_minmax },
372 },
373 /* auto handler — bool */
374 {
375 .desc = "sysctl_test_api_ctltbl_entry_V_bool",
376 .table = CTLTBL_ENTRY_V(ctltbl_bool),
377 .expected = { .procname = "ctltbl_bool", .mode = 0444,
378 .data = &ctltbl_bool, .maxlen = sizeof(bool),
379 .proc_handler = proc_dobool },
380 },
381 /* VNMH — char[] cannot be auto-dispatched by _Generic (each char[N]
382 * is a distinct type); explicit handler is required for strings.
383 * .maxlen must be sizeof(array), not sizeof(char *). */
384 {
385 .desc = "sysctl_test_api_ctltbl_entry_VNMH_string",
> 386 .table = CTLTBL_ENTRY_VNMH(ctltbl_str, "foo", 0644,
387 proc_dostring),
388 .expected = { .procname = "foo", .mode = 0644,
389 .data = ctltbl_str, .maxlen = sizeof(ctltbl_str),
390 .proc_handler = proc_dostring },
391 },
392 };
393
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
parent reply other threads:[~2026-03-30 19:52 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <d2589e4a49044b0d857ee1b9da495ca73d19ae57.1774463505.git.wen.yang@linux.dev>]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202603310301.Ogh6kR6N-lkp@intel.com \
--to=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=wen.yang@linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox