linux-kernel - Re: [PATCH] s390 appldata_base: Remove module

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

Message-ID: <alpine.LFD.0.999.0708240419120.14807@enigma.security.iitk.ac.in>
Date:	Fri, 24 Aug 2007 04:21:58 +0530 (IST)
From:	Satyam Sharma <satyam@...radead.org>
To:	Martin Schwidefsky <schwidefsky@...ibm.com>
cc:	Gerald Schaefer <gerald.schaefer@...ibm.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] s390 appldata_base: Remove module_exit function and
 modular stuff

Hi Martin,


On Thu, 23 Aug 2007, Martin Schwidefsky wrote:

> On Thu, 2007-08-23 at 07:51 +0530, Satyam Sharma wrote: 
> > > BTW the __exit module_exit function appldata_exit() of this driver
> > > fails to unregister_hotcpu_notifier() the notifier_block that was
> > > registered by appldata_init() during module startup. This will lead
> > > to oops if hotplug notification comes after module has been unloaded.
> > > Let's fix this by unregistering the notifier appropriately (before
> > > appldata_offline_cpu()'ing the CPUs).
> > 
> > Heh, no wonder we never saw this oops before. This code can never be
> > built as a module! :-) But why does it define and implement an __exit
> > marked module_exit(appldata_exit) function in that case ?!
> 
> I has been a module once for debugging purposes That the module_exit
> function still exists is "historical". We can remove it since it is dead
> code.

Thanks, fair enough. Does this mean you're acking them -- I'd prefer both
the patches to be applied separately. Rationale: we should still fix the
appldata_exit() related oops _before_ removing it so that in future if
this needs to be made modular for debugging purpose again, just the
removal patch can be reverted, but the oops would still be fixed in the
commit before it.


Satyam
-
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/