![]() | |||||||||||||
Messages in this thread |
On Sun, 14 Dec 2003, Anton Blanchard wrote: > > Pardon my French, but this patch sure looks like crap. > > I agree. Your patch wins, and that comment would even pass akpm best > practices. Heh. I may not comment my code much normally, but for the last few weeks I've been aiming for one-liners, and making sure that I can read every single patch and say with some comfort that I can pretty much _guarantee_ that the patch is right. I'll go back to my lazy ways in 2.7.x, I'm sure. Anyway, even if I think the patch is "obviously correct", can you do me a favor and test it on the load that you've seen failing? Just to be anal. Linus - 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/ | ||||||||||||
| Last update: 2005-03-22 13:59 [from the cache] ©2003-2008 | |||||||||||||