Re: [RFC 4/5] PCI: vmd: Stop overriding dma_map_ops

From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC 4/5] PCI: vmd: Stop overriding dma_map_ops
Date: Fri, 03 Jan 2020 23:00:16 +0800	[thread overview]
Message-ID: <202001032250.pBOeHe73%lkp@intel.com> (raw)
In-Reply-To: <1577823863-3303-5-git-send-email-jonathan.derrick@intel.com>

[-- Attachment #1: Type: text/plain, Size: 5222 bytes --]

Hi Jon,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on pci/next]
[also build test ERROR on iommu/next tip/x86/core v5.5-rc4 next-20191220]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Jon-Derrick/Clean-up-VMD-DMA-Map-Ops/20200103-175834
base:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: x86_64-lkp (attached as .config)
compiler: gcc-7 (Debian 7.5.0-3) 7.5.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All error/warnings (new ones prefixed by >>):

   In file included from arch/x86/include/asm/percpu.h:45:0,
                    from arch/x86/include/asm/current.h:6,
                    from include/linux/sched.h:12,
                    from arch/x86/pci/common.c:8:
   arch/x86/pci/common.c: In function 'set_dev_domain_options':
>> arch/x86/pci/common.c:670:35: error: 'struct pci_sysdata' has no member named 'vmd_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                                      ^
   include/linux/kernel.h:995:26: note: in definition of macro 'container_of'
     void *__mptr = (void *)(ptr);     \
                             ^~~
>> arch/x86/pci/common.c:670:22: note: in expansion of macro 'to_pci_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                         ^~~~~~~~~~
   In file included from arch/x86/include/asm/current.h:5:0,
                    from include/linux/sched.h:12,
                    from arch/x86/pci/common.c:8:
>> arch/x86/pci/common.c:670:35: error: 'struct pci_sysdata' has no member named 'vmd_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                                      ^
   include/linux/compiler.h:330:9: note: in definition of macro '__compiletime_assert'
      if (!(condition))     \
            ^~~~~~~~~
   include/linux/compiler.h:350:2: note: in expansion of macro '_compiletime_assert'
     _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
     ^~~~~~~~~~~~~~~~~~~
   include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^~~~~~~~~~~~~~~~~~
   include/linux/kernel.h:996:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
     BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
     ^~~~~~~~~~~~~~~~
   include/linux/kernel.h:996:20: note: in expansion of macro '__same_type'
     BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
                       ^~~~~~~~~~~
>> include/linux/pci.h:486:23: note: in expansion of macro 'container_of'
    #define to_pci_dev(n) container_of(n, struct pci_dev, dev)
                          ^~~~~~~~~~~~
>> arch/x86/pci/common.c:670:22: note: in expansion of macro 'to_pci_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                         ^~~~~~~~~~
>> arch/x86/pci/common.c:670:35: error: 'struct pci_sysdata' has no member named 'vmd_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                                      ^
   include/linux/compiler.h:330:9: note: in definition of macro '__compiletime_assert'
      if (!(condition))     \
            ^~~~~~~~~
   include/linux/compiler.h:350:2: note: in expansion of macro '_compiletime_assert'
     _compiletime_assert(condition, msg, __compiletime_assert_, __LINE__)
     ^~~~~~~~~~~~~~~~~~~
   include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
    #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
                                        ^~~~~~~~~~~~~~~~~~
   include/linux/kernel.h:996:2: note: in expansion of macro 'BUILD_BUG_ON_MSG'
     BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
     ^~~~~~~~~~~~~~~~
   include/linux/kernel.h:997:6: note: in expansion of macro '__same_type'
        !__same_type(*(ptr), void),   \
         ^~~~~~~~~~~
>> include/linux/pci.h:486:23: note: in expansion of macro 'container_of'
    #define to_pci_dev(n) container_of(n, struct pci_dev, dev)
                          ^~~~~~~~~~~~
>> arch/x86/pci/common.c:670:22: note: in expansion of macro 'to_pci_dev'
      pdev->dma_parent = to_pci_dev(sd->vmd_dev);
                         ^~~~~~~~~~

vim +670 arch/x86/pci/common.c

   664	
   665	static void set_dev_domain_options(struct pci_dev *pdev)
   666	{
   667		if (is_vmd(pdev->bus)) {
   668			struct pci_sysdata *sd = pdev->bus->sysdata;
   669	
 > 670			pdev->dma_parent = to_pci_dev(sd->vmd_dev);
   671			pdev->hotplug_user_indicators = 1;
   672		}
   673	}
   674	

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 28801 bytes --]