public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@linux-foundation.org>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [GIT PATCH] intermediate SCSI updates
Date: Mon, 13 Oct 2008 14:22:13 -0700 (PDT)	[thread overview]
Message-ID: <alpine.LFD.2.00.0810131416410.3288@nehalem.linux-foundation.org> (raw)
In-Reply-To: <1223925606.5566.20.camel@localhost.localdomain>



On Mon, 13 Oct 2008, James Bottomley wrote:
> 
> Not exactly.  It has to be rebased to run as a postmerge tree, but it
> does get tested by me (admittedly on my limited set of machines, which
> don't include any actual devices that do block integrity) every time I
> rebase.

What I'm upset about is that this has apparently gotten not even some 
trivial testing of the _default_ build. I'm not talking about any odd 
config options here. I'm literally talking about the only _sane_ config 
option case.

You yourself admit that even you don't have any actual devices that can 
support the block integrity stuff, yet you have apparently only compile- 
tested the insane case of still enabling that thing and apparently nobody 
else has bothered either.

Was this in linux-next?

Is linux-next coverage REALLY so weak that it doesn't even test the 
default config options, much less any random options? What's the point of 
linux-next then?

Again, the date on that thing is claimed to be September 19th, although it 
was obviously committed later.

> However, does this work for you?  It fixes the problem for me.

I could trivially have fixed the compile issue. That's not what upsets me. 
What upsets me is that this set of patches apparently had almost nobody 
looking at them at all before they got sent to me.

If it was some odd and unusual config option, I'd be less upset. hey, 
stuff happens. But it sure as heck was nothing of the sort!

			Linus

  reply	other threads:[~2008-10-13 21:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-13 14:45 [GIT PATCH] intermediate SCSI updates James Bottomley
2008-10-13 17:22 ` Linus Torvalds
2008-10-13 19:20   ` James Bottomley
2008-10-13 21:22     ` Linus Torvalds [this message]
2008-10-13 22:02       ` Paul Mackerras
2008-10-13 22:22       ` Martin K. Petersen
2008-10-13 23:44       ` Stephen Rothwell
2008-10-14 12:49       ` James Bottomley
2008-10-17 15:11 ` [GIT PATCH] intermediate SCSI updates (v2) James Bottomley

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.LFD.2.00.0810131416410.3288@nehalem.linux-foundation.org \
    --to=torvalds@linux-foundation.org \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    /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