netdev - Re: [PATCH net-next] net: netdev_kobject_init: annotate with _

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 for Android: free password hash cracker in your pocket

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

Message-ID: <52CAE445.5080201@cogentembedded.com>
Date:	Mon, 06 Jan 2014 21:13:41 +0400
From:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:	Daniel Borkmann <dborkman@...hat.com>
CC:	davem@...emloft.net, netdev@...r.kernel.org
Subject: Re: [PATCH net-next] net: netdev_kobject_init: annotate with __init

On 06.01.2014 20:18, Daniel Borkmann wrote:

>>> netdev_kobject_init() is only being called from __init context,
>>> that is, net_dev_init(), so annotate it with __init as well, thus
>>> the kernel can take this as a hint that the function is used only
>>> during the initialization phase and free up used memory resources
>>> after its invocation.

>>> Signed-off-by: Daniel Borkmann <dborkman@...hat.com>
>> [...]

>>> diff --git a/net/core/net-sysfs.h b/net/core/net-sysfs.h
>>> index bd7751e..2745a1b 100644
>>> --- a/net/core/net-sysfs.h
>>> +++ b/net/core/net-sysfs.h
>>> @@ -1,7 +1,7 @@
>>>   #ifndef __NET_SYSFS_H__
>>>   #define __NET_SYSFS_H__
>>>
>>> -int netdev_kobject_init(void);
>>> +int __init netdev_kobject_init(void);

>>     There's no need to also annotate function prototype.

> Hm, is that general convention?

    More or less, though I've seen some prototypes annotated.

> Having this in a header file
> annotated (even if not strictly necessary) would probably
> better prevent from possible misuse resp. section mismatches
> if people forget to look into the actual c file where the
> function is being defined.

    Dunno about vi users, I'm using Alt-. in EMACS when I want to find a 
function, and it gets me right to the definition, not the prototype.

>> WBR, Sergei

PS: The patch was merged already, so my comment seems to have been posted too 
late anyway.

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html