Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-05 Thread Todd Fujinaka
I've tried to read the code but I'm not sure where the strings are getting parsed. I'm afraid I'm only familiar with the Ethernet drivers. In any case, the patch didn't help and I had to add this second Marvell line. I'm not sure if you can just wildcard the second and third fields or not. I'm

Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-03 Thread Alexander Duyck
On Tue, Feb 2, 2016 at 10:48 PM, Kirill A. Shutemov wrote: > On Tue, Feb 02, 2016 at 09:45:48PM -0500, Martin K. Petersen wrote: >> > "Kirill" == Kirill A Shutemov writes: >> >> Kirill> I have the same problem. >> >> Kirill> Shouldn't we put quirk

Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-03 Thread Martin K. Petersen
> "Kirill" == Kirill A Shutemov writes: Kirill> It didn't help me. OK, that's fine. I'll queue your patch if we can get somebody to review it... -- Martin K. Petersen Oracle Linux Engineering -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-02 Thread Kirill A. Shutemov
On Wed, Jan 20, 2016 at 10:35:15PM -0800, Alexander Duyck wrote: > Ultimately neither of these bugs were my root cause. It turns out the > Marvel Console SCSI device in my system needed to have a flag set to > disable VPD access in order to keep things from looping through the error > repeatedly.

Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-02 Thread Kirill A. Shutemov
On Tue, Feb 02, 2016 at 09:45:48PM -0500, Martin K. Petersen wrote: > > "Kirill" == Kirill A Shutemov writes: > > Kirill> I have the same problem. > > Kirill> Shouldn't we put quirk for that? > > I was hoping that Hannes' patch would do the trick so we could avoid >

Re: [PATCH 0/2] scsi: Fix endless loop of ATA hard resets due to VPD reads

2016-02-02 Thread Martin K. Petersen
> "Kirill" == Kirill A Shutemov writes: Kirill> I have the same problem. Kirill> Shouldn't we put quirk for that? I was hoping that Hannes' patch would do the trick so we could avoid blacklisting: https://patchwork.kernel.org/patch/8079011/ -- Martin K.