From: Finn Thain <fthain@telegraphics.com.au>
To: "Martin K. Petersen" <martin.petersen@oracle.com>
Cc: James Bottomley <James.Bottomley@hansenpartnership.com>,
Michael Schmitz <schmitzmic@gmail.com>,
linux-m68k@vger.kernel.org, linux-scsi@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5
Date: Tue, 1 Mar 2016 14:32:23 +1100 (AEDT) [thread overview]
Message-ID: <alpine.LNX.2.00.1603011431300.3734@nippy.intranet> (raw)
In-Reply-To: <yq1d1rfc4iw.fsf@sermon.lab.mkp.net>
On Mon, 29 Feb 2016, Martin K. Petersen wrote:
> >>>>> "Finn" == Finn Thain <fthain@telegraphics.com.au> writes:
>
> Finn> These patches fix some exception handling and autosense bugs that
> Finn> I accidentally introduced in v4.5-rc1.
>
> Finn> drivers/scsi/NCR5380.c | 133 +++++++++++++++++++------------------------
> Finn> drivers/scsi/atari_NCR5380.c | 133 +++++++++++++++++++------------------------
> Finn> 2 files changed, 118 insertions(+), 148 deletions(-)
>
> This is a pretty big lump of changes for a 4.5 bug fix!
I'm sure this is a lot of rework when compared to the high standard set by
well-funded corporate contributions.
However, I don't have anyone paying me to write and execute thorough test
plans for error paths on a wide variety of different hardware platforms.
Nonetheless, these error paths have now been thoroughly tested on several
platforms.
Please keep in mind that these fixes are all rework of the changes I made
in -rc1. So this submission is
2 files changed, 118 insertions(+), 148 deletions(-)
which is all rework of the previous submission, which was
18 files changed, 2940 insertions(+), 3688 deletions(-)
Better than 5% rate of rework. Bugs-per-line-of-code should drop quite
quickly at that rate, presuming fixes get merged.
>
> Given the limited exposure based on the nature of the affected hardware
> I have queued them for 4.6.
>
The affected hardware doesn't matter (though it is likely outside of any
commercial support contract). It's the affected users that interest me.
--
next prev parent reply other threads:[~2016-03-01 3:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-22 23:07 Finn Thain
2016-02-22 23:07 ` [PATCH 1/6] ncr5380: Correctly clear command pointers and lists after bus reset Finn Thain
2016-02-22 23:07 ` [PATCH 2/6] ncr5380: Dont release lock for PIO transfer Finn Thain
2016-02-22 23:07 ` [PATCH 3/6] ncr5380: Dont re-enter NCR5380_select() Finn Thain
2016-02-22 23:07 ` [PATCH 4/6] ncr5380: Forget aborted commands Finn Thain
2016-02-22 23:07 ` [PATCH 5/6] ncr5380: Fix NCR5380_select() EH checks and result handling Finn Thain
2016-02-22 23:07 ` [PATCH 6/6] ncr5380: Call scsi_eh_prep_cmnd() and scsi_eh_restore_cmnd() as and when appropriate Finn Thain
2016-03-01 2:31 ` [PATCH 0/6] ncr5380: Exception handling fixes for v4.5 Martin K. Petersen
2016-03-01 3:32 ` Finn Thain [this message]
2016-03-01 13:00 ` Martin K. Petersen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.LNX.2.00.1603011431300.3734@nippy.intranet \
--to=fthain@telegraphics.com.au \
--cc=James.Bottomley@hansenpartnership.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-m68k@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=schmitzmic@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®