Re: [Xen-devel] [PATCH 00/11] x86: support up to 16Tb

[Keir Fraser <keir@xxxxxxx>]

On 23/01/2013 09:56, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:

>>>> On 23.01.13 at 10:33, Keir Fraser <keir@xxxxxxx> wrote:
>> There's an 'ifdef PAGE_LIST_NULL' in patch 11 in x86/setup.c. Is that really
>> needed?
> 
> That's there so that once we go beyond 16Tb the code won't need
> to change. In particular, because of the implied growth of struct
> page_info, I'm envisioning such support to become optional (to be
> enabled at build time).

Defer the ifdef until it's needed, then when it's added its in the sensible
place (ie. where PAGE_LIST_NULL really does become build-time optional).

 -- Keir

> Jan
> 



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