From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Paul Bolle <pebolle@tiscali.nl>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Alan Stern <stern@rowland.harvard.edu>,
"James E.J. Bottomley" <JBottomley@parallels.com>,
Matthew Dharm <mdharm-usb@one-eyed-alien.net>,
linux-scsi@vger.kernel.org, usb-storage@lists.one-eyed-alien.net,
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/3] scsi: usb-storage: hide errors for five devices
Date: Tue, 15 May 2012 17:36:13 +0100 [thread overview]
Message-ID: <20120515173613.3c8c2f1f@pyramind.ukuu.org.uk> (raw)
In-Reply-To: <1337099344.13455.9.camel@x61.thuisdomein>
On Tue, 15 May 2012 18:29:04 +0200
Paul Bolle <pebolle@tiscali.nl> wrote:
> On Tue, 2012-05-15 at 08:35 -0700, Greg Kroah-Hartman wrote:
> > > > Of course, an easy way out would be to downgrade both the "Asking for
> > > > cache data failed" and the "No Caching mode page present" errors to
> > > > notices. But the SCSI people might disagree with that approach.
> > >
> > > Well, let's see what they say.
> >
> > What ever happened here, are these 3 patches acceptable, or do they need
> > to be reworked or something else?
>
> Nothing happened after Alan's message.
>
> So, currently only the first patch was acceptable (to Alan, that is, but
> it touched the SCSI code).
>
> At least, I haven't seen a reply from the SCSI people. Perhaps the best
> thing to do is to submit a series of one or two trivial patches that
> just downgrade the errors involved to notices. That should lead to
> feedback by the SCSI people.
Possibly we should have a hostadapter flag indicating if the cache noise
is expected nuisance or not. That way if hardcore SCSI on real cables
wants to keeep it then it can still be shut up for USB where its the
norm and just irrelevant noise.
next prev parent reply other threads:[~2012-05-15 16:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-08 10:19 Paul Bolle
2012-05-08 14:02 ` Alan Stern
2012-05-08 15:37 ` Paul Bolle
2012-05-08 17:03 ` Alan Stern
2012-05-15 15:35 ` Greg Kroah-Hartman
2012-05-15 16:29 ` Paul Bolle
2012-05-15 16:36 ` Alan Cox [this message]
2012-05-22 8:48 ` Paul Bolle
2012-05-22 9:25 ` Alan Cox
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=20120515173613.3c8c2f1f@pyramind.ukuu.org.uk \
--to=alan@lxorguk.ukuu.org.uk \
--cc=JBottomley@parallels.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=mdharm-usb@one-eyed-alien.net \
--cc=pebolle@tiscali.nl \
--cc=stern@rowland.harvard.edu \
--cc=usb-storage@lists.one-eyed-alien.net \
/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®