Re: [PATCH for review] [123/145] i386: make fault notifier unconditional and export it

From: Andi Kleen
Date: Sun Aug 13 2006 - 16:15:29 EST

On Sunday 13 August 2006 17:28, Adrian Bunk wrote:
> > It's needed for external debuggers and overhead is very small.
> >...
> We are currently trying to remove exports not used by any in-kernel
> code.

That ``we'' doesn't include me at least.

> The patch description also lacks the name of what you call "external
> debuggers" (I assume the exports are not for a theoretical usage but for
> an already existing debugger?).

The fault chain is needed for pretty much any debugger, including
kgdb, kdb, nlkd. The one in this case was NLKD.

> Especially nowadays where people demand deprecation periods for removing
> exports without any in-kernel users there must be a _very_ good
> justification when adding such exports.

I've always exported symbols when people can make a reasonable case that they
need it for extern free non broken by design code.

On the other hand I have no problems with removing unused exports
that don't have such a case or are clearly not a useful external

> BTW1: The subject of this email is wrong (it's the x86_64 patch).

Fixed, thanks

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at