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

Re: [Xen-devel] [PATCH 2/2] vm_event: Add altp2m info to HVM events as well



On 06/01/16 11:42, Tamas K Lengyel wrote:


On Wed, Jan 6, 2016 at 12:32 PM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
>>> On 23.12.15 at 15:53, <tamas@xxxxxxxxxxxxx> wrote:
> @@ -83,6 +84,12 @@ static int hvm_event_traps(uint8_t sync, vm_event_request_t *req)
>Â Â Â Â Â vm_event_vcpu_pause(curr);
>Â Â Â }
>
> +Â Â if ( altp2m_active(currd) )
> +Â Â {
> +Â Â Â Â req->flags |= VM_EVENT_FLAG_ALTERNATE_P2M;
> +Â Â Â Â req->altp2m_idx = vcpu_altp2m(curr).p2midx;
> +Â Â }

So far this info was provided just for MEM_ACCESS events. Now
you provide it for a few more ones, but wouldn't this then better
be supplied for all of them, namely also the other two MEM ones?

AFAIK altp2m is currently incompatible with sharing. I'm not 100% sure but I think it's also incompatible with paging.

I don't think they are strictly incompatible; I don't see a technical reason preventing some development work to make them function together.

Whether this happens or not is a very different matter.

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

 


Rackspace

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