Hi Douglas, kernel test robot noticed the following build warnings:
[auto build test WARNING on v6.10-rc4] [also build test WARNING on linus/master next-20240618] [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/Douglas-Anderson/kdb-Get-rid-of-minlen-for-the-md-command/20240618-084245 base: v6.10-rc4 patch link: https://lore.kernel.org/r/20240617173426.11.I899d035485269f5110a3323fbb1680fbba718e4c%40changeid patch subject: [PATCH 11/13] kdb: Abstract out parsing for mdWcN config: arc-randconfig-001-20240619 (https://download.01.org/0day-ci/archive/20240619/202406190433.u3alc3xi-...@intel.com/config) compiler: arc-elf-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240619/202406190433.u3alc3xi-...@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 <l...@intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202406190433.u3alc3xi-...@intel.com/ All warnings (new ones prefixed by >>): >> kernel/debug/kdb/kdb_main.c:1606: warning: Function parameter or struct >> member 'bytesperword' not described in 'kdb_md_parse_arg0' vim +1606 kernel/debug/kdb/kdb_main.c 1593 1594 /** 1595 * kdb_md_parse_arg0() - Parse argv[0] for "md" command 1596 * 1597 * @cmd: The name of the command, like "md" 1598 * @arg0: The value of argv[0]. 1599 * @repeat: If argv0 modifies repeat count we'll adjust here. 1600 * @bytesperword Ifargv0 modifies bytesperword we'll adjust here. 1601 * 1602 * Return: true if this was a valid cmd; false otherwise. 1603 */ 1604 static bool kdb_md_parse_arg0(const char *cmd, const char *arg0, 1605 int *repeat, int *bytesperword) > 1606 { 1607 int cmdlen = strlen(cmd); 1608 1609 /* arg0 must _start_ with the command string or it's a no-go. */ 1610 if (strncmp(cmd, arg0, cmdlen) != 0) 1611 return false; 1612 1613 /* If it's just the base command, we're done and it's good. */ 1614 if (arg0[cmdlen] == '\0') 1615 return true; 1616 1617 /* 1618 * The first byte after the base command must be bytes per word, a 1619 * digit. The actual value of bytesperword will be validated later. 1620 */ 1621 if (!isdigit(arg0[cmdlen])) 1622 return false; 1623 *bytesperword = (int)(arg0[cmdlen] - '0'); 1624 cmdlen++; 1625 1626 /* After the bytes per word must be end of string or a 'c'. */ 1627 if (arg0[cmdlen] == '\0') 1628 return true; 1629 if (arg0[cmdlen] != 'c') 1630 return false; 1631 cmdlen++; 1632 1633 /* After the "c" is the repeat. */ 1634 return kstrtouint(arg0 + cmdlen, 10, repeat) == 0; 1635 } 1636 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki _______________________________________________ Kgdb-bugreport mailing list Kgdb-bugreport@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
