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

From: kbuild test robot
Date: Sat Jun 24 2017 - 12:45:39 EST


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@xxxxxxx>
:::::: CC: Joerg Roedel <jroedel@xxxxxxx>

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

Attachment: .config.gz
Description: application/gzip