linux-kernel - Re: [PATCHv2 3/8] USB: gadget: __init and _

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87hbmtq2ua.fsf@erwin.mina86.com>
Date:	Thu, 29 Apr 2010 16:02:55 -0700 (PDT)
From:	Michal Nazarewicz <mina86@...a86.com>
To:	Greg KH <greg@...ah.com>
Cc:	Michal Nazarewicz <m.nazarewicz@...sung.com>,
	linux-usb@...r.kernel.org,
	Kyungmin Park <kyungmin.park@...sung.com>,
	Marek Szyprowski <m.szyprowski@...sung.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCHv2 3/8] USB: gadget: __init and __exit tags removed

Greg KH <greg@...ah.com> writes:

> On Fri, Apr 09, 2010 at 09:21:20PM +0200, Michal Nazarewicz wrote:
>> __init, __initdata and __exit tags have have been removed from
>> various files to make it possible for gadgets that do not use
>> the __init/__exit tags to use those.
>> 
>> Files in question are related to:
>> * the core composite framework,
>> * the mass storage function (fixing a section mismatch) and
>> * ethernet driver (ACM, ECM, RNDIS).
>> 
>> Signed-off-by: Michal Nazarewicz <m.nazarewicz@...sung.com>
>> Cc: Kyungmin Park <kyungmin.park@...sung.com>
>> Cc: Marek Szyprowski <m.szyprowski@...sung.com>
>> ---
>>  drivers/usb/gadget/composite.c      |   21 ++++++++++-----------
>>  drivers/usb/gadget/config.c         |    4 ++--
>>  drivers/usb/gadget/epautoconf.c     |   12 ++++++------
>>  drivers/usb/gadget/f_acm.c          |   32 ++++++++++++++++----------------
>>  drivers/usb/gadget/f_ecm.c          |   33 +++++++++++++++++----------------
>>  drivers/usb/gadget/f_mass_storage.c |    2 +-
>>  drivers/usb/gadget/f_rndis.c        |   34 ++++++++++++++++++----------------
>>  drivers/usb/gadget/u_ether.c        |    4 ++--
>>  8 files changed, 72 insertions(+), 70 deletions(-)
>> 
>> diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c
>> index 09289bb..ff155ca 100644
>> --- a/drivers/usb/gadget/composite.c
>> +++ b/drivers/usb/gadget/composite.c
>> @@ -85,7 +85,7 @@ MODULE_PARM_DESC(iSerialNumber, "SerialNumber string");
>>   * This function returns the value of the function's bind(), which is
>>   * zero for success else a negative errno value.
>>   */
>> -int __init usb_add_function(struct usb_configuration *config,
>> +int __cold usb_add_function(struct usb_configuration *config,
>
> What is the "__cold" for?

__init, among other things implies __cold:

include/linux/init.h:43: #define __init   __section(.init.text) __cold notrace

and so I left __cold leaving out __section.

-- 
Pozdrawiam                                            _     _
 .o. | Wasal Jasnie Oswieconej Pani Informatyki     o' \,=./ `o
 ..o | Michal "mina86" Nazarewicz  <mina86*tlen.pl>    (o o)
 ooo +---<jid:mina86-jabber.org>---<tlen:mina86>---ooO--(_)--Ooo--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/