Re: PATCH] debug: add notifier chain debugging

From: Tony Luck
Date: Mon Aug 25 2008 - 18:39:26 EST


On Fri, Aug 15, 2008 at 3:29 PM, Arjan van de Ven <arjan@xxxxxxxxxxxxx> wrote:
> + if (!kernel_text_address((unsigned long)n->notifier_call)) {
> + WARN(1, "Invalid notifier registered!");
> + return 0;
> + }

This breaks on ia64 (and pa-risc I think) where function pointers don't point
directly at the code, they point to a {code,data} structure which is itself
located in data space, not text space.

-Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/