netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: cpeterso@cpeterso.com
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, mpm@selenic.com
Subject: Re: [PATCH] drivers/net: remove network drivers' last few uses of IRQF_SAMPLE_RANDOM
Date: Wed, 14 Apr 2010 23:42:42 -0700 (PDT)	[thread overview]
Message-ID: <20100414.234242.159666963.davem@davemloft.net> (raw)
In-Reply-To: <1270877380-23813-1-git-send-email-cpeterso@cpeterso.com>

From: Chris Peterson <cpeterso@cpeterso.com>
Date: Sat, 10 Apr 2010 01:29:40 -0400

> feature-removal-schedule.txt says a new /dev/random entropy model is planned 
> (for July 2009). Until then, this patch removes the few remaining uses of 
> IRQF_SAMPLE_RANDOM from drivers/net/*. 12 network drivers are affected, one 
> more than last year (netxen_nic_main.c). If idle servers need more entropy, 
> they ought to use a hardware RNG or an entropy-gathering userspace daemon.
> 
> Signed-off-by: Chris Peterson <cpeterso@cpeterso.com>

Well, that feature-removal-schedule.txt entry states that new
add_*_randomness() interfaces will be created such that things
like network devices can indicate what kind of entropy they
are providing.

Those interfaces have not yet been created as far as I can tell, and
the idea is that network drivers could use those not-yet-written new
interfaces.

Until that time I feel it does more harm than good to remove these
annotations since frankly there never ever was good agreement about
whether we should do this or not.

It may seem stupid and pointless to just do nothing, but at this time
I think that's still the best thing to do.

Therefore, I'm not applying your patch, sorry.

  reply	other threads:[~2010-04-15  6:42 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-10  5:29 [PATCH] drivers/net: remove network drivers' last few uses of IRQF_SAMPLE_RANDOM Chris Peterson
2010-04-15  6:42 ` David Miller [this message]
  -- strict thread matches above, loose matches on Subject: below --
2008-05-15  7:11 Chris Peterson
2008-05-15 13:21 ` Alan Cox
2008-05-15 16:07   ` Brandeburg, Jesse
2008-05-15 16:39     ` Alan Cox
2008-05-15 18:14       ` Jeff Garzik
2008-05-15 18:47         ` Kok, Auke
2008-05-15 19:10           ` Jeff Garzik
2008-05-15 18:50         ` Rick Jones
2008-05-15 19:11           ` Jeff Garzik
2008-05-16 23:28         ` Rick Jones
2008-05-15 18:04     ` Jeff Garzik
2008-05-15 18:17       ` Rick Jones
2008-05-15 18:31         ` Jeff Garzik
2008-05-15 18:47           ` Kok, Auke
2008-05-15 19:21             ` Jeff Garzik
2008-05-15 20:01               ` Chris Peterson
2008-05-15 20:16                 ` Jeff Garzik
2008-05-15 20:39                   ` Kok, Auke
2008-05-15 21:47                 ` Theodore Tso
2008-05-15 21:58                   ` Jeff Garzik
2008-05-15 22:29                     ` Henrique de Moraes Holschuh
2008-05-15 22:44                       ` Jeff Garzik
2008-05-15 23:02                         ` Henrique de Moraes Holschuh
2008-05-15 23:36                           ` Theodore Tso
2008-05-15 23:46                             ` Henrique de Moraes Holschuh
2008-05-15 23:33                         ` Theodore Tso
2008-05-15 23:58                           ` Henrique de Moraes Holschuh
2008-05-16 13:21               ` Lennart Sorensen
2008-05-16 13:40                 ` Jeff Garzik
2008-05-16 13:59                   ` Will Newton
2008-05-16 14:15                     ` Lennart Sorensen
2008-05-16 14:27                     ` Jeff Garzik
2008-05-16 15:10                 ` Alan Cox
2008-05-16 17:36                   ` Lennart Sorensen
2008-05-16 18:11                     ` Alan Cox
2008-05-16 18:40                       ` Kok, Auke
2008-05-16 18:41                       ` Lennart Sorensen
2008-05-16 18:42                         ` Lennart Sorensen
2008-05-16 20:04                         ` Alan Cox
2008-05-16 20:39                           ` Lennart Sorensen
2008-05-16 20:46                             ` Alan Cox
2008-05-16 20:34                       ` Benny Amorsen
2008-05-25 15:02                         ` Glen Turner
2008-05-25 19:33                           ` Benny Amorsen
2008-05-17  4:55                       ` Chris Peterson
2008-05-25 15:09                         ` Glen Turner
2008-05-25 23:27                           ` Theodore Tso
2008-05-26 13:43                             ` Alejandro Riveira Fernández
2008-05-26 15:14                               ` Bill Fink
2008-05-26 21:07                                 ` Krzysztof Halasa
2008-05-26 21:52                                   ` Bill Fink
2008-05-26 22:11                                     ` Ben Hutchings
2008-05-27 16:44                                 ` Rick Jones
2008-05-30 19:50                                 ` Pavel Machek
2008-05-25 14:55             ` Glen Turner
     [not found]           ` <482C8550 <20080516161029.44ded734@core>
2008-05-16 20:08             ` Gilles Espinasse
2008-05-17 22:02               ` Adrian Bunk
2008-05-15 21:55     ` Adrian Bunk
2008-05-15 22:04       ` Jeff Garzik
2008-05-15 22:27         ` Theodore Tso
2008-05-15 22:13       ` Jesper Juhl
2008-05-15 22:34         ` Theodore Tso
2008-05-15 22:57           ` Jesper Juhl
2008-05-18  0:36       ` Matt Mackall
2008-05-15 22:42     ` Jeff Garzik

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=20100414.234242.159666963.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=cpeterso@cpeterso.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mpm@selenic.com \
    --cc=netdev@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).