From: "Maciej W. Rozycki" <macro@linux-mips.org>
To: Bill Pemberton <wfp5p@virginia.edu>
Cc: gregkh@linuxfoundation.org, netdev@vger.kernel.org
Subject: Re: [PATCH 080/493] fddi: remove use of __devexit_p
Date: Mon, 19 Nov 2012 19:16:44 +0000 (GMT) [thread overview]
Message-ID: <alpine.LFD.2.02.1211191912040.25230@eddie.linux-mips.org> (raw)
In-Reply-To: <alpine.LFD.2.02.1211191842180.25230@eddie.linux-mips.org>
On Mon, 19 Nov 2012, Maciej W. Rozycki wrote:
> > CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
> > needed.
>
> Shouldn't this be switching to __exit_p() instead? Likewise all the
> other changes concerned (i.e. s/__dev\(init\|exit\)/__\1/), for the sake
> of drivers built into the kernel proper (yes, there are people out there
> still doing that). Am I missing something?
>
> This change scores my NAK as it stands, until updated accordingly or
> further justified.
I have unconfused myself now, so please replace the above with the
following question: what about configurations (e.g. buses) that not
support hotplug at all? For example apart from PCI the defxx driver
concerned here supports the TURBOchannel bus that by design does not have
the concept of live option card removal (no such circuitry). So should
now the precious memory be wasted on systems that will never ever handle
hotplug?
Maciej
next prev parent reply other threads:[~2012-11-19 19:16 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 [this message]
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
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
2012-11-19 18:20 ` [PATCH 086/493] net: " Bill Pemberton
2012-11-26 17:35 ` [Pv-drivers] " Dmitry Torokhov
[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: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.1211191912040.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).