On 03/19/2012 03:17 PM, Toshikazu Nakayama wrote:
> This patch add support fleescale ppce500mc in E500 processor chipset.
> And make platform specific code shape-up so that new support can get easy.
> 
Toshikazu Nakayama-san,

The patches look good to me.

> +     if (machdep->flags & PAE)
>               pte = ULONGLONG(machdep->ptbl + PAGEOFFSET((ulong)page_table));
May be we could introduce a macro to replace

(machdep->flags & PAE)
There are quite a lot of places where we use it now.

Something like,

#define IS_PAE()    (machdep->flags & PAE)


What do you think ?

Reviewed-by: Suzuki K. Poulose <[email protected]>

Thanks
Suzuki

--
Crash-utility mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/crash-utility

Reply via email to