On 29/06/16 17:34, Jan Beulich wrote: >>>> On 29.06.16 at 17:00, <[email protected]> wrote: >> --- a/arch/x86/xen/enlighten.c >> +++ b/arch/x86/xen/enlighten.c >> @@ -591,7 +591,7 @@ static void xen_load_gdt(const struct desc_ptr *dtr) >> { >> unsigned long va = dtr->address; >> unsigned int size = dtr->size + 1; >> - unsigned pages = (size + PAGE_SIZE - 1) / PAGE_SIZE; >> + unsigned pages = DIV_ROUND_UP(size, PAGE_SIZE); >> unsigned long frames[pages]; >> int f; >> >> @@ -640,7 +640,7 @@ static void __init xen_load_gdt_boot(const struct >> desc_ptr *dtr) >> { >> unsigned long va = dtr->address; >> unsigned int size = dtr->size + 1; >> - unsigned pages = (size + PAGE_SIZE - 1) / PAGE_SIZE; >> + unsigned pages = DIV_ROUND_UP(size, PAGE_SIZE); >> unsigned long frames[pages]; >> int f; >> > > Perhaps even more readable would be PFN_DOWN()?
Or PFN_UP() to be correct? Juergen
