Re: [PATCH v4 20/24] iommu/mediatek: Support report iova 34bit translation fault in ISR

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

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

Hi Yong,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on iommu/next]
[also build test ERROR on robh/for-next linus/master v5.10-rc4 next-20201117]
[cannot apply to mediatek/for-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/Yong-Wu/MT8192-IOMMU-support/20201111-204421
base:   https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next
config: arm-randconfig-r003-20201117 (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/2ac9ae3e1803f422950cdad221f033bb9ba9503d
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Yong-Wu/MT8192-IOMMU-support/20201111-204421
        git checkout 2ac9ae3e1803f422950cdad221f033bb9ba9503d
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm 

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

All errors (new ones prefixed by >>):

   drivers/iommu/mtk_iommu.c: In function 'mtk_iommu_isr':
>> drivers/iommu/mtk_iommu.c:289:13: error: implicit declaration of function 'FIELD_GET'; did you mean 'FOLL_GET'? [-Werror=implicit-function-declaration]
     289 |   va34_32 = FIELD_GET(F_MMU_INVAL_VA_34_32_MASK, fault_iova);
         |             ^~~~~~~~~
         |             FOLL_GET
   cc1: some warnings being treated as errors

vim +289 drivers/iommu/mtk_iommu.c

   265	
   266	static irqreturn_t mtk_iommu_isr(int irq, void *dev_id)
   267	{
   268		struct mtk_iommu_data *data = dev_id;
   269		struct mtk_iommu_domain *dom = data->m4u_dom;
   270		unsigned int fault_larb, fault_port, sub_comm = 0;
   271		u32 int_state, regval, va34_32, pa34_32;
   272		u64 fault_iova, fault_pa;
   273		bool layer, write;
   274	
   275		/* Read error info from registers */
   276		int_state = readl_relaxed(data->base + REG_MMU_FAULT_ST1);
   277		if (int_state & F_REG_MMU0_FAULT_MASK) {
   278			regval = readl_relaxed(data->base + REG_MMU0_INT_ID);
   279			fault_iova = readl_relaxed(data->base + REG_MMU0_FAULT_VA);
   280			fault_pa = readl_relaxed(data->base + REG_MMU0_INVLD_PA);
   281		} else {
   282			regval = readl_relaxed(data->base + REG_MMU1_INT_ID);
   283			fault_iova = readl_relaxed(data->base + REG_MMU1_FAULT_VA);
   284			fault_pa = readl_relaxed(data->base + REG_MMU1_INVLD_PA);
   285		}
   286		layer = fault_iova & F_MMU_FAULT_VA_LAYER_BIT;
   287		write = fault_iova & F_MMU_FAULT_VA_WRITE_BIT;
   288		if (MTK_IOMMU_HAS_FLAG(data->plat_data, IOVA_34_EN)) {
 > 289			va34_32 = FIELD_GET(F_MMU_INVAL_VA_34_32_MASK, fault_iova);
   290			pa34_32 = FIELD_GET(F_MMU_INVAL_PA_34_32_MASK, fault_iova);
   291			fault_iova = fault_iova & F_MMU_INVAL_VA_31_12_MASK;
   292			fault_iova |=  (u64)va34_32 << 32;
   293			fault_pa |= (u64)pa34_32 << 32;
   294		}
   295	
   296		fault_port = F_MMU_INT_ID_PORT_ID(regval);
   297		if (MTK_IOMMU_HAS_FLAG(data->plat_data, HAS_SUB_COMM)) {
   298			fault_larb = F_MMU_INT_ID_COMM_ID(regval);
   299			sub_comm = F_MMU_INT_ID_SUB_COMM_ID(regval);
   300		} else {
   301			fault_larb = F_MMU_INT_ID_LARB_ID(regval);
   302		}
   303		fault_larb = data->plat_data->larbid_remap[fault_larb][sub_comm];
   304	
   305		if (report_iommu_fault(&dom->domain, data->dev, fault_iova,
   306				       write ? IOMMU_FAULT_WRITE : IOMMU_FAULT_READ)) {
   307			dev_err_ratelimited(
   308				data->dev,
   309				"fault type=0x%x iova=0x%llx pa=0x%llx larb=%d port=%d layer=%d %s\n",
   310				int_state, fault_iova, fault_pa, fault_larb, fault_port,
   311				layer, write ? "write" : "read");
   312		}
   313	
   314		/* Interrupt clear */
   315		regval = readl_relaxed(data->base + REG_MMU_INT_CONTROL0);
   316		regval |= F_INT_CLR_BIT;
   317		writel_relaxed(regval, data->base + REG_MMU_INT_CONTROL0);
   318	
   319		mtk_iommu_tlb_flush_all(data);
   320	
   321		return IRQ_HANDLED;
   322	}
   323	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

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

[-- Attachment #3: Type: text/plain, Size: 170 bytes --]

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek