Re: [helgaas-pci:for-linus 4/4] drivers/pci/controller/pci-hyperv.c:1486:9: error: implicit declaration of function 'hv_set_msi_entry_from_desc'

[Bjorn Helgaas <helgaas@kernel.org>]

On Wed, Mar 30, 2022 at 03:50:49PM -0700, Nathan Chancellor wrote:
> On Thu, Mar 31, 2022 at 06:43:32AM +0800, kernel test robot wrote:
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git for-linus
> > head:   18146f25ac6695ce2ed09503de46dafd2b1f36a6
> > commit: 18146f25ac6695ce2ed09503de46dafd2b1f36a6 [4/4] PCI: hv: Remove unused hv_set_msi_entry_from_desc()
> > config: arm64-allyesconfig (https://download.01.org/0day-ci/archive/20220331/202203310629.C8KmndLW-lkp@intel.com/config)
> > compiler: aarch64-linux-gcc (GCC) 11.2.0
> > reproduce (this is a W=1 build):
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/commit/?id=18146f25ac6695ce2ed09503de46dafd2b1f36a6
> >         git remote add helgaas-pci https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git
> >         git fetch --no-tags helgaas-pci for-linus
> >         git checkout 18146f25ac6695ce2ed09503de46dafd2b1f36a6
> >         # save the config file to linux build tree
> >         mkdir build_dir
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm64 SHELL=/bin/bash
> > 
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp@intel.com>
> > 
> > All errors (new ones prefixed by >>):
> > 
> >    drivers/pci/controller/pci-hyperv.c: In function 'hv_irq_unmask':
> > >> drivers/pci/controller/pci-hyperv.c:1486:9: error: implicit declaration of function 'hv_set_msi_entry_from_desc' [-Werror=implicit-function-declaration]
> >     1486 |         hv_set_msi_entry_from_desc(&params->int_entry.msi_entry, msi_desc);
> >          |         ^~~~~~~~~~~~~~~~~~~~~~~~~~
> >    cc1: some warnings being treated as errors
> 
> 18146f25ac66 ("PCI: hv: Remove unused hv_set_msi_entry_from_desc()")
> does not have d06957d7a692 ("PCI: hv: Avoid the retarget interrupt
> hypercall in irq_unmask() on ARM64") as a parent, since it is only in
> mainline with no tag, whereas 18146f25ac66 is based on v5.17-rc7, which
> does not have the problem that 18146f25ac66 addresses.
> 
> I think that rebasing the pending fixes on the 5.18 PCI changes would
> properly resolve this.

Yep, sorry, I blew it.  I rebased my for-linus branch to 148a65047695
("Merge tag 'pci-v5.18-changes' of
git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci"), which
does have d06957d7a692, and added this fix (PCI: hv: Remove unused
hv_set_msi_entry_from_desc()).