linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sarah Sharp <sarah.a.sharp@linux.intel.com>
To: Matthias Schniedermeyer <ms@citd.de>,
	Greg KH <gregkh@linuxfoundation.org>
Cc: "Holger Hoffstätte" <holger.hoffstaette@googlemail.com>,
	"Jonathan Nieder" <jrnieder@gmail.com>,
	stable@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-usb@vger.kernel.org, "Tilman Schmidt" <tilman@imap.cc>,
	kernel-team@lists.ubuntu.com,
	"Herton Ronaldo Krzesinski" <herton.krzesinski@canonical.com>
Subject: Re: [regression] external HDD in USB3 enclosure cannot be dynamically removed (Re: Linux 3.7.5)
Date: Wed, 13 Feb 2013 13:08:46 -0800	[thread overview]
Message-ID: <20130213210846.GI8427@xanatos> (raw)
In-Reply-To: <20130213200413.GA17648@citd.de>

On Wed, Feb 13, 2013 at 09:04:13PM +0100, Matthias Schniedermeyer wrote:
> On 13.02.2013 11:33, Sarah Sharp wrote:
> > On Wed, Feb 13, 2013 at 06:16:56PM +0100, Matthias Schniedermeyer wrote:
> > > On 13.02.2013 09:28, Holger Hoffstätte wrote:
> > > > On 12.02.2013 21:42, Sarah Sharp wrote:
> > > > > [..]
> > > > > There was a further set of patches queued for 3.9 to deal with connected
> > > > > devices going to the Inactive state, but they looked like they were too
> > > > > big for stable:
> > > > > 
> > > > > d3b9d7a USB: Fix connected device switch to Inactive state.
> > > > > a24a607 USB: Rip out recursive call on warm port reset.
> > > > > 2d4fa94 USB: Prepare for refactoring by adding extra udev checks.
> > > > > 0fe51aa USB: Don't use EHCI port sempahore for USB 3.0 hubs.
> > 
> > Holger and Matthias, can you double check that cherry picking just those
> > four patches on top of 3.7.7 or 3.8 works as well?
> 
> 3.7.7 + those 4 patches workes for me.
> 
> As i'm not very firm in git i generated separate diffs for each commit 
> and applying them in the order 2d4fa94 0fe51aa d3b9d7a a24a607 worked 
> without hunks dropped.
> 
> I have attached `git diff` against vanilla 3.7.7, so you can check that 
> i got it right.

Yep, that diff looks fine compared to the git diff of those four patches.

Greg,

How do you want to handle this?  The above four patches should go into
3.8 and stable, but they're not currently in Linus' tree and it's
probably too late in the cycle to merge them this week.  Should we just
wait until 3.9 is out and put the patches into the stable trees then?

My email shows that the bad commit
f7965c0846d74b270e246c1470ca955d5078eb07 has been added to the 3.2, 3.4,
and 3.7 stable trees, as well as Canonical's 3.7 stable tree.  I'm also
fine with just reverting that commit from 3.8 and stable.

Sarah Sharp

  reply	other threads:[~2013-02-13 21:08 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-28  4:56 Linux 3.7.5 Greg KH
2013-01-28  4:57 ` Greg KH
     [not found] ` <pan.2013.01.28.11.42.16.556318@googlemail.com>
2013-01-28 14:39   ` Greg KH
     [not found]     ` <pan.2013.01.28.20.33.44.918987@googlemail.com>
2013-01-28 20:46       ` [regression] external HDD in USB3 enclosure cannot be dynamically removed (Re: Linux 3.7.5) Jonathan Nieder
2013-02-12  0:40         ` Sarah Sharp
2013-02-12  0:54           ` Jonathan Nieder
2013-02-12  8:36             ` Holger Hoffstätte
2013-02-12 10:35           ` Holger Hoffstätte
2013-02-12 20:42             ` Sarah Sharp
2013-02-13  8:28               ` Holger Hoffstätte
2013-02-13 17:16                 ` Matthias Schniedermeyer
2013-02-13 19:33                   ` Sarah Sharp
2013-02-13 20:04                     ` Matthias Schniedermeyer
2013-02-13 21:08                       ` Sarah Sharp [this message]
2013-02-13 21:31                         ` Greg KH
2013-02-13 22:12                           ` Sarah Sharp
2013-03-01 16:48                             ` Greg KH
2013-03-05 22:10                               ` Stable backport request, Was: " Sarah Sharp
2013-03-06  2:11                                 ` Greg KH

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=20130213210846.GI8427@xanatos \
    --to=sarah.a.sharp@linux.intel.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=herton.krzesinski@canonical.com \
    --cc=holger.hoffstaette@googlemail.com \
    --cc=jrnieder@gmail.com \
    --cc=kernel-team@lists.ubuntu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=ms@citd.de \
    --cc=stable@vger.kernel.org \
    --cc=tilman@imap.cc \
    /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;
as well as URLs for NNTP newsgroup(s).