Re: nested functions used by multiboot2 loader corrupt stack

[Robert Millan]

On Thu, Jan 17, 2008 at 04:15:23PM +0800, Bean wrote:
> 
> You need to add NESTED_FUNC_ATTR to nested callback function that use
> local variable. here is the patch:

Glad to see you found the reason!

But I don't get it.  I see that:

#define NESTED_FUNC_ATTR __attribute__ ((__regparm__ (2)))

so does this mean in one of the calls the caller and callee disagreed about
how the third param is passed?

Also, we have a lot of nested functions without this macro.  How does one
distinguish the ones that need it from the ones that don't?

-- 
Robert Millan

<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call… if you are unable to speak?
(as seen on /.)