> -----Original Message----- > From: Bjorn Helgaas [mailto:helgaas@xxxxxxxxxx] > Sent: Friday, June 16, 2017 3:10 AM > To: Jean Delvare <jdelvare@xxxxxxx> > Cc: K, Narendra <Narendra_K@xxxxxxxx>; x86@xxxxxxxxxx; linux- > pci@xxxxxxxxxxxxxxx; Hargrave, Jordan <Jordan_Hargrave@xxxxxxxx>; Iyer, > Shyam <Shyam_Iyer@xxxxxxxx>; bhelgaas@xxxxxxxxxx > Subject: Re: dmi type 0xB1 record - unknown flag > > On Fri, Jun 02, 2017 at 04:13:11PM +0200, Jean Delvare wrote: > > ... > > I think the whole thing can be simplified like this: > > > > From: Jean Delvare <jdelvare@xxxxxxx> > > Subject: x86/PCI: Simplify Dell DMI B1 quirk > > > > No need for such convoluted code, when all we need is to call one > > function in one specific case. > > > > Signed-off-by: Jean Delvare <jdelvare@xxxxxxx> > > Applied to pci/misc for v4.13, thanks! > > As far as I can tell, the new code is functionally equivalent to the old code, so > I don't think there's any risk here. > Hi Jean/Bjorn, Sorry for the delay. I applied the patch to the mainline kernel version 4.12-rc5 and performed sanity test on a DellEMC PowerEdge R730XD system. It is working as expected. 'dmesg' shows the message that 'pci=bfsort' is enabled. I also booted kernel version 4.12-rc5 with patch applied on a DellEMC PowerEdge 1950 system which is listed in the ' pciprobe_dmi_table' quicks table. It is working as expected and has not caused regression on this system. 'dmesg' shows the message that 'pci=bfsort' is enabled. [...] > > > > What do you think? Thank you for simplifying the code and for the patch. With regards, Narendra K
