[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202409040112.BKj8VlQT-lkp@intel.com>
Date: Wed, 4 Sep 2024 01:19:47 +0800
From: kernel test robot <lkp@...el.com>
To: Yang Jihong <yangjihong@...edance.com>, irogers@...gle.com,
acme@...hat.com, namhyung@...nel.org,
linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Cc: oe-kbuild-all@...ts.linux.dev, yangjihong@...edance.com
Subject: Re: [PATCH] tools lib subcmd: Use array data to save built usage
string
Hi Yang,
kernel test robot noticed the following build errors:
[auto build test ERROR on linus/master]
[also build test ERROR on v6.11-rc6 next-20240903]
[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/Yang-Jihong/tools-lib-subcmd-Use-array-data-to-save-built-usage-string/20240903-164426
base: linus/master
patch link: https://lore.kernel.org/r/20240903084048.766892-1-yangjihong%40bytedance.com
patch subject: [PATCH] tools lib subcmd: Use array data to save built usage string
config: x86_64-randconfig-101-20240903 (https://download.01.org/0day-ci/archive/20240904/202409040112.BKj8VlQT-lkp@intel.com/config)
compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240904/202409040112.BKj8VlQT-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409040112.BKj8VlQT-lkp@intel.com/
All errors (new ones prefixed by >>):
scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]
scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples
/usr/bin/ld: tools/objtool/libsubcmd/libsubcmd.a(libsubcmd-in.o): in function `parse_options_subcommand':
>> tools/lib/subcmd/parse-options.c:643: undefined reference to `scnprintf'
>> /usr/bin/ld: tools/lib/subcmd/parse-options.c:647: undefined reference to `scnprintf'
>> /usr/bin/ld: tools/lib/subcmd/parse-options.c:647: undefined reference to `scnprintf'
/usr/bin/ld: tools/lib/subcmd/parse-options.c:651: undefined reference to `scnprintf'
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[4]: *** [Makefile:75: tools/objtool/objtool] Error 1
make[3]: *** [Makefile:72: objtool] Error 2
make[2]: *** [Makefile:1360: tools/objtool] Error 2
make[2]: Target 'prepare' not remade because of errors.
make[1]: *** [Makefile:224: __sub-make] Error 2
make[1]: Target 'prepare' not remade because of errors.
make: *** [Makefile:224: __sub-make] Error 2
make: Target 'prepare' not remade because of errors.
vim +643 tools/lib/subcmd/parse-options.c
631
632 int parse_options_subcommand(int argc, const char **argv, const struct option *options,
633 const char *const subcommands[], const char *usagestr[], int flags)
634 {
635 struct parse_opt_ctx_t ctx;
636
637 /* build usage string if it's not provided */
638 if (subcommands && !usagestr[0]) {
639 int n;
640 static char buf[USAGESTR_BUF_SIZE];
641 int buf_size = sizeof(buf);
642
> 643 n = scnprintf(buf, buf_size, "%s %s [<options>] {",
644 subcmd_config.exec_name, argv[0]);
645
646 for (int i = 0; subcommands[i] && n < buf_size - 1; i++) {
> 647 n += scnprintf(buf + n, buf_size - n, "%s%s",
648 i ? "|" : "", subcommands[i]);
649 }
650 if (n < buf_size - 1)
651 n += scnprintf(buf + n, buf_size - n, "}");
652
653 /* only provided if a complete string is built */
654 if (n < buf_size - 1)
655 usagestr[0] = buf;
656 }
657
658 parse_options_start(&ctx, argc, argv, flags);
659 switch (parse_options_step(&ctx, options, usagestr)) {
660 case PARSE_OPT_HELP:
661 exit(129);
662 case PARSE_OPT_DONE:
663 break;
664 case PARSE_OPT_LIST_OPTS:
665 while (options->type != OPTION_END) {
666 if (options->long_name)
667 printf("--%s ", options->long_name);
668 options++;
669 }
670 putchar('\n');
671 exit(130);
672 case PARSE_OPT_LIST_SUBCMDS:
673 if (subcommands) {
674 for (int i = 0; subcommands[i]; i++)
675 printf("%s ", subcommands[i]);
676 }
677 putchar('\n');
678 exit(130);
679 default: /* PARSE_OPT_UNKNOWN */
680 if (ctx.argv[0][1] == '-')
681 astrcatf(&error_buf, "unknown option `%s'",
682 ctx.argv[0] + 2);
683 else
684 astrcatf(&error_buf, "unknown switch `%c'", *ctx.opt);
685 usage_with_options(usagestr, options);
686 }
687
688 return parse_options_end(&ctx);
689 }
690
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists