RE: [EXT] Re: [PATCH v9 2/4] irqchip: Add IMX MU MSI controller driver

From: Frank Li
Date: Thu Sep 08 2022 - 10:27:11 EST




> -----Original Message-----
> From: Marc Zyngier <maz@xxxxxxxxxx>
> Sent: Thursday, September 8, 2022 2:02 AM
> To: kernel test robot <lkp@xxxxxxxxx>; Frank Li <frank.li@xxxxxxx>
> Cc: tglx@xxxxxxxxxxxxx; robh+dt@xxxxxxxxxx;
> krzysztof.kozlowski+dt@xxxxxxxxxx; shawnguo@xxxxxxxxxx;
> s.hauer@xxxxxxxxxxxxxx; kw@xxxxxxxxx; bhelgaas@xxxxxxxxxx; kbuild-
> all@xxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx; Peng Fan
> <peng.fan@xxxxxxx>; Aisheng Dong <aisheng.dong@xxxxxxx>;
> jdmason@xxxxxxxx; kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; dl-linux-
> imx <linux-imx@xxxxxxx>; kishon@xxxxxx; lorenzo.pieralisi@xxxxxxx;
> ntb@xxxxxxxxxxxxxxx; lznuaa@xxxxxxxxx; imx@xxxxxxxxxxxxxxx;
> manivannan.sadhasivam@xxxxxxxxxx
> Subject: [EXT] Re: [PATCH v9 2/4] irqchip: Add IMX MU MSI controller driver
>
> Caution: EXT Email
>
> Frank,
>
> On Thu, 08 Sep 2022 01:03:27 +0100,
> kernel test robot <lkp@xxxxxxxxx> wrote:
> >
> > Hi Frank,
> >
> > I love your patch! Yet something to improve:
> >
> > [auto build test ERROR on jonmason-ntb/ntb-next]
> > [also build test ERROR on robh/for-next linus/master v6.0-rc4 next-
> 20220907]
> > [cannot apply to tip/irq/core]
> > [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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit-
> scm.com%2Fdocs%2Fgit-format-
> patch%23_base_tree_information&amp;data=05%7C01%7CFrank.Li%40nxp.c
> om%7Cc409668bc0994f1df11708da91681c6f%7C686ea1d3bc2b4c6fa92cd99c
> 5c301635%7C0%7C0%7C637982173598672297%7CUnknown%7CTWFpbGZsb
> 3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0
> %3D%7C3000%7C%7C%7C&amp;sdata=LgRkCWQ%2BX0GLT5qUaLvIS9SuYMB
> 6tX%2FerPNl3KwW7Tc%3D&amp;reserved=0]
> >
> > url:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub
> .com%2Fintel-lab-lkp%2Flinux%2Fcommits%2FFrank-Li%2FPCI-EP-driver-
> support-MSI-doorbell-from-host%2F20220907-
> 115114&amp;data=05%7C01%7CFrank.Li%40nxp.com%7Cc409668bc0994f1d
> f11708da91681c6f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6
> 37982173598672297%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM
> DAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7
> C&amp;sdata=hpU6XQmibxCGo0S8J7VuPuZDhe6OwCzR92ld9UvODYw%3D&
> amp;reserved=0
> > base:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub
> .com%2Fjonmason%2Fntb&amp;data=05%7C01%7CFrank.Li%40nxp.com%7C
> c409668bc0994f1df11708da91681c6f%7C686ea1d3bc2b4c6fa92cd99c5c3016
> 35%7C0%7C0%7C637982173598672297%7CUnknown%7CTWFpbGZsb3d8eyJ
> WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%
> 7C3000%7C%7C%7C&amp;sdata=3y7%2BH6wcN%2FxcV8swP3QR0lUihWhqw
> qjlTtgUAVAPVg8%3D&amp;reserved=0 ntb-next
> > config: s390-randconfig-s033-20220907
> (https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdow
> nload.01.org%2F0day-
> ci%2Farchive%2F20220908%2F202209080757.hQMfrrfm-
> lkp%40intel.com%2Fconfig&amp;data=05%7C01%7CFrank.Li%40nxp.com%7C
> c409668bc0994f1df11708da91681c6f%7C686ea1d3bc2b4c6fa92cd99c5c3016
> 35%7C0%7C0%7C637982173598672297%7CUnknown%7CTWFpbGZsb3d8eyJ
> WIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%
> 7C3000%7C%7C%7C&amp;sdata=lFaWJd2d3ob06d3qNilgFovocFU%2FN4Goz
> 7jBTXLvCss%3D&amp;reserved=0)
> > compiler: s390-linux-gcc (GCC) 12.1.0
> > reproduce:
> > wget
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fraw.gi
> thubusercontent.com%2Fintel%2Flkp-
> tests%2Fmaster%2Fsbin%2Fmake.cross&amp;data=05%7C01%7CFrank.Li%40
> nxp.com%7Cc409668bc0994f1df11708da91681c6f%7C686ea1d3bc2b4c6fa92
> cd99c5c301635%7C0%7C0%7C637982173598672297%7CUnknown%7CTWFp
> bGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI
> 6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=8C2pQ57ym0gNi0wHXT6KEj8%
> 2BAgMQKph8UhK6GPo%2BNIc%3D&amp;reserved=0 -O ~/bin/make.cross
> > chmod +x ~/bin/make.cross
> > # apt-get install sparse
> > # sparse version: v0.6.4-39-gce1a6720-dirty
> > #
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub
> .com%2Fintel-lab-
> lkp%2Flinux%2Fcommit%2Fc1f079e633c10b4f2f1f3c8f52e447d13fda8ddb&a
> mp;data=05%7C01%7CFrank.Li%40nxp.com%7Cc409668bc0994f1df11708da9
> 1681c6f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6379821735
> 98672297%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
> V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sda
> ta=oBYu8TZJ15AxtGO2IdtGJdE80fYIJTwF4RlkBkeO6hA%3D&amp;reserved=0
> > git remote add linux-review
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub
> .com%2Fintel-lab-
> lkp%2Flinux&amp;data=05%7C01%7CFrank.Li%40nxp.com%7Cc409668bc099
> 4f1df11708da91681c6f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%
> 7C637982173598672297%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA
> wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7
> C%7C&amp;sdata=AQn3FPjP13rnvvkA7TxRs3gpfd26siSoKM%2B%2Fwzw2J94
> %3D&amp;reserved=0
> > git fetch --no-tags linux-review Frank-Li/PCI-EP-driver-support-MSI-
> doorbell-from-host/20220907-115114
> > git checkout c1f079e633c10b4f2f1f3c8f52e447d13fda8ddb
> > # save the config file
> > mkdir build_dir && cp config build_dir/.config
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0
> make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir
> ARCH=s390 SHELL=/bin/bash
> >
> > If you fix the issue, kindly add following tag where applicable
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> >
> > All errors (new ones prefixed by >>):
> >
> > s390-linux-ld: drivers/irqchip/irq-imx-mu-msi.o: in function
> `imx_mu_of_init':
> > >> drivers/irqchip/irq-imx-mu-msi.c:316: undefined reference to
> `devm_platform_ioremap_resource_byname'
>
> This is about the 4th time this breakage gets reported. You keep
> reposting this series without addressing it. What is it going to take
> for you to finally fix it? Clearly, I'm not going to bother taking a
> series that has pending build breakages.

[Frank Li] I also frustrate it now. Robot use random config and can't
Report all problems once. Recently update to gcc 12.x. Build broken
Happen at other place at my environment.

>
> M.
>
> --
> Without deviation from the norm, progress is not possible.