linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: stable <stable@kernel.org>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	John Linville <linville@tuxdriver.com>
Subject: Re: [stable] [PATCH v2.6.27] mac80211: fix two issues in debugfs
Date: Fri, 10 Oct 2008 09:50:45 -0700	[thread overview]
Message-ID: <20081010165045.GA1309@kroah.com> (raw)
In-Reply-To: <1223656232.3748.28.camel@johannes.berg>

On Fri, Oct 10, 2008 at 06:30:32PM +0200, Johannes Berg wrote:
> On Fri, 2008-10-10 at 09:01 -0700, Greg KH wrote:
> > On Fri, Oct 10, 2008 at 05:52:49PM +0200, Johannes Berg wrote:
> > > This fixes RHBZ 466264, whenever the master interface is
> > > renamed this code would BUG_ON. Also fixes a separately
> > > reported bug with the debugfs dir being NULL.
> > > 
> > > This patch is not applicable to the next kernel version
> > > because both these issues have been fixed, the first one
> > > by not having the master interface have a ieee80211_ptr
> > > at all, and the second one by also leaving the function
> > > early.
> > 
> > So this will not be going into .28, right? 
> 
> No, it won't, the code has changed in ways that won't make this happen
> any more. The fix with the (!dir) part is on it's way, I could split
> that out separately if you'd like, but then we'd have to fix up the
> rejects because that patch looks different too even though the result
> will be very similar.

That's ok, just want to have things clear if people are looking and
notice it's not in Linus's tree when it goes into -stable.


> >  What -stable trees should it
> > go to, .27?  .26?  .25?
> 
> Good question. The latter fix with (!dir) applies to .26 as well, but
> the issue is hit so infrequently (only when adding the debugfs dir has
> failed before) that I'm not sure it's worth it. And the master interface
> issue isn't present before .27.

Ok, so only for .27 is fine with me, I'll queue it up.

thanks,

greg k-h

  reply	other threads:[~2008-10-10 16:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-10 15:52 [PATCH v2.6.27] mac80211: fix two issues in debugfs Johannes Berg
2008-10-10 16:01 ` [stable] " Greg KH
2008-10-10 16:30   ` Johannes Berg
2008-10-10 16:50     ` Greg KH [this message]
2008-10-15 21:38 ` patch mac80211-fix-two-issues-in-debugfs.patch added to 2.6.27-stable tree gregkh

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=20081010165045.GA1309@kroah.com \
    --to=greg@kroah.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=stable@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;
as well as URLs for NNTP newsgroup(s).