CC: kbuild-all(a)lists.01.org In-Reply-To: <20201229032513.486395-4-yi.l.liu@intel.com> References: <20201229032513.486395-4-yi.l.liu@intel.com> TO: Liu Yi L TO: baolu.lu(a)linux.intel.com TO: joro(a)8bytes.org TO: will(a)kernel.org CC: kevin.tian(a)intel.com CC: jacob.jun.pan(a)linux.intel.com CC: ashok.raj(a)intel.com CC: yi.l.liu(a)intel.com CC: jun.j.tian(a)intel.com CC: yi.y.sun(a)intel.com CC: iommu(a)lists.linux-foundation.org Hi Liu, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on v5.11-rc1] [also build test WARNING on next-20201223] [cannot apply to iommu/next] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Liu-Yi-L/iommu-vt-d-Misc-fixes-on-scalable-mode/20201229-113203 base: 5c8fe583cce542aa0b84adc939ce85293de36e5e :::::: branch date: 5 hours ago :::::: commit date: 5 hours ago config: i386-randconfig-m021-20201229 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot Reported-by: Dan Carpenter New smatch warnings: drivers/iommu/intel/iommu.c:1471 domain_update_iotlb() error: we previously assumed 'info' could be null (see line 1472) Old smatch warnings: drivers/iommu/intel/iommu.c:920 device_to_iommu() error: we previously assumed 'pdev' could be null (see line 893) drivers/iommu/intel/iommu.c:3764 intel_iommu_add() warn: should '(1 << sp)' be a 64 bit type? vim +/info +1471 drivers/iommu/intel/iommu.c 93a23a7271dfb81 drivers/pci/intel-iommu.c Yu Zhao 2009-05-18 1463 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1464 static void domain_update_iotlb(struct dmar_domain *domain) 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1465 { 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1466 struct device_domain_info *info; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1467 bool has_iotlb_device = false; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1468 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1469 assert_spin_locked(&device_domain_lock); 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1470 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 @1471 list_for_each_entry(info, &domain->devices, link) 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 @1472 if (info && info->ats_enabled) { 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1473 has_iotlb_device = true; 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1474 break; 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1475 } 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1476 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1477 if (!has_iotlb_device) { 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1478 struct subdev_domain_info *sinfo; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1479 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1480 list_for_each_entry(sinfo, &domain->subdevices, link_domain) { 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1481 info = get_domain_info(sinfo->pdev); 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1482 if (info && info->ats_enabled) { 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1483 has_iotlb_device = true; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1484 break; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1485 } 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1486 } 1aaf68f8927fb5c drivers/iommu/intel/iommu.c Liu Yi L 2020-12-29 1487 } 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1488 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1489 domain->has_iotlb_device = has_iotlb_device; 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1490 } 0824c5920b16fe1 drivers/iommu/intel-iommu.c Omer Peleg 2016-04-20 1491 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org