Re: [EXT] Re: [PATCH] PCI/VPD: Remove VPD quirk for QLogic 1077:2261

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

 



On Thu, Feb 25, 2021 at 11:32:28AM -0800, Arun Easi wrote:
> On Thu, 25 Feb 2021, 8:31am, Bjorn Helgaas wrote:
> > On Wed, Feb 24, 2021 at 03:00:18PM -0800, Arun Easi wrote:
> > > On Fri, 18 Dec 2020, 5:04pm, Arun Easi wrote:
> > > 
> > > > The VPD quirk was added by [0] to avoid a system NMI; this issue
> > > > has been long fixed in the HBA firmware. In addition, PCI also has
> > > > the logic to check the VPD size [1], so this quirk can be reverted
> > > > now. More details in the thread:
> > > >     "VPD blacklist of Marvell QLogic 1077/2261" [2].
> > > > 
> > > > [0] 0d5370d1d852 ("PCI: Prevent VPD access for QLogic ISP2722")
> > > > [1] 104daa71b396 ("PCI: Determine actual VPD size on first access")
> > > > [2] https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_linux-2Dpci_alpine.LRH.2.21.9999.2012161641230.28924-40irv1user01.caveonetworks.com_&d=DwIBAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=P-q_Qkt75qFy33SvdD2nAxAyN87eO1d-mFO-lqNOomw&m=Bw8qGbVsETqSibSD8JVMAxZh8BCn1cHuskKjbarfuT8&s=IMvYnIBgaHJkzF2-GgIrGymbRguV287NVLG1_KcP_po&e= 
> > > > 
> > > > Signed-off-by: Arun Easi <aeasi@xxxxxxxxxxx>
> > > > CC: stable@xxxxxxxxxxxxxxx      # v4.6+
> > > > ---
> > > 
> > > Wondering if there is something needed from my side. I could not find this 
> > > in the v5.12 list.
> > 
> > Sorry, I blew it on this one (and other VPD patches from Heiner).
> > It's too late for v5.12, but I'll try again for v5.13.
> > 
> 
> No worries. As these are mostly one-liners and straight forward changes, 
> posting during RC is not an option for 5.12 inclusion?

Not unless it's critical or it fixes a bug we introduced during the
merge window.  This doesn't seem like either, but please clarify if it
is.

>From the commit log, it looks like this is just a cleanup with no
benefit to the user.  If that's the case, there's no reason to
backport to stable either.

Bjorn



[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