On Wed, May 22, 2013 at 10:57:33AM +0200, Thomas Renninger wrote: > Signed-off-by: Thomas Renninger <trenn at suse.de> > Signed-off-by: Thomas Renninger <Thomas Renninger" trenn at suse.de> > --- > kexec/arch/i386/crashdump-x86.c | 10 +++++----- > 1 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/kexec/arch/i386/crashdump-x86.c b/kexec/arch/i386/crashdump-x86.c > index e44fceb..17c8744 100644 > --- a/kexec/arch/i386/crashdump-x86.c > +++ b/kexec/arch/i386/crashdump-x86.c > @@ -559,11 +559,11 @@ static int add_memmap(struct memory_range *memmap_p, unsigned long long addr, > else if (addr > mend) > tidx = i+1; > } > - /* Insert the memory region. */ > - for (j = nr_entries-1; j >= tidx; j--) > - memmap_p[j+1] = memmap_p[j]; > - memmap_p[tidx].start = addr; > - memmap_p[tidx].end = addr + size - 1; > + /* Insert the memory region. */ > + for (j = nr_entries-1; j >= tidx; j--) > + memmap_p[j+1] = memmap_p[j]; While you are at it could you also put spaces around the '-' and '+' in the lines above? > + memmap_p[tidx].start = addr; > + memmap_p[tidx].end = addr + size - 1; > > dbgprintf("Memmap after adding segment\n"); > for (i = 0; i < CRASH_MAX_MEMMAP_NR; i++) { > -- > 1.7.6.1 > > > _______________________________________________ > kexec mailing list > kexec at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/kexec >