linux-kernel - Re: [PATCH 2/2] lib: cleanup dead code from rbtree

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] [day] [month] [year] [list]

Date:   Tue, 19 Dec 2017 23:21:58 +0530
From:   Pravin Shedge <pravin.shedge4linux@...il.com>
To:     Davidlohr Bueso <dave@...olabs.net>
Cc:     Andrew Morton <akpm@...ux-foundation.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] lib: cleanup dead code from rbtree_test.c

On Sun, Dec 17, 2017 at 9:25 PM, Davidlohr Bueso <dave@...olabs.net> wrote:
> On Sun, 17 Dec 2017, Pravin Shedge wrote:
>
>> lib/rbtree_test.c code allows to compile either as a loadable modules or
>> builtin into the kernel.
>>
>> Current code returns -EAGAIN on successful termination from module_init.
>> Such a fail will directly unload the module and hence there is no scope
>> to execute rbtree_test_exit on module_exit.
>>
>> This patch will cleanup dead code from rbtree_test.c
>
>
> Being completely harmless I'd rather leave this as is. If removed, the
> module breaks generic layout.
>
Right this code is not harmless but at the same time I am not sure
How other modules [from lib/test_sort.c] added in kernel ?

Little curious, Just for my knowledge
 how it breaks generic layout ? and what will be the impact ?

> Thanks,
> Davidlohr
>
>
>>
>> Signed-off-by: Pravin Shedge <pravin.shedge4linux@...il.com>
>> ---
>> lib/rbtree_test.c | 6 ------
>> 1 file changed, 6 deletions(-)
>>
>> diff --git a/lib/rbtree_test.c b/lib/rbtree_test.c
>> index 7d36c1e..5cd2f26 100644
>> --- a/lib/rbtree_test.c
>> +++ b/lib/rbtree_test.c
>> @@ -397,13 +397,7 @@ static int __init rbtree_test_init(void)
>>         return -EAGAIN; /* Fail will directly unload the module */
>> }
>>
>> -static void __exit rbtree_test_exit(void)
>> -{
>> -       printk(KERN_ALERT "test exit\n");
>> -}
>> -
>> module_init(rbtree_test_init)
>> -module_exit(rbtree_test_exit)
>>
>> MODULE_LICENSE("GPL");
>> MODULE_AUTHOR("Michel Lespinasse");
>> --
>> 2.7.4
>>
>