netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Maciej W. Rozycki" <macro@linux-mips.org>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Bill Pemberton <wfp5p@virginia.edu>, netdev@vger.kernel.org
Subject: Re: [PATCH 080/493] fddi: remove use of __devexit_p
Date: Mon, 26 Nov 2012 04:35:27 +0000 (GMT)	[thread overview]
Message-ID: <alpine.LFD.2.02.1211260425240.25230@eddie.linux-mips.org> (raw)
In-Reply-To: <20121122002337.GA16151@kroah.com>

On Wed, 21 Nov 2012, Greg KH wrote:

> >  TURBOchannel, although valid, is an old exotic case that might not be 
> > worth arguing for, except for purity maybe.  But there are surely many 
> > contemporary systems out there that are known they are never going to 
> > support hot device replacement.  Consider most of the embedded systems for 
> > example, where devices may even physically be cast into a single SOC (with 
> > no prospect of chipping off any pieces ever ;) ), that certainly could not 
> > care less of device replacement, but they do care a lot about memory 
> > consumption.
> 
> Even those don't care about less than 5k of memory, do they?

 I guess you're right.  As long as it's not 5k per driver + 
who_knows_how_much per platform for some generic stuff that is.  Of course 
this is still a waste, but I can accept it as a compromise between the use 
of machine resources and the cost of maintenance.

> >  Was this implication considered, discussed and diregarded as not 
> > important enough compared to benefits from hardcoding HOTPLUG support?  
> 
> Yes, I don't know of any modern system that does not enable
> CONFIG_HOTPLUG, do you?

 I've never enabled it outside x86 to be honest.  None of the platforms I 
care of supports it in hardware.

> > I'm seriously asking for a pointer, not trying to cause any stir-up -- 
> > regrettably I fail to follow most discussions these days, but I would like 
> > to know what the background behind this decision was.  Thanks a lot!
> 
> See Russell's response in this thread for details if you are curious.

 Hmm, not in my inbox for some reason (unless I'm blind), but thanks for 
the pointer, I'll see if I can chase it online in some mailing list 
archive.

  Maciej

  reply	other threads:[~2012-11-26  4:35 UTC|newest]

Thread overview: 77+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1353349642-3677-1-git-send-email-wfp5p@virginia.edu>
2012-11-19 18:19 ` [PATCH 010/493] isdn: remove CONFIG_HOTPLUG ifdefs Bill Pemberton
2012-11-19 18:19 ` [PATCH 017/493] wireless: " Bill Pemberton
2012-11-19 18:19 ` [PATCH 018/493] net: " Bill Pemberton
2012-11-19 18:19 ` [PATCH 026/493] atm: remove use of __devexit_p in fore200e.c Bill Pemberton
2012-11-19 18:19 ` [PATCH 029/493] ethernet: remove use of __devexit_p in hydra.c Bill Pemberton
2012-11-19 18:19 ` [PATCH 039/493] atm: remove use of __devexit_p Bill Pemberton
2012-11-19 18:20 ` [PATCH 070/493] isdn: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 078/493] arcnet: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 079/493] can: " Bill Pemberton
2012-11-19 18:45   ` Marc Kleine-Budde
2012-11-19 18:20 ` [PATCH 080/493] fddi: " Bill Pemberton
2012-11-19 18:49   ` Maciej W. Rozycki
2012-11-19 18:57     ` Ben Hutchings
2012-11-19 19:08     ` David Miller
2012-11-19 19:16     ` Maciej W. Rozycki
2012-11-19 19:29       ` Greg KH
2012-11-19 21:22         ` Jan Engelhardt
2012-11-19 21:30           ` Bill Pemberton
2012-11-21 23:49         ` Maciej W. Rozycki
2012-11-22  0:23           ` Greg KH
2012-11-26  4:35             ` Maciej W. Rozycki [this message]
2012-11-26  9:33               ` David Laight
2012-11-19 18:20 ` [PATCH 081/493] hippi: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 082/493] ieee802154: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 083/493] irda: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 084/493] phy: " Bill Pemberton
     [not found] ` <1353349642-3677-1-git-send-email-wfp5p-4Ng6DfrEGID2fBVCVOL8/A@public.gmane.org>
2012-11-19 18:20   ` [PATCH 085/493] net/wireless: " Bill Pemberton
2012-11-19 20:56     ` Arend van Spriel
2012-11-22  6:04     ` Hin-Tak Leung
2012-11-19 18:21   ` [PATCH 126/493] rfkill: " Bill Pemberton
2012-11-19 18:20 ` [PATCH 086/493] net: " Bill Pemberton
2012-11-26 17:35   ` [Pv-drivers] " Dmitry Torokhov
2012-11-19 18:21 ` [PATCH 136/493] ethernet: " Bill Pemberton
2012-11-20  9:31   ` Nicolas Ferre
2012-11-19 18:22 ` [PATCH 189/493] ssb: remove use of __devinit Bill Pemberton
2012-11-19 18:22 ` [PATCH 196/493] net/wireless: " Bill Pemberton
2012-11-19 20:03   ` Larry Finger
     [not found]   ` <1353349642-3677-196-git-send-email-wfp5p-4Ng6DfrEGID2fBVCVOL8/A@public.gmane.org>
2012-11-19 20:57     ` Arend van Spriel
2012-11-22  6:13     ` Hin-Tak Leung
2012-11-19 18:22 ` [PATCH 198/493] ethernet: " Bill Pemberton
2012-11-21 11:55   ` Russell King - ARM Linux
2012-11-19 18:22 ` [PATCH 200/493] irda: " Bill Pemberton
2012-11-19 19:06   ` David Miller
2012-11-19 19:21     ` Greg KH
2012-11-19 19:24       ` David Miller
2012-11-19 19:30         ` Greg KH
2012-11-19 20:25       ` Stephen Hemminger
2012-11-19 18:22 ` [PATCH 201/493] phy: " Bill Pemberton
2012-11-19 18:22 ` [PATCH 202/493] can: " Bill Pemberton
2012-11-19 18:48   ` Marc Kleine-Budde
2012-11-19 18:22 ` [PATCH 203/493] net: " Bill Pemberton
2012-11-26 17:38   ` [Pv-drivers] " Dmitry Torokhov
2012-11-19 18:22 ` [PATCH 204/493] atm: " Bill Pemberton
2012-11-19 18:23 ` [PATCH 232/493] isdn: " Bill Pemberton
2012-11-19 18:23 ` [PATCH 249/493] drivers: connector: " Bill Pemberton
2012-11-19 18:23 ` [PATCH 264/493] rfkill: " Bill Pemberton
2012-11-19 18:24 ` [PATCH 300/493] atm: remove use of __devinitdata Bill Pemberton
2012-11-19 18:24 ` [PATCH 311/493] isdn: " Bill Pemberton
2012-11-19 18:24 ` [PATCH 331/493] ethernet: " Bill Pemberton
2012-11-21 12:08   ` Russell King - ARM Linux
2012-11-19 18:24 ` [PATCH 333/493] net: " Bill Pemberton
2012-11-19 18:50   ` Marc Kleine-Budde
2012-11-19 20:02   ` Larry Finger
2012-11-19 18:25 ` [PATCH 351/493] atm: remove use of __devinitconst Bill Pemberton
2012-11-19 18:25 ` [PATCH 378/493] ethernet: " Bill Pemberton
2012-11-19 18:25 ` [PATCH 379/493] net: " Bill Pemberton
2012-11-19 18:51   ` Marc Kleine-Budde
2012-11-19 18:25 ` [PATCH 389/493] atm: remove use of __devexit Bill Pemberton
2012-11-19 18:25 ` [PATCH 410/493] isdn: " Bill Pemberton
2012-11-19 18:26 ` [PATCH 438/493] drivers: connector: " Bill Pemberton
2012-11-19 18:26 ` [PATCH 470/493] ethernet: " Bill Pemberton
2012-11-19 18:27 ` [PATCH 471/493] net: " Bill Pemberton
2012-11-19 18:56   ` Marc Kleine-Budde
2012-11-19 20:01   ` Larry Finger
2012-11-22  6:11   ` Hin-Tak Leung
2012-11-26 17:39   ` [Pv-drivers] " Dmitry Torokhov
2012-11-19 18:27 ` [PATCH 490/493] rfkill: " Bill Pemberton

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=alpine.LFD.2.02.1211260425240.25230@eddie.linux-mips.org \
    --to=macro@linux-mips.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=netdev@vger.kernel.org \
    --cc=wfp5p@virginia.edu \
    /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).