On Tue, Jul 09, 2002 at 12:33:13PM -0700, Rick Lindsley wrote:
> Wait! You're still not getting it. I'm not against removing the
> BKL. I'm not saying we should add it to more places. What I am
> complaining about (and I'm not the only one) is the method that
> people who are trying to remove the BKL from various kernel
> subsystems are using.
>
> [ ... ]
>
> So please, no more "hit and run" BKL patches that break things.
> Please come offering to help, with detailed reasons why BKL usage
> is wrong in the specific portions of the code, and how possibly it
> might be cleaned up, with patches that have _actually been
> tested_.
>
> I understand. But there's a bit of a problem here. We don't have
> every device available to us that uses the BKL.
Wait. The patches I specifically mention are in regards to devices that
both of you should have access to. If not, you could have simply walked
over and asked me for one. A single USB mouse or keyboard would have
proved that both the input and the driverfs patches were wrong. Since
the patch was for these types of subsystems, not even attempting to test
it out is indefensible.
> The maintainers, I presume, do.
This too is incorrect. I do not have some devices for drivers that I
have created and maintain. I think this is also true for the majority
of the joystick drivers. Granted, this isn't the best thing, but
companies aren't giving us free devices to write our free drivers for.
We rely on users for testing sometimes (debugging through email does
work, albeit slowly :)
> What is the proper response when a maintainer won't apply a change,
> won't help test it, and won't even help derive a correct one?
Post it to a public mailing list (like lkml) where people in general
care about these things. Enough hounding will either shame the
maintainer into replying, or you might find yourself becoming the
maintainer if no one steps up (like ATM :)
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Mon Jul 15 2002 - 22:00:16 EST