Re: [PATCH] iommu/amd: Fix interrupt remapping when disable guest_mode

[kbuild test robot <lkp@intel.com>]

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

Hi Suravee,

[auto build test WARNING on iommu/next]
[also build test WARNING on v4.12-rc6 next-20170623]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Suravee-Suthikulpanit/iommu-amd-Fix-interrupt-remapping-when-disable-guest_mode/20170624-162701
base:   https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next
config: x86_64-randconfig-b0-06242320 (attached as .config)
compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/iommu/amd_iommu.c: In function 'irte_ga_prepare':
>> drivers/iommu/amd_iommu.c:4002: warning: unused variable 'dev_data'

vim +/dev_data +4002 drivers/iommu/amd_iommu.c

880ac60e Suravee Suthikulpanit 2016-08-23  3986  {
880ac60e Suravee Suthikulpanit 2016-08-23  3987  	union irte *irte = (union irte *) entry;
880ac60e Suravee Suthikulpanit 2016-08-23  3988  
880ac60e Suravee Suthikulpanit 2016-08-23  3989  	irte->val                = 0;
880ac60e Suravee Suthikulpanit 2016-08-23  3990  	irte->fields.vector      = vector;
880ac60e Suravee Suthikulpanit 2016-08-23  3991  	irte->fields.int_type    = delivery_mode;
880ac60e Suravee Suthikulpanit 2016-08-23  3992  	irte->fields.destination = dest_apicid;
880ac60e Suravee Suthikulpanit 2016-08-23  3993  	irte->fields.dm          = dest_mode;
880ac60e Suravee Suthikulpanit 2016-08-23  3994  	irte->fields.valid       = 1;
880ac60e Suravee Suthikulpanit 2016-08-23  3995  }
880ac60e Suravee Suthikulpanit 2016-08-23  3996  
880ac60e Suravee Suthikulpanit 2016-08-23  3997  static void irte_ga_prepare(void *entry,
880ac60e Suravee Suthikulpanit 2016-08-23  3998  			    u32 delivery_mode, u32 dest_mode,
d98de49a Suravee Suthikulpanit 2016-08-23  3999  			    u8 vector, u32 dest_apicid, int devid)
880ac60e Suravee Suthikulpanit 2016-08-23  4000  {
880ac60e Suravee Suthikulpanit 2016-08-23  4001  	struct irte_ga *irte = (struct irte_ga *) entry;
d98de49a Suravee Suthikulpanit 2016-08-23 @4002  	struct iommu_dev_data *dev_data = search_dev_data(devid);
880ac60e Suravee Suthikulpanit 2016-08-23  4003  
880ac60e Suravee Suthikulpanit 2016-08-23  4004  	irte->lo.val                      = 0;
880ac60e Suravee Suthikulpanit 2016-08-23  4005  	irte->hi.val                      = 0;
880ac60e Suravee Suthikulpanit 2016-08-23  4006  	irte->lo.fields_remap.int_type    = delivery_mode;
880ac60e Suravee Suthikulpanit 2016-08-23  4007  	irte->lo.fields_remap.dm          = dest_mode;
880ac60e Suravee Suthikulpanit 2016-08-23  4008  	irte->hi.fields.vector            = vector;
880ac60e Suravee Suthikulpanit 2016-08-23  4009  	irte->lo.fields_remap.destination = dest_apicid;
880ac60e Suravee Suthikulpanit 2016-08-23  4010  	irte->lo.fields_remap.valid       = 1;

:::::: The code at line 4002 was first introduced by commit
:::::: d98de49a53e48f51332e97568127e722415e1232 iommu/amd: Enable vAPIC interrupt remapping mode by default

:::::: TO: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
:::::: CC: Joerg Roedel <jroedel@suse.de>

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

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