[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] 32-on-64 support in xen-unstable?



>>> Chris Lalancette <clalance@xxxxxxxxxx> 06.07.09 16:18 >>>
>Hello,
>     I've been browsing through the preemptible pagetable stuff, and ran across
>a piece of code that I don't understand or is buggy.  Looking at
>arch/x86/mm.c:new_guest_cr3(), we have this code for 32-on-64 support:

You probably overlooked the == 0 et the end of the expression:

>    if ( is_pv_32on64_domain(d) )
>    {
>        okay = paging_mode_refcounts(d)
>            ? 0 /* Old code was broken, but what should it be? */
>            : mod_l4_entry(
>                    __va(pagetable_get_paddr(curr->arch.guest_table)),
>                    l4e_from_pfn(
>                        mfn,
>                        (_PAGE_PRESENT|_PAGE_RW|_PAGE_USER|_PAGE_ACCESSED)),
>                    pagetable_get_pfn(curr->arch.guest_table), 0, 0, curr) == 
> 0;
                                                                         ^^^^

Which basically is equivalent to what you were trying to suggest.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.