Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-17 Thread Jan Beulich
>>> Konrad Rzeszutek Wilk 04/15/16 2:55 AM >>> >On Thu, Apr 14, 2016 at 09:17:14AM -0600, Jan Beulich wrote: >> >>> Andrew Cooper 04/14/16 5:14 PM >>> >> >On 14/04/16 15:26, Konrad Rzeszutek Wilk wrote: >> >> @@ -312,8 +307,8 @@ struct xsplice_patch_func { >> >> }; >> >> >> >> >> >> -The s

Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-14 Thread Konrad Rzeszutek Wilk
On Thu, Apr 14, 2016 at 09:17:14AM -0600, Jan Beulich wrote: > >>> Andrew Cooper 04/14/16 5:14 PM >>> > >On 14/04/16 15:26, Konrad Rzeszutek Wilk wrote: > >> @@ -312,8 +307,8 @@ struct xsplice_patch_func { > >> }; > >> > >> > >> -The size of the structure is 64 bytes or 52 bytes if compiled

Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-14 Thread Jan Beulich
>>> Andrew Cooper 04/14/16 5:14 PM >>> >On 14/04/16 15:26, Konrad Rzeszutek Wilk wrote: >> @@ -312,8 +307,8 @@ struct xsplice_patch_func { >> }; >> >> >> -The size of the structure is 64 bytes or 52 bytes if compiled under 32-bit >> -hypervisors. >> +The size of the structure is 64 bytes on

Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-14 Thread Andrew Cooper
On 14/04/16 15:26, Konrad Rzeszutek Wilk wrote: > @@ -312,8 +307,8 @@ struct xsplice_patch_func { > }; > > > -The size of the structure is 64 bytes or 52 bytes if compiled under 32-bit > -hypervisors. > +The size of the structure is 64 bytes on 64-bit hypervisors. It will be > +52 on 32-bit

Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-14 Thread Julien Grall
On 14/04/16 15:26, Konrad Rzeszutek Wilk wrote: Julien, Stefano, Hi Konrad, Are you guys OK with this? It does compile under ARM/ARM64 without trouble. If you are OK I will re-instate Julien's Ack on the patch that uses and introduces this: (xsplice: Implement support for applying/reverting/r

Re: [Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-14 Thread Konrad Rzeszutek Wilk
> Julien, Stefano, I dropped Julien's Ack on > > [PATCH v8.1 11/27] xsplice: Implement payload loading > > as Jan's expressed a desire not to have BITS_PER_LONG exposed > in the public headers. I've put CONFIG_ARM_32 in there. On IRC > Stefano was OK (relucantly) with it. > > Patchset has been

[Xen-devel] [PATCH v8.1] xSplice v1 design and implementation.

2016-04-13 Thread Konrad Rzeszutek Wilk
Hey! Changelog: v8: since http://lists.xen.org/archives/html/xen-devel/2016-04/msg01873.html - Posting the _RIGHT_ set of patches. v7: http://lists.xen.org/archives/html/xen-devel/2016-04/msg01476.html - Ingested newer version of x86/mm: Introduce modify_xen_mappings() - Implemented faster sym