[Xen-devel] misleading names in xen memory management code

["Li, Xin B" <xin.b.li@xxxxxxxxx>]

Keir, I think we have some misleading names in xen memory management
code:
1) #define page_to_pfn(_page)  ((unsigned long)((_page) - frame_table))
Actually, it should be page_to_mfn.

2) #define l1e_from_pfn(pfn, flags)   \
    ((l1_pgentry_t) { ((intpte_t)(pfn) << PAGE_SHIFT) |
put_pte_flags(flags) })
Seems l1e_from_mfn(mfn, flags) is much more clear.

Am I right?

-Xin


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