public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Michael Buesch <mb@bu3sch.de>
To: Ingo Molnar <mingo@elte.hu>
Cc: "John W. Linville" <linville@tuxdriver.com>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	Daniel Walker <dwalker@mvista.com>,
	akpm@linux-foundation.org, stefano.brivio@polimi.it,
	Ray Lee <ray-lk@madrabbit.org>,
	matthias.kaehlcke@gmail.com, linux-kernel@vger.kernel.org,
	mbuesch@freenet.de, linux@bohmer.net, kjwinchester@gmail.com,
	jonathan@jonmasters.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	bcm43xx-dev@lists.berlios.de
Subject: Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semaphore to mutex
Date: Sun, 16 Dec 2007 12:23:00 +0100	[thread overview]
Message-ID: <200712161223.01374.mb@bu3sch.de> (raw)
In-Reply-To: <20071216092243.GB27280@elte.hu>

On Sunday 16 December 2007 10:22:43 Ingo Molnar wrote:
> 
> * John W. Linville <linville@tuxdriver.com> wrote:
> 
> > > It's not that simple.  For example, regression testing will be a 
> > > major PITA if one needs to switch back and forth from the new driver 
> > > to the old one in the process.
> > 
> > Not really true -- a single system can easily have firmware installed 
> > for b43, b43legacy, and bcm43xx at the same time and switch back and 
> > forth between them.
> 
> as long as the version 4 firmware blob is present in the system, will 
> testers have a fully fluid test- and work-flow when migrating across 
> from bcm43xx to b43, without any other changes to an existing Linux 
> installation? (i.e. no udev tweaks, no forced upgrades of components, 
> etc.)
> 
> Will it Just Work in bisection as well, when a tester's kernel 
> flip/flops between bcm43xx and b43 - like it does for the other 3000+ 
> drivers in the kernel?
> 
> Note that we are _NOT_ interested in "might" or "can" scenarios. We are 
> interested in preserving the _existing_ bcm43xx installed base and how 
> much of a seemless migration the b43 transition will be. _THAT_ is what 
> the "no regressions" upstream rule is about, not the "ideal distro" 
> scenario you outline above. It is YOUR total obligation as a kernel 
> maintainer to ensure that you dont break old installations. How hard is 
> that to understand? This is not rocket science.

I see no reason for b43 to break, if the firmware is properly installed.
In fact, almost all installation related bugreports we receive are
caused by missing or incorrectly installed firmware.
I would really _like_ to make installing firmware easier or make the
whole need for it vanish, but I simply can not at this point.
But anyway, installing it is not rocket science, either. The only thing
you have to know is where your distribution stores the firmware image files.
If you know that it's a matter of invoking one b43-fwcutter command
to install it. This process can be automated in the distribution's rpm
or deb package scripts.

b43lagacy/ssb is completely featured with module autoload support.
So if you have firmware installed it will automatically load all required
modules and create the network device(s) for it without any user interaction.

If that doesn't work, then stupid distributions are shipping braindamaged
udev scripts that pin a mac address to a specific driver name (see another
mail in this thread). I can _not_ fix this from within the kernel and
I will absolutely shift all responsibility and blame for this to the
maintainers of the distribution's udev scripts.
That's not a b43 specific problem then. Other drivers do break with these
scripts, too.

-- 
Greetings Michael.

  reply	other threads:[~2007-12-16 11:25 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20071213003023.117964080@mvista.com>
2007-12-12  8:00 ` [PATCH 1/3] ps3: vuart: fix error path locking Daniel Walker
2007-12-13  2:00   ` Geoff Levand
2007-12-19  1:10     ` Andrew Morton
2007-12-19  1:54       ` Daniel Walker
2007-12-19  3:04       ` Geoff Levand
2007-12-20 19:32         ` Daniel Walker
2007-12-20 20:06           ` Andrew Morton
2007-12-20 20:13             ` Daniel Walker
2007-12-12  8:00 ` [PATCH 2/3] driver base: memory: semaphore to mutex Daniel Walker
2007-12-12  8:00 ` [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem " Daniel Walker
2007-12-13  0:48   ` Michael Buesch
2007-12-13  1:17     ` Ray Lee
2007-12-13 13:45       ` Michael Buesch
2007-12-14  0:05         ` Ray Lee
2007-12-14  0:43           ` Michael Buesch
2007-12-14  0:55             ` Harvey Harrison
2007-12-14 10:45               ` Michael Buesch
2007-12-14  1:12             ` Ray Lee
2007-12-14 10:49               ` Michael Buesch
2007-12-14 11:15                 ` Ingo Molnar
2007-12-14 11:39                   ` Michael Buesch
2007-12-14 12:16                     ` Ingo Molnar
2007-12-14 12:31                       ` Michael Buesch
2007-12-14 12:53                         ` Ingo Molnar
2007-12-14 13:08                           ` Michael Buesch
2007-12-14 16:06                             ` Ray Lee
2007-12-14 16:49                               ` Michael Buesch
2007-12-14 18:01                                 ` Ray Lee
2007-12-14 18:11                                   ` Ingo Molnar
2007-12-14 18:47                                     ` Ray Lee
2007-12-14 18:22                                   ` Michael Buesch
2007-12-14 18:45                                     ` Ray Lee
2007-12-14 19:05                                       ` Michael Buesch
2007-12-14 19:25                                         ` Ray Lee
2007-12-14 19:38                                           ` Michael Buesch
2007-12-14 19:55                                             ` Ray Lee
2007-12-14 20:13                                               ` Michael Buesch
2007-12-14 21:39                                                 ` Ray Lee
2007-12-14 12:59                         ` Simon Holm Thøgersen
2007-12-14 13:04                           ` Michael Buesch
2007-12-15  0:51                             ` Rafael J. Wysocki
2007-12-15  0:58                               ` Larry Finger
2007-12-15  1:59                                 ` mvtodevnull
2007-12-15  2:27                                   ` Larry Finger
2007-12-15  4:37                                     ` mvtodevnull
2007-12-15  4:39                                       ` mvtodevnull
2007-12-15  7:18                                         ` Larry Finger
2007-12-15 12:38                                           ` mvtodevnull
2007-12-17  2:17                                             ` mvtodevnull
2007-12-17  6:52                                               ` Larry Finger
2007-12-17  7:17                                                 ` mvtodevnull
2007-12-17  9:49                                                   ` Michael Buesch
2007-12-17 10:15                                                     ` mvtodevnull
2007-12-17 10:35                                                       ` mvtodevnull
2007-12-17 22:04                                                         ` mvtodevnull
2007-12-17 22:45                                                           ` Michael Buesch
2007-12-17 23:12                                                             ` mvtodevnull
2007-12-17 23:18                                                               ` Michael Buesch
2007-12-17 23:27                                                                 ` mvtodevnull
2007-12-18  1:16                                                   ` Larry Finger
2007-12-18  2:34                                                     ` mvtodevnull
2007-12-19  2:12                                                       ` mvtodevnull
2007-12-19  8:11                                                         ` Larry Finger
     [not found]                                                           ` <200712191046.54147.mb@bu3sch.de>
2007-12-19 13:48                                                             ` Ingo Molnar
2007-12-15 10:57                               ` Michael Buesch
2007-12-15 23:18                                 ` Rafael J. Wysocki
2007-12-15 23:27                                   ` Michael Buesch
2007-12-16  2:30                                     ` Larry Finger
2007-12-16 11:13                                       ` Michael Buesch
2007-12-16 13:56                                     ` Johannes Berg
2007-12-16 14:35                                       ` Rafael J. Wysocki
2007-12-16 14:18                                         ` Johannes Berg
2007-12-14  2:09             ` Larry Finger
2007-12-14 10:56             ` Ingo Molnar
2007-12-14 14:14               ` John W. Linville
2007-12-14 14:40                 ` stefano.brivio
2007-12-14 16:27                   ` Ray Lee
2007-12-14 16:45                     ` Ray Lee
2007-12-14 16:59                       ` Michael Buesch
2007-12-14 17:22                         ` Ray Lee
2007-12-14 17:59                         ` Ingo Molnar
2007-12-14 18:38                           ` Michael Buesch
2007-12-15  1:25                             ` Rafael J. Wysocki
2007-12-15 21:43                               ` John W. Linville
2007-12-15 23:26                                 ` Rafael J. Wysocki
2007-12-16  3:20                                   ` Stefan Lippers-Hollmann
2007-12-16  9:22                                 ` Ingo Molnar
2007-12-16 11:23                                   ` Michael Buesch [this message]
2007-12-13  1:20     ` Daniel Walker
2007-12-13 10:13       ` Ingo Molnar
2007-12-13 11:11         ` Michael Buesch
2007-12-13 14:23           ` John W. Linville

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=200712161223.01374.mb@bu3sch.de \
    --to=mb@bu3sch.de \
    --cc=akpm@linux-foundation.org \
    --cc=bcm43xx-dev@lists.berlios.de \
    --cc=dwalker@mvista.com \
    --cc=jonathan@jonmasters.org \
    --cc=kjwinchester@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@bohmer.net \
    --cc=linville@tuxdriver.com \
    --cc=matthias.kaehlcke@gmail.com \
    --cc=mbuesch@freenet.de \
    --cc=mingo@elte.hu \
    --cc=ray-lk@madrabbit.org \
    --cc=rjw@sisk.pl \
    --cc=stefano.brivio@polimi.it \
    --cc=torvalds@linux-foundation.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