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

[Andrew Cooper <andrew.cooper3@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1148 bytes --]

On 06/01/16 11:42, Tamas K Lengyel wrote:
>
>
> On Wed, Jan 6, 2016 at 12:32 PM, Jan Beulich <JBeulich@suse.com
> <mailto:JBeulich@suse.com>> wrote:
>
>     >>> On 23.12.15 at 15:53, <tamas@tklengyel.com <mailto:tamas@tklengyel.com>> 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

[-- Attachment #1.2: Type: text/html, Size: 2787 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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