RE: [External] Re: [PATCH v1] PCI: Set no io resource for bridges that behind VMD controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Sep 25, 2022 at 8:57 PM +0800, Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote:
> On Sun, Sep 25, 2022 at 09:52:21AM +0000, Xiaochun XC17 Li wrote:
> > > -----Original Message-----
> > > From: kernel test robot <lkp@xxxxxxxxx>
> > > Sent: Friday, September 23, 2022 11:21 PM
> > > To: Xiaochun Lee <lixiaochun.2888@xxxxxxx>;
> > > nirmal.patel@xxxxxxxxxxxxxxx; jonathan.derrick@xxxxxxxxx
> > > Cc: kbuild-all@xxxxxxxxxxxx; lpieralisi@xxxxxxxxxx; robh@xxxxxxxxxx;
> > > kw@xxxxxxxxx; bhelgaas@xxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx;
> linux-
> > > kernel@xxxxxxxxxxxxxxx; Xiaochun XC17 Li <lixc17@xxxxxxxxxx>
> > > Subject: [External] Re: [PATCH v1] PCI: Set no io resource for
> > > bridges that behind VMD controller
> > >
> > > Hi Xiaochun,
> > >
> > > Thank you for the patch! Yet something to improve:
> > >
> > > [auto build test ERROR on helgaas-pci/next] [also build test ERROR
> > > on linus/master v6.0-rc6 next-20220923] [If your patch is applied to
> > > the wrong git tree, kindly drop us a note.
> > Hi, thanks for your reply, this patch is only expected to be applied
> > on x86 or x64, actually function "is_vmd()" is just defined in
> > arch/x86/include/asm/pci.h, do we need to support it on arch um?
> 
> Yes, all possible Kconfig configurations must build cleanly.  You may have
> to add additional ifdef tests or an is_vmd() stub.

Thanks, I'll fix it and complier it base arch um, the patch v2 will be sent out.

> 
> For future reference, your email reply doesn't follow the usual Linux
> mailing list style, so it is unnecessarily hard to read.  In particular, it lacks
> the line that shows what you're responding to.
> It would look something like this:
> 
>   On Fri, Sep 23, 2022 at 11:21PM +0800, kernel test robot wrote:

OK, thanks! 
I did a test just now, it seems to meet the format requirements.
I sent emails via outlook, and I've tried to set the reply format, 
unfortunately, the time zone and date time  are not resolved correctly.

> 
> Here's a sample:
> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.
> kernel.org%2Fr%2FYyGghUdcrOdrR0ep%40smile.fi.intel.com&amp;data=05
> %7C01%7Clixc17%40lenovo.com%7C27c59eea4afb46c8e24608da9ef5752d%
> 7C5c7d0b28bdf8410caa934df372b16203%7C0%7C0%7C6379970750209852
> 91%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luM
> zIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000%7C%7C%7C&amp;sdata=Y8
> O9DzJ%2FW8JlAz0LM8o6LgnQcWUbYlxseM1wWbLvUKI%3D&amp;reserved
> =0
> 
> More background:
> https://apc01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fen.
> wikipedia.org%2Fwiki%2FPosting_style%23Interleaved_style&amp;data=05
> %7C01%7Clixc17%40lenovo.com%7C27c59eea4afb46c8e24608da9ef5752d%
> 7C5c7d0b28bdf8410caa934df372b16203%7C0%7C0%7C6379970750209852
> 91%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luM
> zIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000%7C%7C%7C&amp;sdata=3Q
> oamhkXi78AOmFMuCRLnL52Ed8yEZ79Vx72pEpE4cs%3D&amp;reserved=0




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux