From: Greg KH <greg@kroah.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: USB development list <linux-usb-devel@lists.sourceforge.net>,
linux-scsi@vger.kernel.org,
USB Storage list <usb-storage@lists.one-eyed-alien.net>,
linux-usb@vger.kernel.org, David Brown <usb-storage2@davidb.org>,
James Bottomley <James.Bottomley@HansenPartnership.com>,
Hans de Goede <j.w.r.degoede@hhs.nl>,
Boaz Harrosh <bharrosh@panasas.com>,
Matthew Dharm <mdharm-scsi@one-eyed-alien.net>
Subject: Re: PATCH: usb-storage-set-last-sector-bug-flag.patch
Date: Wed, 23 Jan 2008 10:12:39 -0800 [thread overview]
Message-ID: <20080123181239.GA21836@kroah.com> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0801201642460.8504-100000@netrider.rowland.org>
On Sun, Jan 20, 2008 at 04:45:36PM -0500, Alan Stern wrote:
> On Sun, 20 Jan 2008, Greg KH wrote:
>
> > On Sun, Jan 20, 2008 at 11:27:29AM +0100, Hans de Goede wrote:
> > > Hi all,
> > >
> > > This patch sets the last_sector_bug flag to 1 for all USB disks. This is
> > > needed to makes the cardreader on various HP multifunction printers work.
> > >
> > > Since the performance impact is negible we set this flag for all USB disks
> > > to avoid an unusual_devs.h nightmare.
> >
> > Oh great, now my "working just fine" USB devices, which happen to have
> > data in the last sector, suddenly stop working.
> >
> > That's not acceptable :(
>
> These patches really should not impact existing devices. If they do
> then something is definitely wrong.
>
> Can you provide detailed logging information showing your problem? For
> example, a usbmon trace would be good. Better yet, a usbmon trace
> without the patches and a usbmon trace with the patches, for
> comparison.
I don't have a device with such a problem, I just am worried that we are
now suddenly keeping access from the last sector for devices that
currently did work just fine.
If you all guarantee that this will not happen, hey, I'm happy, and I'll
gladly point all bug reports on to you all :)
thanks,
greg k-h
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
next prev parent reply other threads:[~2008-01-23 18:12 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-20 10:27 PATCH: usb-storage-set-last-sector-bug-flag.patch Hans de Goede
2008-01-20 20:56 ` Greg KH
[not found] ` <20080120205625.GB28842-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2008-01-20 21:03 ` James Bottomley
[not found] ` <1200863008.3105.30.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2008-01-20 21:25 ` Hans de Goede
2008-01-20 21:39 ` [usb-storage] " Guillaume Bedot
2008-01-20 21:45 ` Alan Stern
2008-01-23 18:12 ` Greg KH [this message]
[not found] ` <20080123181239.GA21836-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2008-01-23 18:39 ` James Bottomley
2008-01-23 18:50 ` Greg KH
2008-01-24 17:07 ` Stefan Richter
2008-01-24 17:21 ` Greg KH
2008-01-25 17:12 ` James Bottomley
2008-01-25 19:23 ` Matthew Dharm
[not found] ` <4798C5B4.1000208-MtYdepGKPcBMYopoZt5u/LNAH6kLmebB@public.gmane.org>
2008-01-24 17:22 ` Stefan Richter
[not found] ` <47932211.90105-fbo2DhPpy/Q@public.gmane.org>
2008-01-22 8:26 ` [usb-storage] " Guillaume Bedot
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=20080123181239.GA21836@kroah.com \
--to=greg@kroah.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=bharrosh@panasas.com \
--cc=j.w.r.degoede@hhs.nl \
--cc=linux-scsi@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
--cc=linux-usb@vger.kernel.org \
--cc=mdharm-scsi@one-eyed-alien.net \
--cc=stern@rowland.harvard.edu \
--cc=usb-storage2@davidb.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.