Re: [Xen-devel] [PATCH 07/16] arm: avoid placing Xen over any modules.

[Tim Deegan <tim@xxxxxxx>]

At 13:30 +0000 on 03 Sep (1346679047), Ian Campbell wrote:
> @@ -85,17 +126,28 @@ static paddr_t __init get_xen_paddr(void)
>      /* Find the highest bank with enough space. */
>      for ( i = 0; i < mi->nr_banks; i++ )
>      {
> -        if ( mi->bank[i].size >= min_size )
> +        const struct membank *bank = &mi->bank[i];
> +        paddr_t s, e;
> +
> +        if ( bank->size >= min_size )
>          {
> -            t = mi->bank[i].start + mi->bank[i].size - min_size;
> -            if ( t > paddr )
> -                paddr = t;
> +            e = consider_modules(bank->start, bank->start + bank->size,
> +                                 min_size, XEN_PADDR_ALIGN, 0);
> +            if ( !e )continue;

Missing newline.  With that fixed,
Acked-by: Tim Deegan <tim@xxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel