From: "John W. Linville" <linville@tuxdriver.com>
To: Larry Finger <larry.finger@lwfinger.net>
Cc: Michael Buesch <mb@bu3sch.de>, Tomas Winkler <tomasw@gmail.com>,
stefano.brivio@polimi.it,
Johannes Berg <johannes@sipsolutions.net>,
wireless <linux-wireless@vger.kernel.org>
Subject: Re: mac80211 crash in ieee80211_sta_scan_work
Date: Mon, 28 Jan 2008 13:19:00 -0500 [thread overview]
Message-ID: <20080128181900.GF5835@tuxdriver.com> (raw)
In-Reply-To: <479E14FA.2010206@lwfinger.net>
On Mon, Jan 28, 2008 at 10:46:34AM -0700, Larry Finger wrote:
> Michael Buesch wrote:
>> On Monday 28 January 2008 16:12:00 John W. Linville wrote:
>>> I'm going to hold it back from 2.6.25. We can work on it for 2.6.26.
>> Please apply this patch and apply fixes inside of the 2.6.25 development
>> cycle. We are in a _development_ kernel. Kernels _do_ break in development
>> stages. That is the whole reason why development kernels exist.
> I agree with Michael on this issue. Due to the uncertainty of the quality
> of my Internet connection this winter, I no longer subscribe to
> linux-wireless, and I'm not sure of all the discussion that went into the
> API change. I have, however, seen the FIXME's in b43. In addition, getting
> the code into 2.6.25-rc1 will get a lot more testers.
I think you guys are missing a couple of points...
One is that the 2.6.25 development cycle is closed. The merge window
is now open and when it closes the stabilization period will begin,
leading until the release of 2.6.25 in several weeks or 2-3 months.
Ideally any patches sent in the merge window should already have spent
time in the -mm tree. While that isn't always strictly enforced,
it does imply that huge patches with known breakages and unknown
remedies are not entirely welcome. :-)
The other point is that I am happy to keep this API change in
the wireless-2.6 tree for the 2.6.26 development cycle (which has
effectively already started). Those using wireless-2.6 and -mm will
have plenty of exposure to this patch. There will be plenty of time
to develop on top of this patch while avoiding unnecessary breakage
for normal users.
Please understand that this is the normal process. Since the API
change that was only merged into my tree on the same day as 2.6.24 was
released, merging the patch for 2.6.25 would be the (IMHO unnecessary)
exception.
Thanks
John
--
John W. Linville
linville@tuxdriver.com
next prev parent reply other threads:[~2008-01-28 18:33 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-28 9:07 mac80211 crash in ieee80211_sta_scan_work Larry Finger
2008-01-28 9:29 ` Tomas Winkler
2008-01-28 9:37 ` stefano.brivio
2008-01-28 9:48 ` Tomas Winkler
2008-01-28 15:12 ` John W. Linville
2008-01-28 17:07 ` Michael Buesch
2008-01-28 17:46 ` Larry Finger
2008-01-28 18:19 ` John W. Linville [this message]
2008-01-28 18:39 ` Michael Buesch
2008-01-28 15:18 ` Michael Buesch
2008-01-28 12:35 ` Bas Hulsken
2008-01-28 17:25 ` Jory A. Pratt
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=20080128181900.GF5835@tuxdriver.com \
--to=linville@tuxdriver.com \
--cc=johannes@sipsolutions.net \
--cc=larry.finger@lwfinger.net \
--cc=linux-wireless@vger.kernel.org \
--cc=mb@bu3sch.de \
--cc=stefano.brivio@polimi.it \
--cc=tomasw@gmail.com \
/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).