Re: [PATCH] x86, pci: Add quirk for unsizeable Broadwell EP bar
From: Andi Kleen
Date: Thu Feb 04 2016 - 22:36:24 EST
> > But would actually anything use it?
>
> You mean, would anything actually use the lspci output? I don't know,
> but why would we want it to print garbage?
In he kernel. I don't think lspci is that interesting.
>
> And the kernel certainly uses the struct resource. Setting
> IORESOURCE_PCI_FIXED is not a way of saying "please ignore this
> resource."
There is already another quirk that uses the same technique to handle
a bad bar. I also didn't notice any bad side effects. Again what would it be
used for?
I looked into the new device ops you asked for, but it is fairly
complicated as the ops are not per device but per bus, and there
can be many copies of this device, and the pci_dev is not passed,
so it needs complicated pattern matching on the devfn.
Doing it like the existing quirk is much simpler, and seems to work
just fine.
-Andi
--
ak@xxxxxxxxxxxxxxx -- Speaking for myself only.