lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date: Tue, 26 Dec 2023 05:59:15 +0800
From: kernel test robot <lkp@...el.com>
To: Matthew Wilcox <willy@...radead.org>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
	Jonathan Corbet <corbet@....net>
Subject: arch/x86/kernel/uprobes.c:758: warning: Function parameter or member
 'auprobe' not described in 'arch_uprobe_analyze_insn'

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   861deac3b092f37b2c5e6871732f3e11486f7082
commit: 3a025e1d1c2ea42fa497c9c6b21c284e0f69e28b Add optional check for bad kernel-doc comments
date:   6 years ago
config: i386-defconfig (https://download.01.org/0day-ci/archive/20231226/202312260514.MC64urpm-lkp@intel.com/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231226/202312260514.MC64urpm-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/202312260514.MC64urpm-lkp@intel.com/

All warnings (new ones prefixed by >>):

   arch/x86/kernel/uprobes.c: In function 'branch_setup_xol_ops':
   arch/x86/kernel/uprobes.c:726:8: warning: this statement may fall through [-Wimplicit-fallthrough=]
      opc1 = OPCODE2(insn) - 0x10;
      ~~~~~^~~~~~~~~~~~~~~~~~~~~~
   arch/x86/kernel/uprobes.c:727:2: note: here
     default:
     ^~~~~~~
>> arch/x86/kernel/uprobes.c:758: warning: Function parameter or member 'auprobe' not described in 'arch_uprobe_analyze_insn'
>> arch/x86/kernel/uprobes.c:758: warning: Excess function parameter 'arch_uprobe' description in 'arch_uprobe_analyze_insn'
--
>> arch/x86/pci/i386.c:372: warning: Function parameter or member 'pcibios_assign_resources' not described in 'fs_initcall'
--
>> arch/x86/power/cpu.c:76: warning: Function parameter or member 'ctxt' not described in '__save_processor_state'
>> arch/x86/power/cpu.c:198: warning: Function parameter or member 'ctxt' not described in '__restore_processor_state'
--
>> crypto/asymmetric_keys/asymmetric_type.c:160: warning: Function parameter or member 'kid1' not described in 'asymmetric_key_id_same'
>> crypto/asymmetric_keys/asymmetric_type.c:160: warning: Function parameter or member 'kid2' not described in 'asymmetric_key_id_same'
>> crypto/asymmetric_keys/asymmetric_type.c:176: warning: Function parameter or member 'kid1' not described in 'asymmetric_key_id_partial'
>> crypto/asymmetric_keys/asymmetric_type.c:176: warning: Function parameter or member 'kid2' not described in 'asymmetric_key_id_partial'
--
>> crypto/asymmetric_keys/pkcs7_trust.c:29: warning: Function parameter or member 'pkcs7' not described in 'pkcs7_validate_trust_one'
>> crypto/asymmetric_keys/pkcs7_trust.c:29: warning: Function parameter or member 'sinfo' not described in 'pkcs7_validate_trust_one'
>> crypto/asymmetric_keys/pkcs7_trust.c:29: warning: Function parameter or member 'trust_keyring' not described in 'pkcs7_validate_trust_one'
--
   drivers/acpi/osl.c: In function 'acpi_os_vprintf':
   drivers/acpi/osl.c:160:2: warning: function 'acpi_os_vprintf' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
     vsprintf(buffer, fmt, args);
     ^~~~~~~~
>> drivers/acpi/osl.c:315: warning: cannot understand function prototype: 'void __iomem *__ref acpi_os_map_iomem(acpi_physical_address phys, acpi_size size) '
--
>> drivers/acpi/utils.c:474: warning: Function parameter or member 'handle' not described in 'acpi_handle_path'
>> drivers/acpi/utils.c:496: warning: Function parameter or member 'level' not described in 'acpi_handle_printk'
>> drivers/acpi/utils.c:496: warning: Function parameter or member 'handle' not described in 'acpi_handle_printk'
>> drivers/acpi/utils.c:496: warning: Function parameter or member 'fmt' not described in 'acpi_handle_printk'
>> drivers/acpi/utils.c:525: warning: Function parameter or member 'descriptor' not described in '__acpi_handle_debug'
>> drivers/acpi/utils.c:525: warning: Function parameter or member 'handle' not described in '__acpi_handle_debug'
>> drivers/acpi/utils.c:525: warning: Function parameter or member 'fmt' not described in '__acpi_handle_debug'
--
>> drivers/acpi/nvs.c:95: warning: Function parameter or member 'start' not described in 'suspend_nvs_register'
>> drivers/acpi/nvs.c:95: warning: Function parameter or member 'size' not described in 'suspend_nvs_register'
--
>> drivers/acpi/device_sysfs.c:285: warning: Function parameter or member 'dev' not described in 'acpi_device_uevent_modalias'
>> drivers/acpi/device_sysfs.c:285: warning: Function parameter or member 'env' not described in 'acpi_device_uevent_modalias'
>> drivers/acpi/device_sysfs.c:330: warning: Function parameter or member 'dev' not described in 'acpi_device_modalias'
>> drivers/acpi/device_sysfs.c:330: warning: Function parameter or member 'buf' not described in 'acpi_device_modalias'
>> drivers/acpi/device_sysfs.c:330: warning: Function parameter or member 'size' not described in 'acpi_device_modalias'
--
>> drivers/acpi/pci_root.c:85: warning: Function parameter or member 'handle' not described in 'acpi_is_root_bridge'
--
>> drivers/acpi/acpi_apd.c:47: warning: Function parameter or member 'properties' not described in 'apd_device_desc'
>> drivers/acpi/acpi_apd.c:133: warning: Function parameter or member 'adev' not described in 'acpi_apd_create_device'
>> drivers/acpi/acpi_apd.c:133: warning: Function parameter or member 'id' not described in 'acpi_apd_create_device'
..


vim +758 arch/x86/kernel/uprobes.c

7ba6db2d688bdf Oleg Nesterov              2014-04-05  749  
2b144498350860 Srikar Dronamraju          2012-02-09  750  /**
0326f5a94ddea3 Srikar Dronamraju          2012-03-13  751   * arch_uprobe_analyze_insn - instruction analysis including validity and fixups.
2b144498350860 Srikar Dronamraju          2012-02-09  752   * @mm: the probed address space.
3ff54efdfaace9 Srikar Dronamraju          2012-02-22  753   * @arch_uprobe: the probepoint information.
7eb9ba5ed312ec Ananth N Mavinakayanahalli 2012-06-08  754   * @addr: virtual address at which to install the probepoint
2b144498350860 Srikar Dronamraju          2012-02-09  755   * Return 0 on success or a -ve number on error.
2b144498350860 Srikar Dronamraju          2012-02-09  756   */
7eb9ba5ed312ec Ananth N Mavinakayanahalli 2012-06-08  757  int arch_uprobe_analyze_insn(struct arch_uprobe *auprobe, struct mm_struct *mm, unsigned long addr)
2b144498350860 Srikar Dronamraju          2012-02-09 @758  {
2b144498350860 Srikar Dronamraju          2012-02-09  759  	struct insn insn;
83cd591485e558 Oleg Nesterov              2014-04-25  760  	u8 fix_ip_or_call = UPROBE_FIX_IP;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  761  	int ret;
2b144498350860 Srikar Dronamraju          2012-02-09  762  
2ae1f49ae1978f Oleg Nesterov              2014-04-19  763  	ret = uprobe_init_insn(auprobe, &insn, is_64bit_mm(mm));
ddb69f276c4af8 Oleg Nesterov              2014-03-31  764  	if (ret)
2b144498350860 Srikar Dronamraju          2012-02-09  765  		return ret;
7b2d81d48a2d8e Ingo Molnar                2012-02-17  766  
7ba6db2d688bdf Oleg Nesterov              2014-04-05  767  	ret = branch_setup_xol_ops(auprobe, &insn);
7ba6db2d688bdf Oleg Nesterov              2014-04-05  768  	if (ret != -ENOSYS)
7ba6db2d688bdf Oleg Nesterov              2014-04-05  769  		return ret;
7ba6db2d688bdf Oleg Nesterov              2014-04-05  770  
ddb69f276c4af8 Oleg Nesterov              2014-03-31  771  	/*
97aa5cddbe9e01 Oleg Nesterov              2014-04-22  772  	 * Figure out which fixups default_post_xol_op() will need to perform,
5cdb76d6f0b657 Oleg Nesterov              2014-06-01  773  	 * and annotate defparam->fixups accordingly.
ddb69f276c4af8 Oleg Nesterov              2014-03-31  774  	 */
ddb69f276c4af8 Oleg Nesterov              2014-03-31  775  	switch (OPCODE1(&insn)) {
ddb69f276c4af8 Oleg Nesterov              2014-03-31  776  	case 0x9d:		/* popf */
5cdb76d6f0b657 Oleg Nesterov              2014-06-01  777  		auprobe->defparam.fixups |= UPROBE_FIX_SETF;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  778  		break;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  779  	case 0xc3:		/* ret or lret -- ip is correct */
ddb69f276c4af8 Oleg Nesterov              2014-03-31  780  	case 0xcb:
ddb69f276c4af8 Oleg Nesterov              2014-03-31  781  	case 0xc2:
ddb69f276c4af8 Oleg Nesterov              2014-03-31  782  	case 0xca:
83cd591485e558 Oleg Nesterov              2014-04-25  783  	case 0xea:		/* jmp absolute -- ip is correct */
83cd591485e558 Oleg Nesterov              2014-04-25  784  		fix_ip_or_call = 0;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  785  		break;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  786  	case 0x9a:		/* call absolute - Fix return addr, not ip */
83cd591485e558 Oleg Nesterov              2014-04-25  787  		fix_ip_or_call = UPROBE_FIX_CALL;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  788  		break;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  789  	case 0xff:
ddb69f276c4af8 Oleg Nesterov              2014-03-31  790  		switch (MODRM_REG(&insn)) {
ddb69f276c4af8 Oleg Nesterov              2014-03-31  791  		case 2: case 3:			/* call or lcall, indirect */
83cd591485e558 Oleg Nesterov              2014-04-25  792  			fix_ip_or_call = UPROBE_FIX_CALL;
83cd591485e558 Oleg Nesterov              2014-04-25  793  			break;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  794  		case 4: case 5:			/* jmp or ljmp, indirect */
83cd591485e558 Oleg Nesterov              2014-04-25  795  			fix_ip_or_call = 0;
83cd591485e558 Oleg Nesterov              2014-04-25  796  			break;
ddb69f276c4af8 Oleg Nesterov              2014-03-31  797  		}
e55848a4f8ee52 Oleg Nesterov              2014-03-31  798  		/* fall through */
ddb69f276c4af8 Oleg Nesterov              2014-03-31  799  	default:
1475ee7fadafc6 Oleg Nesterov              2014-04-27  800  		riprel_analyze(auprobe, &insn);
ddb69f276c4af8 Oleg Nesterov              2014-03-31  801  	}
ddb69f276c4af8 Oleg Nesterov              2014-03-31  802  
5cdb76d6f0b657 Oleg Nesterov              2014-06-01  803  	auprobe->defparam.ilen = insn.length;
5cdb76d6f0b657 Oleg Nesterov              2014-06-01  804  	auprobe->defparam.fixups |= fix_ip_or_call;
7b2d81d48a2d8e Ingo Molnar                2012-02-17  805  
8ad8e9d3fd64f1 Oleg Nesterov              2014-03-31  806  	auprobe->ops = &default_xol_ops;
2b144498350860 Srikar Dronamraju          2012-02-09  807  	return 0;
2b144498350860 Srikar Dronamraju          2012-02-09  808  }
0326f5a94ddea3 Srikar Dronamraju          2012-03-13  809  

:::::: The code at line 758 was first introduced by commit
:::::: 2b144498350860b6ee9dc57ff27a93ad488de5dc uprobes, mm, x86: Add the ability to install and remove uprobes breakpoints

:::::: TO: Srikar Dronamraju <srikar@...ux.vnet.ibm.com>
:::::: CC: Ingo Molnar <mingo@...e.hu>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ