From: Joe Perches <joe@perches.com>
To: Alexey Dobriyan <adobriyan@gmail.com>
Cc: David Miller <davem@davemloft.net>,
mingo@elte.hu, nhorman@tuxdriver.com, simon@fire.lp0.eu,
linux-kernel@vger.kernel.org, akpm@linux-foundation.org,
herbert@gondor.apana.org.au
Subject: Re: BUG: unable to handle kernel NULL pointer dereference at 000000000000000e (reset_prng_context)
Date: Wed, 16 Jul 2008 09:57:26 -0700 [thread overview]
Message-ID: <1216227446.8292.37.camel@localhost> (raw)
In-Reply-To: <20080715214929.GA18268@martell.zuzino.mipt.ru>
On Wed, 2008-07-16 at 01:49 +0400, Alexey Dobriyan wrote:
> On Tue, Jul 15, 2008 at 01:44:07PM -0700, David Miller wrote:
> > From: Ingo Molnar <mingo@elte.hu>
> > > Note that the new crypto/prng.c driver has very bad quality:
> > I think we should merge new drivers as aggressively as possible.
> Well, I don't have strong opinion about this exact statement, but
> Ingo, COULD YOU PLEASE [...] STOP THIS
> CHECKPATCH.PL-AS-INDICATOR HORSESHIT !
Coding style conformity (easy and clinical) and quality
(hard and emotional) are only weakly correlated in software.
I think that Ingo sometimes conflates them and he should be more
selective when using the "quality" term.
I think he recognizes this, he just hasn't done much about it.
http://lkml.org/lkml/2008/2/22/307
Ingo, could you please just substitute conformity for quality
in these sorts of posts? Perhaps you could also change the
name of your code-quality script to style-conformity?
next prev parent reply other threads:[~2008-07-16 17:13 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-14 23:25 BUG: unable to handle kernel NULL pointer dereference at 000000000000000e (reset_prng_context) Simon Arlott
2008-07-15 2:04 ` Neil Horman
2008-07-15 11:33 ` Ingo Molnar
2008-07-15 11:49 ` Ingo Molnar
2008-07-15 11:51 ` Ingo Molnar
2008-07-15 14:21 ` Neil Horman
2008-07-15 20:44 ` David Miller
2008-07-15 21:49 ` Alexey Dobriyan
2008-07-15 22:11 ` Andrew Morton
2008-07-15 22:36 ` Alexey Dobriyan
2008-07-16 0:04 ` Neil Horman
2008-07-16 2:08 ` Herbert Xu
2008-07-16 4:07 ` Herbert Xu
2008-07-16 4:25 ` Andrew Morton
2008-07-16 4:33 ` Herbert Xu
2008-07-16 4:51 ` Andrew Morton
2008-07-16 5:34 ` Herbert Xu
2008-07-16 6:17 ` Andrew Morton
2008-07-16 6:35 ` Herbert Xu
2008-07-16 16:57 ` Joe Perches [this message]
2008-07-18 22:16 ` Ingo Molnar
2008-07-15 2:16 ` Neil Horman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1216227446.8292.37.camel@localhost \
--to=joe@perches.com \
--cc=adobriyan@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=nhorman@tuxdriver.com \
--cc=simon@fire.lp0.eu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.