[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1163707250.10333.24.camel@localhost.localdomain>
Date: Thu, 16 Nov 2006 21:00:50 +0100
From: Thomas Gleixner <tglx@...esys.com>
To: Alan Stern <stern@...land.harvard.edu>
Cc: LKML <linux-kernel@...r.kernel.org>,
Linus Torvalds <torvalds@...l.org>,
john stultz <johnstul@...ibm.com>, Ingo Molnar <mingo@...e.hu>,
David Miller <davem@...emloft.net>,
Arjan van de Ven <arjan@...radead.org>,
Andrew Morton <akpm@...l.org>, Andi Kleen <ak@...e.de>
Subject: BUG: cpufreq notification broken
[PATCH] cpufreq: make the transition_notifier chain use SRCU
(b4dfdbb3c707474a2254c5b4d7e62be31a4b7da9)
breaks cpu frequency notification users, which register the callback on
core_init level. Interestingly enough the registration survives the
uninitialized head, but the registered user is lost by:
static int __init init_cpufreq_transition_notifier_list(void)
{
srcu_init_notifier_head(&cpufreq_transition_notifier_list);
return 0;
}
core_initcall(init_cpufreq_transition_notifier_list);
This affects i386, x86_64 and sparc64 AFAICT, which call
register_notifier early in the arch code.
> The head of the notifier chain needs to be initialized before use;
> this is done by an __init routine at core_initcall time. If this turns
> out not to be a good choice, it can easily be changed.
Hmm, there are no static initializers for srcu and the only way to fix
this up is to move the arch calls to postcore_init.
tglx
-
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