From: Greg KH <gregkh@suse.de>
To: David Miller <davem@davemloft.net>
Cc: david-b@pacbell.net, stern@rowland.harvard.edu,
zwane@arm.linux.org.uk, tytso@mit.edu,
torvalds@linux-foundation.org, stable@kernel.org,
reviews@ml.cw.f00f.org, rdunlap@xenotime.net,
mkrufky@linuxtv.org, linux-usb-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org, jmforbes@linuxtx.org,
dely.l.sy@intel.com, davej@redhat.com, chuckw@quantumlinux.com,
cebbert@redhat.com, cavokz@gmail.com, alan@lxorguk.ukuu.org.uk,
akpm@linux-foundation.org
Subject: Re: [patch 19/26] USB: mutual exclusion for EHCI init and port resets
Date: Mon, 19 Nov 2007 14:52:10 -0800 [thread overview]
Message-ID: <20071119225210.GA23798@suse.de> (raw)
In-Reply-To: <20071119.143241.165308264.davem@davemloft.net>
On Mon, Nov 19, 2007 at 02:32:41PM -0800, David Miller wrote:
> From: David Brownell <david-b@pacbell.net>
> Date: Mon, 19 Nov 2007 11:59:55 -0800
>
> > > This addition certainly won't hurt. Did we ever get any feedback as to
> > > whether it actually helped?
> >
> > ISTR that davem refused to try it, after reporting an
> > intermittent failure on the original patch (which only
> > addresses one end of that hardware race).
> >
> > So, no ... but given we know that closing only one side
> > of the race isn't quite enough, I think we need it anyway.
>
> I think we should put all these changes in because at a minimum
> it's a step in the right direction even if it doesn't close
> the hole %100.
>
> So please let's merge the mutex and the msleep() change in
> ASAP, thanks.
The msleep one is queued up for the next Linus pull, and after it goes
in, I'll add it to the -stable trees as well.
thanks,
greg k-h
next prev parent reply other threads:[~2007-11-19 23:00 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20071119181438.617190424@mini.kroah.org>
2007-11-19 18:17 ` [patch 00/26] 2.6.22-stable review Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 01/26] SLUB: Fix memory leak by not reusing cpu_slab Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 02/26] Fix compat futex hangs Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 03/26] x86: fix TSC clock source calibration error Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 04/26] writeback: dont propagate AOP_WRITEPAGE_ACTIVATE Greg Kroah-Hartman
2007-11-19 19:04 ` Hugh Dickins
2007-11-19 23:05 ` [stable] " Greg KH
2007-11-19 18:18 ` [patch 05/26] fix param_sysfs_builtin name length check Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 06/26] NETFILTER: nf_conntrack_tcp: fix connection reopening Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 07/26] fix the softlockup watchdog to actually work Greg Kroah-Hartman
2007-11-19 19:02 ` Ingo Molnar
2007-11-19 23:02 ` [stable] " Greg KH
2007-11-19 18:18 ` [patch 08/26] Fix TEQL oops Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 09/26] Fix netlink timeouts Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 10/26] Fix error returns in sys_socketpair() Greg Kroah-Hartman
2007-11-19 18:18 ` [patch 11/26] Fix endianness bug in U32 classifier Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 12/26] Fix crypto_alloc_comp() error checking Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 13/26] ALSA: hdsp - Fix zero division Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 14/26] ALSA: hda-codec - Add array terminator for dmic in STAC codec Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 15/26] forcedeth msi bugfix Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 16/26] hptiop: avoid buffer overflow when returning sense data Greg Kroah-Hartman
2007-11-19 18:38 ` Matthew Wilcox
2007-11-19 23:03 ` [stable] " Greg KH
2007-11-19 18:19 ` [patch 17/26] USB: kobil_sct: trivial backport to fix libct Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 18/26] USB: usbserial - fix potential deadlock between write() and IRQ Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 19/26] USB: mutual exclusion for EHCI init and port resets Greg Kroah-Hartman
2007-11-19 18:41 ` David Brownell
2007-11-19 18:43 ` Greg KH
2007-11-19 19:04 ` Alan Stern
2007-11-19 19:59 ` David Brownell
2007-11-19 22:32 ` David Miller
2007-11-19 22:52 ` Greg KH [this message]
2007-11-19 18:19 ` [patch 20/26] i4l: Fix random hard freeze with AVM c4 card Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 21/26] i4l: fix random freezes with AVM B1 drivers Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 22/26] ide: fix serverworks.c UDMA regression Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 23/26] ocfs2: fix write() performance regression Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 24/26] i2c-pasemi: Fix NACK detection Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 25/26] i2c/eeprom: Hide Sony Vaio serial numbers Greg Kroah-Hartman
2007-11-19 18:19 ` [patch 26/26] i2c/eeprom: Recognize VGN as a valid Sony Vaio name prefix Greg Kroah-Hartman
2007-11-19 18:22 ` [patch 00/26] 2.6.22-stable review Greg Kroah-Hartman
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=20071119225210.GA23798@suse.de \
--to=gregkh@suse.de \
--cc=akpm@linux-foundation.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=cavokz@gmail.com \
--cc=cebbert@redhat.com \
--cc=chuckw@quantumlinux.com \
--cc=davej@redhat.com \
--cc=davem@davemloft.net \
--cc=david-b@pacbell.net \
--cc=dely.l.sy@intel.com \
--cc=jmforbes@linuxtx.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
--cc=mkrufky@linuxtv.org \
--cc=rdunlap@xenotime.net \
--cc=reviews@ml.cw.f00f.org \
--cc=stable@kernel.org \
--cc=stern@rowland.harvard.edu \
--cc=torvalds@linux-foundation.org \
--cc=tytso@mit.edu \
--cc=zwane@arm.linux.org.uk \
/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.