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>] [thread-next>] [day] [month] [year] [list]
Message-ID: <201803200516.SWlP9b4m%fengguang.wu@intel.com>
Date:   Tue, 20 Mar 2018 05:53:21 +0800
From:   kbuild test robot <fengguang.wu@...el.com>
To:     Palmer Dabbelt <palmer@...ive.com>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org,
        tipbuild@...or.com, Thomas Gleixner <tglx@...utronix.de>
Subject: [tip:irq/core 12/13] drivers/irqchip/irq-vic.c:295:2: error:
 implicit declaration of function 'set_handle_irq'; did you mean
 'vic_handle_irq'?

Hi Palmer,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git irq/core
head:   ba63053f018721a0ad60b083e21a15e324a4a8c4
commit: 886d70aac148f6015ae3d1379c81b98af3e70981 [12/13] ARM: irq: Convert to GENERIC_IRQ_MULTI_HANDLER
config: arm-netx_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 886d70aac148f6015ae3d1379c81b98af3e70981
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All errors (new ones prefixed by >>):

   drivers/irqchip/irq-vic.c: In function 'vic_register':
>> drivers/irqchip/irq-vic.c:295:2: error: implicit declaration of function 'set_handle_irq'; did you mean 'vic_handle_irq'? [-Werror=implicit-function-declaration]
     set_handle_irq(vic_handle_irq);
     ^~~~~~~~~~~~~~
     vic_handle_irq
   cc1: some warnings being treated as errors

vim +295 drivers/irqchip/irq-vic.c

ce94df9c0 arch/arm/common/vic.c     Linus Walleij   2012-04-20  262  
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  263  /**
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  264   * vic_register() - Register a VIC.
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  265   * @base: The base address of the VIC.
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  266   * @parent_irq: The parent IRQ if cascaded, else 0.
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  267   * @irq: The base IRQ for the VIC.
fa943bed6 arch/arm/common/vic.c     Linus Walleij   2012-04-20  268   * @valid_sources: bitmask of valid interrupts
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  269   * @resume_sources: bitmask of interrupts allowed for resume sources.
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  270   * @node: The device tree node associated with the VIC.
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  271   *
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  272   * Register the VIC with the system device tree so that it can be notified
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  273   * of suspend and resume requests and ensure that the correct actions are
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  274   * taken to re-instate the settings on resume.
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  275   *
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  276   * This also configures the IRQ domain for the VIC.
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  277   */
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  278  static void __init vic_register(void __iomem *base, unsigned int parent_irq,
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  279  				unsigned int irq,
fa943bed6 arch/arm/common/vic.c     Linus Walleij   2012-04-20  280  				u32 valid_sources, u32 resume_sources,
fa943bed6 arch/arm/common/vic.c     Linus Walleij   2012-04-20  281  				struct device_node *node)
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  282  {
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  283  	struct vic_device *v;
5ced33bc0 arch/arm/common/vic.c     Linus Walleij   2012-12-26  284  	int i;
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  285  
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  286  	if (vic_id >= ARRAY_SIZE(vic_devices)) {
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  287  		printk(KERN_ERR "%s: too few VICs, increase CONFIG_ARM_VIC_NR\n", __func__);
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  288  		return;
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  289  	}
f9b28ccbc arch/arm/common/vic.c     Jamie Iles      2011-09-27  290  
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  291  	v = &vic_devices[vic_id];
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  292  	v->base = base;
ce94df9c0 arch/arm/common/vic.c     Linus Walleij   2012-04-20  293  	v->valid_sources = valid_sources;
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  294  	v->resume_sources = resume_sources;
7fb7d8ae1 arch/arm/common/vic.c     Rob Herring     2012-11-20 @295  	set_handle_irq(vic_handle_irq);
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  296  	vic_id++;
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  297  
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  298  	if (parent_irq) {
9f2135419 drivers/irqchip/irq-vic.c Thomas Gleixner 2015-06-21  299  		irq_set_chained_handler_and_data(parent_irq,
9f2135419 drivers/irqchip/irq-vic.c Thomas Gleixner 2015-06-21  300  						 vic_handle_irq_cascaded, v);
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  301  	}
e641b987c drivers/irqchip/irq-vic.c Linus Walleij   2013-11-21  302  
07c9249f1 arch/arm/common/vic.c     Linus Walleij   2012-10-16  303  	v->domain = irq_domain_add_simple(node, fls(valid_sources), irq,
fa943bed6 arch/arm/common/vic.c     Linus Walleij   2012-04-20  304  					  &vic_irqdomain_ops, v);
5ced33bc0 arch/arm/common/vic.c     Linus Walleij   2012-12-26  305  	/* create an IRQ mapping for each valid IRQ */
5ced33bc0 arch/arm/common/vic.c     Linus Walleij   2012-12-26  306  	for (i = 0; i < fls(valid_sources); i++)
5ced33bc0 arch/arm/common/vic.c     Linus Walleij   2012-12-26  307  		if (valid_sources & (1 << i))
5ced33bc0 arch/arm/common/vic.c     Linus Walleij   2012-12-26  308  			irq_create_mapping(v->domain, i);
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij   2013-11-24  309  	/* If no base IRQ was passed, figure out our allocated base */
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij   2013-11-24  310  	if (irq)
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij   2013-11-24  311  		v->irq = irq;
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij   2013-11-24  312  	else
3b4df9dbd drivers/irqchip/irq-vic.c Linus Walleij   2013-11-24  313  		v->irq = irq_find_mapping(v->domain, 0);
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  314  }
bb06b7371 arch/arm/common/vic.c     Hartley Sweeten 2010-01-12  315  

:::::: The code at line 295 was first introduced by commit
:::::: 7fb7d8ae1036f1002fe8b16c15cc26d32046df64 ARM: VIC: set handle_arch_irq in VIC initialization

:::::: TO: Rob Herring <rob.herring@...xeda.com>
:::::: CC: Rob Herring <rob.herring@...xeda.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (12184 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ