Re: [PATCH v7 0/2] PCI: quirks: Cavium ThunderX ACS quirk update

[Bjorn Helgaas]

On Thu, Oct 19, 2017 at 04:59:21AM -0700, Vadim Lomovtsev wrote:
> On Thu, Oct 19, 2017 at 06:26:45AM -0500, Bjorn Helgaas wrote:
> > On Tue, Oct 17, 2017 at 05:47:37AM -0700, Vadim Lomovtsev wrote:
> > > From: Vadim Lomovtsev <Vadim.Lomovtsev@xxxxxxxxxx>
> > > 
> > > version 7:
> > > - update patch description accordingly to review comments;
> > > - split for two patches: for ACS mask change and device id match change;
> > > - remove macro #define of ACS flags, put it back to quirk function;
> > > - remove '__inline_' from device id matching function;
> > > - wrap code comments to fit into 80 columns;
> > > - comment fix: change 0xa00 to 0xf800 for matching function description;
> > > 
> > > version 6:
> > > - comment typo fix: change 0xa00 to 0xa000;
> > > 
> > > version 5:
> > > - update code comments accordingly to review comments;
> > > 
> > > version 4:
> > > - update ACS mask (remove TB and TD bits), update commit message (remove
> > >   ACS register printout);
> > > 
> > > version 3:
> > > - update subject: remove CN8XXX from subject line, replace it with ThunderX;
> > > 
> > > version 2:
> > > - update match function in order to filter only ThunderX devices by device
> > >   ids to properly filter CN8XXX devices, update subject & description with
> > >   ACS register info (v2 was rejected by maillist due to triple X in subject);
> > >   
> > > Vadim Lomovtsev (2):
> > >   PCI: quirks: Set Cavium ACS capability quirk flags to assert
> > >     RR/CR/SV/UF.
> > >   PCI: quirks: Apply Cavium ThunderX ACS quirk only to Root Ports
> > > 
> > >  drivers/pci/quirks.c | 26 ++++++++++++++++++++------
> > >  1 file changed, 20 insertions(+), 6 deletions(-)
> > 
> > If I'm reading this correctly, the first patch is basically fixing a bug in
> > the original Cavium ACS quirk (b404bcfbf035 ("PCI: Add ACS quirk for all
> > Cavium devices")).  Right?
> 
> Yes, first patch fixes improper ACS flags in the patch you mentioned.
> 
> The second one fixes devid check from another patch
> (b77d537d00d0 PCI: Apply Cavium ACS quirk only to CN81xx/CN83xx/CN88xx devices).
> 
> > 
> > I put them on pci/virtualization for v4.15 as follows (patches unchanged,
> > changelogs wordsmithed).
> >
> 
> Good, thank you.
> 
> > You mentioned stable backports.  b404bcfbf035 ("PCI: Add ACS quirk for all
> > Cavium devices") appeared in v4.6.  b77d537d00d0 ("PCI: Apply Cavium ACS
> > quirk only to CN81xx/CN83xx/CN88xx devices") appeared in v4.12.  I assume
> > you would ideally want all this backported as far as v4.6?  We'd have to
> > figure out how to express that in stable tags.
> 
> Ideally yes, I think we would need to have them backported onto stable(s),
> but so far I didn't see any bug-reports which could require these fixes.
> 
> Anyway, in term of backporting them - are there anything I can help with ?

I added the following tags, which I *think* should be sufficient:

  PCI: Set Cavium ACS capability quirk flags to assert RR/CR/SV/UF
  Cc: stable@xxxxxxxxxxxxxxx      # v4.6+: b77d537d00d0: PCI: Apply Cavium ACS quirk only to CN81xx/CN83xx/CN88xx devices

  PCI: Apply Cavium ThunderX ACS quirk to more Root Ports
  Cc: stable@xxxxxxxxxxxxxxx      # v4.12+