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

["Christoph Egger" <Christoph.Egger@amd.com>]

[-- Attachment #1: Type: text/plain, Size: 1118 bytes --]

On Monday 22 January 2007 12:57, Ian Campbell wrote:
> On Mon, 2007-01-22 at 10:48 +0100, Christoph Egger wrote:
> >  #define ELFNOTE_ALIGN(_n_) (((_n_)+3)&~3)
> > -#define ELFNOTE_NAME(_n_) ((void*)(_n_) + sizeof(*(_n_)))
> > +#define ELFNOTE_NAME(_n_) ((char*)(_n_) + sizeof(*(_n_)))
> >  #define ELFNOTE_DESC(_n_) (ELFNOTE_NAME(_n_) +
> > ELFNOTE_ALIGN((_n_)->namesz)) #define ELFNOTE_NEXT(_n_)
> > (ELFNOTE_DESC(_n_) + ELFNOTE_ALIGN((_n_)->descsz))
> >
> > @@ -75,10 +75,10 @@ void kexec_crash_save_cpu(void)
> >      if ( cpu_test_and_set(cpu, crash_saved_cpus) )
> >          return;
> >
> > -    prstatus = ELFNOTE_DESC(note);
> > -
> > -    note = ELFNOTE_NEXT(note);
> > -    xencore = ELFNOTE_DESC(note);
> > +    prstatus = (ELF_Prstatus *)ELFNOTE_DESC(note);
> > +
> > +    note = (Elf_Note *)ELFNOTE_NEXT(note);
>
> Perhaps you could include the cast in the ELFNOTE_NEXT() macro instead
> of repeating it everywhere it is used?

Here we go. Patch attached.

> 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.


Christoph

[-- Attachment #2: xen_kexec.diff --]
[-- Type: text/x-diff, Size: 1407 bytes --]

diff -r 7e28a8c150ed xen/common/kexec.c
--- a/xen/common/kexec.c	Sat Jan 20 14:33:43 2007 +0000
+++ b/xen/common/kexec.c	Mon Jan 22 13:30:18 2007 +0100
@@ -27,9 +27,9 @@ typedef long ret_t;
 typedef long ret_t;
 
 #define ELFNOTE_ALIGN(_n_) (((_n_)+3)&~3)
-#define ELFNOTE_NAME(_n_) ((void*)(_n_) + sizeof(*(_n_)))
+#define ELFNOTE_NAME(_n_) ((char*)(_n_) + sizeof(*(_n_)))
 #define ELFNOTE_DESC(_n_) (ELFNOTE_NAME(_n_) + ELFNOTE_ALIGN((_n_)->namesz))
-#define ELFNOTE_NEXT(_n_) (ELFNOTE_DESC(_n_) + ELFNOTE_ALIGN((_n_)->descsz))
+#define ELFNOTE_NEXT(_n_) ((Elf_Note *)(ELFNOTE_DESC(_n_) + ELFNOTE_ALIGN((_n_)->descsz)))
 
 static DEFINE_PER_CPU(void *, crash_notes);
 
@@ -75,10 +75,10 @@ void kexec_crash_save_cpu(void)
     if ( cpu_test_and_set(cpu, crash_saved_cpus) )
         return;
 
-    prstatus = ELFNOTE_DESC(note);
+    prstatus = (ELF_Prstatus *)ELFNOTE_DESC(note);
 
     note = ELFNOTE_NEXT(note);
-    xencore = ELFNOTE_DESC(note);
+    xencore = (crash_xen_core_t *)ELFNOTE_DESC(note);
 
     elf_core_save_regs(&prstatus->pr_reg, xencore);
 }
@@ -87,7 +87,7 @@ crash_xen_info_t *kexec_crash_save_info(
 crash_xen_info_t *kexec_crash_save_info(void)
 {
     int cpu = smp_processor_id();
-    crash_xen_info_t *info = ELFNOTE_DESC(xen_crash_note);
+    crash_xen_info_t *info = (crash_xen_info_t *)ELFNOTE_DESC(xen_crash_note);
 
     BUG_ON(!cpu_test_and_set(cpu, crash_saved_cpus));
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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