Need to report.
>-----Original Message----- >From: kernel test robot <[email protected]> >Sent: 2020年6月8日 15:34 >To: [email protected] >Cc: lkp <[email protected]> >Subject: [kbuild] Re: [PATCH 2/2] iommu: Add gfp parameter to io_pgtable_ops- >>map() > >CC: [email protected] >In-Reply-To: ><837d7131e83b06ce222bda678fe2e8ac7ba095f2.1591587029.git.baolin.wang@l >inux.alibaba.com> >References: ><837d7131e83b06ce222bda678fe2e8ac7ba095f2.1591587029.git.baolin.wang@l >inux.alibaba.com> >TO: Baolin Wang <[email protected]> >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] >TO: [email protected] > >Hi Baolin, > >I love your patch! Perhaps something to improve: > >[auto build test WARNING on iommu/next] >[also build test WARNING on soc/for-next linus/master v5.7 next-20200605] [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/Baolin-Wang/Some- >improvements-for-iommu/20200608-114141 >base: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next >:::::: branch date: 4 hours ago >:::::: commit date: 4 hours ago >config: i386-randconfig-s001-20200608 (attached as .config) >compiler: gcc-9 (Debian 9.3.0-13) 9.3.0 >reproduce: > # apt-get install sparse > # sparse version: v0.6.1-247-gcadbd124-dirty > # save the attached .config to linux build tree > make W=1 C=1 ARCH=i386 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' > >If you fix the issue, kindly add following tag as appropriate >Reported-by: kernel test robot <[email protected]> > > >sparse warnings: (new ones prefixed by >>) > >>> drivers/iommu/io-pgtable-arm-v7s.c:922:29: sparse: sparse: not enough >>> arguments for function map > drivers/iommu/io-pgtable-arm-v7s.c:929:30: sparse: sparse: not enough >arguments for function map > drivers/iommu/io-pgtable-arm-v7s.c:949:29: sparse: sparse: not enough >arguments for function map > drivers/iommu/io-pgtable-arm-v7s.c:970:29: sparse: sparse: not enough >arguments for function map > ># https://github.com/0day- >ci/linux/commit/6047fa5558b1fdb8d2d52c9807fd960220478d3d >git remote add linux-review https://github.com/0day-ci/linux git remote update >linux-review git checkout 6047fa5558b1fdb8d2d52c9807fd960220478d3d >vim +922 drivers/iommu/io-pgtable-arm-v7s.c > >e5fc9753b1a831 Robin Murphy 2016-01-26 878 >e5fc9753b1a831 Robin Murphy 2016-01-26 879 static int __init >arm_v7s_do_selftests(void) >e5fc9753b1a831 Robin Murphy 2016-01-26 880 { >e5fc9753b1a831 Robin Murphy 2016-01-26 881 struct io_pgtable_ops >*ops; >e5fc9753b1a831 Robin Murphy 2016-01-26 882 struct io_pgtable_cfg >cfg = { >e5fc9753b1a831 Robin Murphy 2016-01-26 883 .tlb = >&dummy_tlb_ops, >e5fc9753b1a831 Robin Murphy 2016-01-26 884 .oas = 32, >e5fc9753b1a831 Robin Murphy 2016-01-26 885 .ias = 32, >4f41845b340783 Will Deacon 2019-06-25 886 .coherent_walk = true, >4f41845b340783 Will Deacon 2019-06-25 887 .quirks = >IO_PGTABLE_QUIRK_ARM_NS, >e5fc9753b1a831 Robin Murphy 2016-01-26 888 .pgsize_bitmap >= SZ_4K | SZ_64K | SZ_1M | SZ_16M, >e5fc9753b1a831 Robin Murphy 2016-01-26 889 }; >e5fc9753b1a831 Robin Murphy 2016-01-26 890 unsigned int iova, size, >iova_start; >e5fc9753b1a831 Robin Murphy 2016-01-26 891 unsigned int i, loopnr = >0; >e5fc9753b1a831 Robin Murphy 2016-01-26 892 >e5fc9753b1a831 Robin Murphy 2016-01-26 893 selftest_running = true; >e5fc9753b1a831 Robin Murphy 2016-01-26 894 >e5fc9753b1a831 Robin Murphy 2016-01-26 895 cfg_cookie = &cfg; >e5fc9753b1a831 Robin Murphy 2016-01-26 896 >e5fc9753b1a831 Robin Murphy 2016-01-26 897 ops = >alloc_io_pgtable_ops(ARM_V7S, &cfg, &cfg); >e5fc9753b1a831 Robin Murphy 2016-01-26 898 if (!ops) { >e5fc9753b1a831 Robin Murphy 2016-01-26 899 pr_err("selftest: >failed to allocate io pgtable ops\n"); >e5fc9753b1a831 Robin Murphy 2016-01-26 900 return -EINVAL; >e5fc9753b1a831 Robin Murphy 2016-01-26 901 } >e5fc9753b1a831 Robin Murphy 2016-01-26 902 >e5fc9753b1a831 Robin Murphy 2016-01-26 903 /* >e5fc9753b1a831 Robin Murphy 2016-01-26 904 * Initial sanity checks. >e5fc9753b1a831 Robin Murphy 2016-01-26 905 * Empty page tables >shouldn't provide any translations. >e5fc9753b1a831 Robin Murphy 2016-01-26 906 */ >e5fc9753b1a831 Robin Murphy 2016-01-26 907 if (ops- >>iova_to_phys(ops, 42)) >e5fc9753b1a831 Robin Murphy 2016-01-26 908 return >__FAIL(ops); >e5fc9753b1a831 Robin Murphy 2016-01-26 909 >e5fc9753b1a831 Robin Murphy 2016-01-26 910 if (ops- >>iova_to_phys(ops, SZ_1G + 42)) >e5fc9753b1a831 Robin Murphy 2016-01-26 911 return >__FAIL(ops); >e5fc9753b1a831 Robin Murphy 2016-01-26 912 >e5fc9753b1a831 Robin Murphy 2016-01-26 913 if (ops- >>iova_to_phys(ops, SZ_2G + 42)) >e5fc9753b1a831 Robin Murphy 2016-01-26 914 return >__FAIL(ops); >e5fc9753b1a831 Robin Murphy 2016-01-26 915 >e5fc9753b1a831 Robin Murphy 2016-01-26 916 /* >e5fc9753b1a831 Robin Murphy 2016-01-26 917 * Distinct mappings of >different granule sizes. >e5fc9753b1a831 Robin Murphy 2016-01-26 918 */ >e5fc9753b1a831 Robin Murphy 2016-01-26 919 iova = 0; >4ae8a5c528c0b1 Kefeng Wang 2016-09-21 920 for_each_set_bit(i, >&cfg.pgsize_bitmap, BITS_PER_LONG) { >e5fc9753b1a831 Robin Murphy 2016-01-26 921 size = 1UL << i; >e5fc9753b1a831 Robin Murphy 2016-01-26 @922 if (ops- >>map(ops, iova, iova, size, IOMMU_READ | > >:::::: The code at line 922 was first introduced by commit >:::::: e5fc9753b1a831466be9b45d0bf926eeaa8b84a0 iommu/io-pgtable: Add >ARMv7 short descriptor support > >:::::: TO: Robin Murphy <[email protected]> >:::::: CC: Will Deacon <[email protected]> > >--- >0-DAY CI Kernel Test Service, Intel Corporation >https://lists.01.org/hyperkitty/list/[email protected] _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
