On Wed, 2008-05-28 at 12:20 -0500, James Bottomley wrote:The example that sums it up for me is this one: http://lkml.org/lkml/2008/5/18/20 We have people making minor cosmetic changes, and not paying even the _slightest_ attention to what they're doing. This one's a particularly scary example because it's something even the most non-technical person should have spotted; there's _no_ excuse. It's the cosmetic equivalent of a naïve warning fix that leaves the actual bug in place. I think you're right that the status quo is damaging, and I don't see it getting any better with the current quality of 'janitoring'. I think the only way we can salvage anything useful from the janitors project is to keep a close rein on what tasks are actually undertaken. But we've pushed back on people doing this kind of thing before, and pointed them both at the obvious things they've missed in the context of their patches, and other more useful things they could be doing -- but we've often received responses along the lines of "but I don't want to have to _think_!". It's hard to know where to go from there, and it's not exactly surprising that we end up frustrated. -- dwmw2 --
| Michal Piotrowski | Re: Linux 2.6.21-rc4 |
| Satyam Sharma | [PATCH 0/8] i386: bitops: Cleanup, sanitize, optimize |
| Greg Kroah-Hartman | [PATCH 001/196] Chinese: Add the known_regression URI to the HOWTO |
| David Woodhouse | Re: [bug?] tg3: Failed to load firmware "tigon/tg3_tso.bin" |
git: | |
| Jarek Poplawski | [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Gerrit Renker | [PATCH 15/37] dccp: Set per-connection CCIDs via socket options |
| Alexey Dobriyan | Re: [GIT]: Networking |
| Natalie Protasevich | [BUG] New Kernel Bugs |
