Re: [Xen-devel] [PATCH] linux/x86: Advertise which page table entries contain MFNs

[Keir Fraser <keir@xxxxxxxxxxxxx>]

On 13/2/07 15:14, "Jan Beulich" <jbeulich@xxxxxxxxxx> wrote:

> .. and hence need to be (un)canonicalized during save/restore.
> 
> The patch should apply without the page table handling adjustment one, yet
> the values passed in the new note assume that patch to be applied.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>

Could we just define _PAGE_PRESENT in the header file (it's not likely to
change!) rather than have to mess up another couple of header files to allow
inclusion in assembly files.

I see the need to change the ELFNOTE macro, but is the vararg macro syntax
ever unavailable when building Linux? Do you have to keep the old ELFNOTE
definition hanging around?

Why are the extra build rules required, which kill EXTRA_AFLAGS? Has it
something to do with trying to cleanly include pgtable.h?

I think I agree with the change to NEXT_PAGE() but the change belongs
upstream, not in our patches.

 -- Keir


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