Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1346346

Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5

From Finn Thain <fthain@telegraphics.com.au>
Newsgroups linux.kernel
Subject Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5
Date 2016-03-01 04:40 +0100
Message-ID <r7Jke-3EW-11@gated-at.bofh.it> (permalink)
References <r57VL-4Xh-3@gated-at.bofh.it> <r7Io9-2Vu-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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.

-- 

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/6] ncr5380: Exception handling fixes for v4.5 Finn Thain <fthain@telegraphics.com.au> - 2016-02-23 00:20 +0100
  [PATCH 4/6] ncr5380: Forget aborted commands Finn Thain <fthain@telegraphics.com.au> - 2016-02-23 00:20 +0100
  Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5 "Martin K. Petersen" <martin.petersen@oracle.com> - 2016-03-01 03:40 +0100
    Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5 Finn Thain <fthain@telegraphics.com.au> - 2016-03-01 04:40 +0100
      Re: [PATCH 0/6] ncr5380: Exception handling fixes for v4.5 "Martin K. Petersen" <martin.petersen@oracle.com> - 2016-03-01 14:10 +0100

csiph-web