Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semaphore to mutex

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Michael Buesch <mb@...>
Cc: Ray Lee <ray-lk@...>, <bcm43xx-dev@...>, Daniel Walker <dwalker@...>, <akpm@...>, <linux-kernel@...>, <linux@...>, <jonathan@...>, <matthias.kaehlcke@...>, <kjwinchester@...>, <mbuesch@...>, John Linville <linville@...>
Date: Friday, December 14, 2007 - 8:16 am

* Michael Buesch <mb@bu3sch.de> wrote:


right in this thread Ray Lee is reporting:

| | Digging a little farther into it, it looks like b43 is barfing 
| | partway through init as the firmware file it's looking for has 
| | changed names. Perhaps that's the issue. I'll take a longer look at 
| | this all tomorrow.

you are really in denial of reality. Just re-read this thread. Upon 
re-reading this thread, try to imagine that you are in place of Ray Lee 
(might be hard), that you had a working bcm43xx driver and that now you 
try to get b43 to work. You are not a kernel hacker who knows this 
driver, just an advanced user who'd like to give you some more feedback 
about your shiny new code. From that perspective, do you think your 
replies were fine, constructive and involved the tester? I sure read 
them as dismissive, they had an annoyed tone (i'm not sure why - he was 
trying to get _YOUR_ code to work) and were borderline arrogant. Looking 
at the replies from Ray Lee it sure seemed to me he had a similar 
impression. In place of Ray Lee, would you report new bugs to the 
maintainer of b43? I sure as hell would avoid it if i could. Do you 
think such incidents help Linux in the long run?

and you even claim:


which questions your basic skills of reading or of empathy. Why is a 
reasonable firmware blob not included in the kernel? If not, why doesnt 
the b43 driver warn in the dmesg (where Ray Lee did look) that no 
firmware was loaded? These are basic driver usability issues, and of 
course they are your fault too.


huh? This is nothing "political". It's the basic rule of maintenance: 
try to be a good maintainer, involve people, forgive their newbie 
mistakes. It's like the driving principle of Intenret protocols: be 
conservative at what you xmit and be liberal at what you rx.

	Ingo
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., Ingo Molnar, (Fri Dec 14, 8:16 am)
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., Rafael J. Wysocki, (Fri Dec 14, 8:51 pm)
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., Rafael J. Wysocki, (Sat Dec 15, 7:18 pm)
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., Rafael J. Wysocki, (Sun Dec 16, 10:35 am)
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., John W. Linville, (Fri Dec 14, 10:14 am)
Re: [PATCH 3/3] net: wireless: bcm43xx: big_buffer_sem semap..., John W. Linville, (Thu Dec 13, 10:23 am)