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]
Date:	Mon, 21 Jun 2010 22:55:44 +0200
From:	Tejun Heo <tj@...nel.org>
To:	Tony Luck <tony.luck@...el.com>
CC:	Ingo Molnar <mingo@...e.hu>, Peter Zijlstra <peterz@...radead.org>,
	lkml <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH UPDATED] sched: adjust when cpu_active and cpuset 	configurations
 are updated during cpu on/offlining

Hello,

On 06/21/2010 08:28 PM, Tony Luck wrote:
> On Tue, Jun 8, 2010 at 12:46 PM, Tejun Heo <tj@...nel.org> wrote:
>> * Ingo's test discovered __cpuinit/exit markups were incorrect.
>>  Fixed.
> 
> No it isn't :-(

Ah, sorry, my original patch was broken on x86 too, so...

>> +static int __cpuexit cpuset_cpu_active(struct notifier_block *nfb,
>> +                                      unsigned long action, void *hcpu)
> ...
>> +static int __cpuexit cpuset_cpu_inactive(struct notifier_block *nfb,
>> +                                        unsigned long action, void *hcpu)
> 
> This patch arrived in linux-next (tag next-20100621) and breaks the
> ia64 build for configurations where CONFIG_HOTPLUG_CPU=n
> with the following cryptic error:
> 
> `.cpuexit.text' referenced in section `.IA_64.unwind.cpuexit.text' of
> kernel/built-in.o: defined in discarded section `.cpuexit.text' of
> kernel/built-in.o
> 
> This is because ia64 link stage drops __exit functions from
> built-in code (under the logic that they can never be called).
> 
> Is the problem in the !CONFIG_HOTPLUG_CPU definition of
> hotcpu_notifier() in <linux/cpu.h> which still references the
> function argument:
> 
> #define hotcpu_notifier(fn, pri)        do { (void)(fn); } while (0)
> 
> Or should these functions not be marked __cpuexit?

I see.  I think the right solution is removing __cpuexit but it's kind
of silly to have different rules on different architectures.  On x86,
__cpuexit currently means "you can drop it if you're not gonna be
removing cpus after system boot"; IOW, __cpuexit is strict subset of
__cpuinit.  If you define it as "don't include it in the text at all
if cpus are not gonna be removed", it actually forces you to carry
more text in the running system.  Is there any reason ia64 drops them
during linking?

Thanks.

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

Powered by blists - more mailing lists