Re: [PATCH 0/5]stop normal completion path entering a timeout req

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

 



On 06/21/18 14:12, Keith Busch wrote:
On Thu, Jun 21, 2018 at 06:21:09PM +0000, Bart Van Assche wrote:
That's not how the SCSI core works.

How does it work? Jianchao is advocating for holes in software
to ignore reacting to hardware events and force unnecessary error
escalation. Putting holes in kernel software can't possibly be our
only recourse.

Why were the patch "blk-mq: Remove generation seqeunce" and related patches rushed upstream without having given anyone a chance to review these patches? The approach of these patches is inferior to at least one alternative that was available. The code that is currently upstream spreads the request state over multiple (three) variables while an alternative approach was presented that stores the request state in a single variable. I think the number of fix-up patches that is needed for the approach that is currently upstream shows that the upstream approach is inferior. I'm referring to "[PATCH v14] blk-mq: Rework blk-mq timeout handling again" (https://marc.info/?l=linux-block&m=152703042412228).

Bart.



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux