Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752391AbXLRSrT (ORCPT ); Tue, 18 Dec 2007 13:47:19 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750872AbXLRSrK (ORCPT ); Tue, 18 Dec 2007 13:47:10 -0500 Received: from smtp2.linux-foundation.org ([207.189.120.14]:46552 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750836AbXLRSrJ (ORCPT ); Tue, 18 Dec 2007 13:47:09 -0500 Date: Tue, 18 Dec 2007 10:45:57 -0800 (PST) From: Linus Torvalds To: Theodore Tso cc: Arjan van de Ven , Tony Luck , Ingo Molnar , linux-kernel@vger.kernel.org, Andrew Morton , protasnb@gmail.com Subject: Re: Top kernel oopses/warnings this week In-Reply-To: <20071218182808.GV7070@thunk.org> Message-ID: References: <4762CF8C.90808@linux.intel.com> <20071217172331.GA23070@elte.hu> <20071217133631.5bbc5842@laptopd505.fenrus.org> <12c511ca0712171458n1bc05b40nab6b4b7c92433c8d@mail.gmail.com> <476703A4.7000401@linux.intel.com> <12c511ca0712171526h242a7de8vadec78cd48747bd@mail.gmail.com> <20071217154727.404e4bc4@laptopd505.fenrus.org> <20071218023141.GM7070@thunk.org> <47676FAE.30200@linux.intel.com> <20071218182808.GV7070@thunk.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 749 Lines: 20 On Tue, 18 Dec 2007, Theodore Tso wrote: > > Well, Matt took over maintenance of the /dev/random driver, but my > take on it is that code readability is more important that saving a > few bytes of generated code or speed; the code paths are only executed > once, so it's hardly a fast path. Quite frankly, I'd argue that while my suggested code wasn't exactly readable, it was more so than the horror it tried to replace. BAD CODE is never readable. At least my suggestion was good code. 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/