On Tue, Nov 03, 2009 at 08:44:59AM +0100, Johannes Berg wrote: > On Tue, 2009-11-03 at 16:16 +0900, Marcel Holtmann wrote: > > > and can we please stop jumping the gun here and going past the subsystem > > maintainers. I think this happens a little bit too much lately. > > I'll rant a bit too -- I've been very annoyed by this many times. Note > this isn't really against you (Dmitry) in particular, just another > case ... but it does tick me off that many times when somebody manages > to blame a failure on a specific commit the first thing they do is ask > somebody way "above" (in terms of patch flow into mainline) the person > writing the patch (like Linus here) to revert it. > I do not understand what the fuss is about. We are pretty far in release process (rc6 is about to be cut I'd expect) and we have an issue that for all practical purposes kills the box on resume. Yes, I want action to be swift in this case and (unless author or maintainer - who were CCed on the email - have otehr solutiuon) the offending commit to be reverted. If it was rc1 or rc2 or 3 I'd feel differently. > It'd help communication and be so much more friendly if the subject was > "found problem with commit ..." instead of "please consider > reverting ..." (which was comparatively friendly already!). You can even > leave the body almost identical, but I think it's presumptuous to > effectively say "hey I know the solution for the problem already". Not at all. I do know the solution since reverting this commit makes by laptop operable again. It may not be the best solution in the long run but a solution nonetheless. > I'll > venture a guess and say that wasn't even the intent, but it certainly > comes across like that if you write an email with this subject, and > start the body with "Hi Linus," not even addressing the patch author, > just adding them to CC out of courtesy. > > Should I think this is accepted practice? > Again, I do not see the problem here. We have a severe regression so yes, I am addressing Linus and asking him to consider reverting bad commit. I also CCing the author and the wireless maintainer so they can object if they have alternative solution. If the problem was in a single driver I might consider doing it differenty but I think this commit affects many wireless drivers. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
