Hello,
On Fri, 6 Jul 2018 12:01:30 +0100, Lorenzo Pieralisi wrote:
> > It is a serious bug, but it is not a regression. I.e whatever is fixed
> > by this patch never worked. So if you take a conservative view on
> > pushing patches to stable and only want to fix regressions, this one
> > doesn't qualify. However, if you take a more open approach and say you
> > want to fix obvious bugs/issues even in older stable kernels, then
> > indeed this patch qualifies.
> >
> > If you are within the kernel developers with this more open approach to
> > sending patches to stable, then you can add:
> >
> > Fixes: 8c39d710363c1 ("PCI: aardvark: Add Aardvark PCI host controller driver")
> > Cc: <stable@xxxxxxxxxxxxxxx>
> >
> > when applying the patch. Unless of course you want me to resend the
> > patch series with this added. Just let met know what you prefer.
>
> It is up to you, I think it qualifies for stable kernels so I would
> send it but it is your choice, please let me know.
It's OK for me to have it in stable kernels. The fix is pretty obvious,
so you can add the Fixes/Cc tags when applying. Unless of course you
want me to resend an updated version.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
