Re: [Xen-devel] [PATCH] Cleanup for xen/common/kexec.c

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Mon, 2007-01-22 at 15:08 +0100, Christoph Egger wrote:
> On Monday 22 January 2007 12:57, Ian Campbell wrote:
> > Perhaps you could include the cast in the ELFNOTE_NEXT() macro instead
> > of repeating it everywhere it is used?
> 
> Here we go. Patch attached.

Thanks, applied.

> 
> > Also, could the elfnote macros could be moved to a header? They are used
> > in xen/common/elf.c too.
> 
> They are similar but not equal.

They were roughly equivalent, the one in kexec.c had just further
abstracted ELFNOTE_ALIGN and the one in elf.c had an extra const on
ELFNOTE_NAME.

I moved them into xen/elf.h.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel