All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jory A. Pratt" <geekypenguin@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: John Linville <linville@tuxdriver.com>,
	Tomas Winkler <tomasw@gmail.com>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	Stefano Brivio <stefano.brivio@polimi.it>,
	Michael Buesch <mb@bu3sch.de>
Subject: Re: should we revert the cfg80211 API patches?
Date: Wed, 30 Jan 2008 10:16:57 -0600	[thread overview]
Message-ID: <47A0A2F9.3020400@gmail.com> (raw)
In-Reply-To: <1201708352.4149.21.camel@johannes.berg>

Johannes Berg wrote:
> John,
>
> It appears that there's more trouble caused by my cfg80211 band API
> patch than people are willing to put up with. Tomas has already asked
> for it to be reverted, and neither Michael (Buesch) nor Stefano want to
> maintain two driver branches (one "stable" 2.6.25 branch and one
> "development" "for the future" branch).
>
> FWIW, I support your decision to not push this particular patch for
> 2.6.25, it really doesn't fall into the "tested well enough to merge
> during merge window" category.
>
> Additionally, since Michael Wu has (privately) announced to stop working
> on wireless, a number of drivers are effectively unmaintained now and
> I'm not sure I can quickly fix the breakage that my patch probably
> caused in those drivers, especially since Michael Wu is the only
> developer with access to all that hardware.
>
> To ease the short term pain, we can remove/revert the commits in
> question (those being 51c4c94e89a2042e8b20d640b49b6b605d71420d,
> 6854a5291cce341751a7e2e195cc3e97d95afeec and
> d0776155b288c20cc936bfd87d9a76255f244ed8).
>
> Maybe I should have waited longer or posted the patches earlier. I
> didn't post them earlier because I had not wanted to disrupt Intel's
> iwlwifi work too much knowing that there were patches, and then those
> patches caused bad breakage with my patch so I had to wait for another
> Intel patchset fixing a number of bugs they introduced... I'll admit
> that timing was horrible.
>
> But, I'll be frank, if the patches are removed/revert I probably won't
> continue maintaining them. I can't do much with these patches but
> continually forward port them on top of new driver changes which is
> boring and useless work. Experience has shown that hardly anybody but me
> [1] actually tests my patches until I push them into your tree, so
> continuing to forward port these patches won't actually help them become
> better but can only make sure they don't completely bitrot into
> oblivion.
>
> The only way forward I see if these patches are reverted is that we
> announce with the reversion that we'll merge them again in N weeks (with
> N being a reasonably small number, say 4-6) and until then people can
> test the patches and send me driver updates that I'll incorporate. But I
> don't see how useful that is vs. just leaving the patches in place and
> you managing the required driver updates.
>
> johannes
>
> [1] the only other people who test it seem to be mostly clueless people
> who want to get AP mode working and then ask me stupid questions in
> private... there are exceptions of course
>   
Reverting the patch is not the answer. Everyone just needs to step up 
and fix what they can. If maintainers are needed a call needs to be put 
out to find one.
The work is in the direction that moves wireless in linux toward that of 
a windows machine that the users want and need.

-Jory

  parent reply	other threads:[~2008-01-30 16:19 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-30 15:52 should we revert the cfg80211 API patches? Johannes Berg
2008-01-30 16:13 ` Michael Buesch
2008-01-30 16:16 ` Jory A. Pratt [this message]
2008-01-30 16:25   ` Tomas Winkler
2008-01-31 15:13     ` Luis R. Rodriguez

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=47A0A2F9.3020400@gmail.com \
    --to=geekypenguin@gmail.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --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 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.