On Wed, May 22, 2013 at 10:57:33AM +0200, Thomas Renninger wrote:
> Signed-off-by: Thomas Renninger <[email protected]>
> Signed-off-by: Thomas Renninger <Thomas Renninger" [email protected]>
> ---
>  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
> [email protected]
> http://lists.infradead.org/mailman/listinfo/kexec
> 

_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec

Reply via email to