* [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
@ 2018-09-27 7:58 Razvan Cojocaru
2018-10-03 13:14 ` Wei Liu
` (3 more replies)
0 siblings, 4 replies; 11+ messages in thread
From: Razvan Cojocaru @ 2018-09-27 7:58 UTC (permalink / raw)
To: xen-devel
Cc: tamas, wei.liu2, Razvan Cojocaru, konrad.wilk, George.Dunlap,
andrew.cooper3, Adrian Pop, tim, julien.grall, sstabellini,
jbeulich, ian.jackson
Currently there is a subop for setting the memaccess of a page, but not
for consulting it. The new HVMOP_altp2m_get_mem_access adds this
functionality.
Both altp2m get/set mem access functions use the struct
xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
been renamed from xen_hvm_altp2m_set_mem_access.
Signed-off-by: Adrian Pop <apop@bitdefender.com>
Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
---
Changes since V5:
- Fixed the build by conditionally-compiling the altp2m code
gated on CONFIG_HVM being #defined.
---
tools/libxc/include/xenctrl.h | 3 +++
tools/libxc/xc_altp2m.c | 33 ++++++++++++++++++++++++++++++---
xen/arch/arm/mem_access.c | 7 +++++--
xen/arch/x86/hvm/hvm.c | 30 ++++++++++++++++++++++++------
xen/arch/x86/mm/mem_access.c | 21 ++++++++++++++++++++-
xen/common/mem_access.c | 2 +-
xen/include/public/hvm/hvm_op.h | 21 ++++++++++++++++++++-
xen/include/public/xen-compat.h | 2 +-
xen/include/xen/mem_access.h | 3 ++-
9 files changed, 106 insertions(+), 16 deletions(-)
diff --git a/tools/libxc/include/xenctrl.h b/tools/libxc/include/xenctrl.h
index dad96a9..618f3cb 100644
--- a/tools/libxc/include/xenctrl.h
+++ b/tools/libxc/include/xenctrl.h
@@ -1949,6 +1949,9 @@ int xc_altp2m_set_mem_access(xc_interface *handle, uint32_t domid,
int xc_altp2m_set_mem_access_multi(xc_interface *handle, uint32_t domid,
uint16_t view_id, uint8_t *access,
uint64_t *gfns, uint32_t nr);
+int xc_altp2m_get_mem_access(xc_interface *handle, uint32_t domid,
+ uint16_t view_id, xen_pfn_t gfn,
+ xenmem_access_t *access);
int xc_altp2m_change_gfn(xc_interface *handle, uint32_t domid,
uint16_t view_id, xen_pfn_t old_gfn,
xen_pfn_t new_gfn);
diff --git a/tools/libxc/xc_altp2m.c b/tools/libxc/xc_altp2m.c
index be5bfd2..844b9f1 100644
--- a/tools/libxc/xc_altp2m.c
+++ b/tools/libxc/xc_altp2m.c
@@ -226,9 +226,9 @@ int xc_altp2m_set_mem_access(xc_interface *handle, uint32_t domid,
arg->version = HVMOP_ALTP2M_INTERFACE_VERSION;
arg->cmd = HVMOP_altp2m_set_mem_access;
arg->domain = domid;
- arg->u.set_mem_access.view = view_id;
- arg->u.set_mem_access.hvmmem_access = access;
- arg->u.set_mem_access.gfn = gfn;
+ arg->u.mem_access.view = view_id;
+ arg->u.mem_access.access = access;
+ arg->u.mem_access.gfn = gfn;
rc = xencall2(handle->xcall, __HYPERVISOR_hvm_op, HVMOP_altp2m,
HYPERCALL_BUFFER_AS_ARG(arg));
@@ -303,3 +303,30 @@ int xc_altp2m_set_mem_access_multi(xc_interface *xch, uint32_t domid,
return rc;
}
+
+int xc_altp2m_get_mem_access(xc_interface *handle, uint32_t domid,
+ uint16_t view_id, xen_pfn_t gfn,
+ xenmem_access_t *access)
+{
+ int rc;
+ DECLARE_HYPERCALL_BUFFER(xen_hvm_altp2m_op_t, arg);
+
+ arg = xc_hypercall_buffer_alloc(handle, arg, sizeof(*arg));
+ if ( arg == NULL )
+ return -1;
+
+ arg->version = HVMOP_ALTP2M_INTERFACE_VERSION;
+ arg->cmd = HVMOP_altp2m_get_mem_access;
+ arg->domain = domid;
+ arg->u.mem_access.view = view_id;
+ arg->u.mem_access.gfn = gfn;
+
+ rc = xencall2(handle->xcall, __HYPERVISOR_hvm_op, HVMOP_altp2m,
+ HYPERCALL_BUFFER_AS_ARG(arg));
+
+ if ( !rc )
+ *access = arg->u.mem_access.access;
+
+ xc_hypercall_buffer_free(handle, arg);
+ return rc;
+}
diff --git a/xen/arch/arm/mem_access.c b/xen/arch/arm/mem_access.c
index ba4ec78..653d960 100644
--- a/xen/arch/arm/mem_access.c
+++ b/xen/arch/arm/mem_access.c
@@ -236,7 +236,7 @@ bool p2m_mem_access_check(paddr_t gpa, vaddr_t gla, const struct npfec npfec)
if ( !p2m->mem_access_enabled )
return true;
- rc = p2m_get_mem_access(v->domain, gaddr_to_gfn(gpa), &xma);
+ rc = p2m_get_mem_access(v->domain, gaddr_to_gfn(gpa), &xma, 0);
if ( rc )
return true;
@@ -441,11 +441,14 @@ long p2m_set_mem_access_multi(struct domain *d,
}
int p2m_get_mem_access(struct domain *d, gfn_t gfn,
- xenmem_access_t *access)
+ xenmem_access_t *access, unsigned int altp2m_idx)
{
int ret;
struct p2m_domain *p2m = p2m_get_hostp2m(d);
+ /* altp2m is not yet implemented on Arm. The altp2m_idx should be 0. */
+ ASSERT(altp2m_idx == 0);
+
p2m_read_lock(p2m);
ret = __p2m_get_mem_access(d, gfn, access);
p2m_read_unlock(p2m);
diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c
index 51fc3ec..8a9abf3 100644
--- a/xen/arch/x86/hvm/hvm.c
+++ b/xen/arch/x86/hvm/hvm.c
@@ -4473,6 +4473,7 @@ static int do_altp2m_op(
case HVMOP_altp2m_get_suppress_ve:
case HVMOP_altp2m_set_mem_access:
case HVMOP_altp2m_set_mem_access_multi:
+ case HVMOP_altp2m_get_mem_access:
case HVMOP_altp2m_change_gfn:
break;
@@ -4600,8 +4601,8 @@ static int do_altp2m_op(
rc = -EINVAL;
else
{
- gfn_t gfn = _gfn(a.u.set_mem_access.gfn);
- unsigned int altp2m_idx = a.u.set_mem_access.view;
+ gfn_t gfn = _gfn(a.u.mem_access.gfn);
+ unsigned int altp2m_idx = a.u.mem_access.view;
bool suppress_ve = a.u.suppress_ve.suppress_ve;
rc = p2m_set_suppress_ve(d, gfn, suppress_ve, altp2m_idx);
@@ -4627,12 +4628,12 @@ static int do_altp2m_op(
break;
case HVMOP_altp2m_set_mem_access:
- if ( a.u.set_mem_access.pad )
+ if ( a.u.mem_access.pad )
rc = -EINVAL;
else
- rc = p2m_set_mem_access(d, _gfn(a.u.set_mem_access.gfn), 1, 0, 0,
- a.u.set_mem_access.hvmmem_access,
- a.u.set_mem_access.view);
+ rc = p2m_set_mem_access(d, _gfn(a.u.mem_access.gfn), 1, 0, 0,
+ a.u.mem_access.access,
+ a.u.mem_access.view);
break;
case HVMOP_altp2m_set_mem_access_multi:
@@ -4668,6 +4669,23 @@ static int do_altp2m_op(
}
break;
+ case HVMOP_altp2m_get_mem_access:
+ if ( a.u.mem_access.pad )
+ rc = -EINVAL;
+ else
+ {
+ xenmem_access_t access;
+
+ rc = p2m_get_mem_access(d, _gfn(a.u.mem_access.gfn), &access,
+ a.u.mem_access.view);
+ if ( !rc )
+ {
+ a.u.mem_access.access = access;
+ rc = __copy_to_guest(arg, &a, 1) ? -EFAULT : 0;
+ }
+ }
+ break;
+
case HVMOP_altp2m_change_gfn:
if ( a.u.change_gfn.pad1 || a.u.change_gfn.pad2 )
rc = -EINVAL;
diff --git a/xen/arch/x86/mm/mem_access.c b/xen/arch/x86/mm/mem_access.c
index 3d50fe0..2b5a621 100644
--- a/xen/arch/x86/mm/mem_access.c
+++ b/xen/arch/x86/mm/mem_access.c
@@ -486,10 +486,29 @@ long p2m_set_mem_access_multi(struct domain *d,
return rc;
}
-int p2m_get_mem_access(struct domain *d, gfn_t gfn, xenmem_access_t *access)
+int p2m_get_mem_access(struct domain *d, gfn_t gfn, xenmem_access_t *access,
+ unsigned int altp2m_idx)
{
struct p2m_domain *p2m = p2m_get_hostp2m(d);
+#ifdef CONFIG_HVM
+ if ( !altp2m_active(d) )
+ {
+ if ( altp2m_idx )
+ return -EINVAL;
+ }
+ else
+ {
+ if ( altp2m_idx >= MAX_ALTP2M ||
+ d->arch.altp2m_eptp[altp2m_idx] == mfn_x(INVALID_MFN) )
+ return -EINVAL;
+
+ p2m = d->arch.altp2m_p2m[altp2m_idx];
+ }
+#else
+ ASSERT(!altp2m_idx);
+#endif
+
return _p2m_get_mem_access(p2m, gfn, access);
}
diff --git a/xen/common/mem_access.c b/xen/common/mem_access.c
index 1bf6824..010e6f8 100644
--- a/xen/common/mem_access.c
+++ b/xen/common/mem_access.c
@@ -99,7 +99,7 @@ int mem_access_memop(unsigned long cmd,
if ( (mao.pfn > domain_get_maximum_gpfn(d)) && mao.pfn != ~0ull )
break;
- rc = p2m_get_mem_access(d, _gfn(mao.pfn), &access);
+ rc = p2m_get_mem_access(d, _gfn(mao.pfn), &access, 0);
if ( rc != 0 )
break;
diff --git a/xen/include/public/hvm/hvm_op.h b/xen/include/public/hvm/hvm_op.h
index cf00cad..5878a25 100644
--- a/xen/include/public/hvm/hvm_op.h
+++ b/xen/include/public/hvm/hvm_op.h
@@ -242,17 +242,31 @@ struct xen_hvm_altp2m_view {
typedef struct xen_hvm_altp2m_view xen_hvm_altp2m_view_t;
DEFINE_XEN_GUEST_HANDLE(xen_hvm_altp2m_view_t);
+#if __XEN_INTERFACE_VERSION__ < 0x00040a00
struct xen_hvm_altp2m_set_mem_access {
/* view */
uint16_t view;
/* Memory type */
- uint16_t hvmmem_access; /* xenmem_access_t */
+ uint16_t access; /* xenmem_access_t */
uint32_t pad;
/* gfn */
uint64_t gfn;
};
typedef struct xen_hvm_altp2m_set_mem_access xen_hvm_altp2m_set_mem_access_t;
DEFINE_XEN_GUEST_HANDLE(xen_hvm_altp2m_set_mem_access_t);
+#endif /* __XEN_INTERFACE_VERSION__ < 0x00040a00 */
+
+struct xen_hvm_altp2m_mem_access {
+ /* view */
+ uint16_t view;
+ /* Memory type */
+ uint16_t access; /* xenmem_access_t */
+ uint32_t pad;
+ /* gfn */
+ uint64_t gfn;
+};
+typedef struct xen_hvm_altp2m_mem_access xen_hvm_altp2m_mem_access_t;
+DEFINE_XEN_GUEST_HANDLE(xen_hvm_altp2m_mem_access_t);
struct xen_hvm_altp2m_set_mem_access_multi {
/* view */
@@ -308,6 +322,8 @@ struct xen_hvm_altp2m_op {
#define HVMOP_altp2m_set_suppress_ve 10
/* Get the "Suppress #VE" bit of a page */
#define HVMOP_altp2m_get_suppress_ve 11
+/* Get the access of a page of memory from a certain view */
+#define HVMOP_altp2m_get_mem_access 12
domid_t domain;
uint16_t pad1;
uint32_t pad2;
@@ -315,7 +331,10 @@ struct xen_hvm_altp2m_op {
struct xen_hvm_altp2m_domain_state domain_state;
struct xen_hvm_altp2m_vcpu_enable_notify enable_notify;
struct xen_hvm_altp2m_view view;
+#if __XEN_INTERFACE_VERSION__ < 0x00040a00
struct xen_hvm_altp2m_set_mem_access set_mem_access;
+#endif /* __XEN_INTERFACE_VERSION__ < 0x00040a00 */
+ struct xen_hvm_altp2m_mem_access mem_access;
struct xen_hvm_altp2m_change_gfn change_gfn;
struct xen_hvm_altp2m_set_mem_access_multi set_mem_access_multi;
struct xen_hvm_altp2m_suppress_ve suppress_ve;
diff --git a/xen/include/public/xen-compat.h b/xen/include/public/xen-compat.h
index b673653..fa6ffb7 100644
--- a/xen/include/public/xen-compat.h
+++ b/xen/include/public/xen-compat.h
@@ -27,7 +27,7 @@
#ifndef __XEN_PUBLIC_XEN_COMPAT_H__
#define __XEN_PUBLIC_XEN_COMPAT_H__
-#define __XEN_LATEST_INTERFACE_VERSION__ 0x00040900
+#define __XEN_LATEST_INTERFACE_VERSION__ 0x00040a00
#if defined(__XEN__) || defined(__XEN_TOOLS__)
/* Xen is built with matching headers and implements the latest interface. */
diff --git a/xen/include/xen/mem_access.h b/xen/include/xen/mem_access.h
index 28cab67..e4d2450 100644
--- a/xen/include/xen/mem_access.h
+++ b/xen/include/xen/mem_access.h
@@ -82,7 +82,8 @@ int p2m_get_suppress_ve(struct domain *d, gfn_t gfn, bool *suppress_ve,
* Get access type for a gfn.
* If gfn == INVALID_GFN, gets the default access type.
*/
-int p2m_get_mem_access(struct domain *d, gfn_t gfn, xenmem_access_t *access);
+int p2m_get_mem_access(struct domain *d, gfn_t gfn, xenmem_access_t *access,
+ unsigned int altp2m_idx);
#ifdef CONFIG_MEM_ACCESS
int mem_access_memop(unsigned long cmd,
--
2.7.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-09-27 7:58 [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page Razvan Cojocaru
@ 2018-10-03 13:14 ` Wei Liu
2018-10-04 12:10 ` Razvan Cojocaru
2018-10-04 16:56 ` Tamas K Lengyel
` (2 subsequent siblings)
3 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2018-10-03 13:14 UTC (permalink / raw)
To: Razvan Cojocaru
Cc: tamas, wei.liu2, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich, xen-devel,
ian.jackson
On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote:
> Currently there is a subop for setting the memaccess of a page, but not
> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
> functionality.
>
> Both altp2m get/set mem access functions use the struct
> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
> been renamed from xen_hvm_altp2m_set_mem_access.
>
> Signed-off-by: Adrian Pop <apop@bitdefender.com>
> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
> ---
> Changes since V5:
> - Fixed the build by conditionally-compiling the altp2m code
> gated on CONFIG_HVM being #defined.
> ---
> tools/libxc/include/xenctrl.h | 3 +++
> tools/libxc/xc_altp2m.c | 33 ++++++++++++++++++++++++++++++---
Acked-by: Wei Liu <wei.liu2@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-03 13:14 ` Wei Liu
@ 2018-10-04 12:10 ` Razvan Cojocaru
0 siblings, 0 replies; 11+ messages in thread
From: Razvan Cojocaru @ 2018-10-04 12:10 UTC (permalink / raw)
To: Wei Liu, george.dunlap, tamas
Cc: sstabellini, konrad.wilk, andrew.cooper3, Adrian Pop, tim,
julien.grall, jbeulich, xen-devel, ian.jackson
On 10/3/18 4:14 PM, Wei Liu wrote:
> On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote:
>> Currently there is a subop for setting the memaccess of a page, but not
>> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
>> functionality.
>>
>> Both altp2m get/set mem access functions use the struct
>> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
>> been renamed from xen_hvm_altp2m_set_mem_access.
>>
>> Signed-off-by: Adrian Pop <apop@bitdefender.com>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>>
>> ---
>> Changes since V5:
>> - Fixed the build by conditionally-compiling the altp2m code
>> gated on CONFIG_HVM being #defined.
>> ---
>> tools/libxc/include/xenctrl.h | 3 +++
>> tools/libxc/xc_altp2m.c | 33 ++++++++++++++++++++++++++++++---
>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
Thanks! Tamas, George, is the patch OK with you?
Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-09-27 7:58 [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page Razvan Cojocaru
2018-10-03 13:14 ` Wei Liu
@ 2018-10-04 16:56 ` Tamas K Lengyel
2018-10-08 7:59 ` Razvan Cojocaru
2018-10-16 16:26 ` George Dunlap
2018-10-31 16:39 ` Wei Liu
3 siblings, 1 reply; 11+ messages in thread
From: Tamas K Lengyel @ 2018-10-04 16:56 UTC (permalink / raw)
To: Razvan Cojocaru
Cc: Stefano Stabellini, Wei Liu, Konrad Rzeszutek Wilk,
George Dunlap, Andrew Cooper, Adrian Pop, Tim Deegan,
Julien Grall, Jan Beulich, Xen-devel, Ian Jackson
On Thu, Sep 27, 2018 at 1:59 AM Razvan Cojocaru
<rcojocaru@bitdefender.com> wrote:
>
> Currently there is a subop for setting the memaccess of a page, but not
> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
> functionality.
>
> Both altp2m get/set mem access functions use the struct
> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
> been renamed from xen_hvm_altp2m_set_mem_access.
>
> Signed-off-by: Adrian Pop <apop@bitdefender.com>
> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
LGTM
Acked-by: Tamas K Lengyel <tamas@tklengyel.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-04 16:56 ` Tamas K Lengyel
@ 2018-10-08 7:59 ` Razvan Cojocaru
0 siblings, 0 replies; 11+ messages in thread
From: Razvan Cojocaru @ 2018-10-08 7:59 UTC (permalink / raw)
To: Tamas K Lengyel, George Dunlap
Cc: Stefano Stabellini, Wei Liu, Konrad Rzeszutek Wilk,
Andrew Cooper, Adrian Pop, Tim Deegan, Julien Grall, Jan Beulich,
Xen-devel, Ian Jackson
On 10/4/18 7:56 PM, Tamas K Lengyel wrote:
> On Thu, Sep 27, 2018 at 1:59 AM Razvan Cojocaru
> <rcojocaru@bitdefender.com> wrote:
>>
>> Currently there is a subop for setting the memaccess of a page, but not
>> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
>> functionality.
>>
>> Both altp2m get/set mem access functions use the struct
>> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
>> been renamed from xen_hvm_altp2m_set_mem_access.
>>
>> Signed-off-by: Adrian Pop <apop@bitdefender.com>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
> LGTM
> Acked-by: Tamas K Lengyel <tamas@tklengyel.com>
Thanks! If I'm not mistaken, we now only need George's ack to commit
this patch (which concludes our #VE-related series - the only thing
still remaining after this is to fix the logdirty VGA issue as far as
we're concerned).
Thanks,
Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-09-27 7:58 [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page Razvan Cojocaru
2018-10-03 13:14 ` Wei Liu
2018-10-04 16:56 ` Tamas K Lengyel
@ 2018-10-16 16:26 ` George Dunlap
2018-10-16 16:31 ` George Dunlap
2018-10-17 7:43 ` Razvan Cojocaru
2018-10-31 16:39 ` Wei Liu
3 siblings, 2 replies; 11+ messages in thread
From: George Dunlap @ 2018-10-16 16:26 UTC (permalink / raw)
To: Razvan Cojocaru, xen-devel
Cc: tamas, wei.liu2, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich,
ian.jackson
On 09/27/2018 08:58 AM, Razvan Cojocaru wrote:
> Currently there is a subop for setting the memaccess of a page, but not
> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
> functionality.
>
> Both altp2m get/set mem access functions use the struct
> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
> been renamed from xen_hvm_altp2m_set_mem_access.
>
> Signed-off-by: Adrian Pop <apop@bitdefender.com>
> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
Sorry it took so long to get to this:
Reviewed-by: George Dunlap <george.dunlap@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-16 16:26 ` George Dunlap
@ 2018-10-16 16:31 ` George Dunlap
2018-10-17 7:43 ` Razvan Cojocaru
1 sibling, 0 replies; 11+ messages in thread
From: George Dunlap @ 2018-10-16 16:31 UTC (permalink / raw)
To: Razvan Cojocaru, xen-devel
Cc: tamas, wei.liu2, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich,
ian.jackson
On 10/16/2018 05:26 PM, George Dunlap wrote:
> On 09/27/2018 08:58 AM, Razvan Cojocaru wrote:
>> Currently there is a subop for setting the memaccess of a page, but not
>> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
>> functionality.
>>
>> Both altp2m get/set mem access functions use the struct
>> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
>> been renamed from xen_hvm_altp2m_set_mem_access.
>>
>> Signed-off-by: Adrian Pop <apop@bitdefender.com>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
> Sorry it took so long to get to this:
>
> Reviewed-by: George Dunlap <george.dunlap@citrix.com>
Although -- looking through the patch, I don't think this needed my ack.
All the code was either directly memaccess or toolstack related; Wei
and Tamas' acks should have been enough for it to go in.
-George
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-16 16:26 ` George Dunlap
2018-10-16 16:31 ` George Dunlap
@ 2018-10-17 7:43 ` Razvan Cojocaru
1 sibling, 0 replies; 11+ messages in thread
From: Razvan Cojocaru @ 2018-10-17 7:43 UTC (permalink / raw)
To: George Dunlap, xen-devel
Cc: tamas, wei.liu2, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich,
ian.jackson
On 10/16/18 7:26 PM, George Dunlap wrote:
> On 09/27/2018 08:58 AM, Razvan Cojocaru wrote:
>> Currently there is a subop for setting the memaccess of a page, but not
>> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
>> functionality.
>>
>> Both altp2m get/set mem access functions use the struct
>> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
>> been renamed from xen_hvm_altp2m_set_mem_access.
>>
>> Signed-off-by: Adrian Pop <apop@bitdefender.com>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
> Sorry it took so long to get to this:
>
> Reviewed-by: George Dunlap <george.dunlap@citrix.com>
No problem at all. Thanks!
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-09-27 7:58 [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page Razvan Cojocaru
` (2 preceding siblings ...)
2018-10-16 16:26 ` George Dunlap
@ 2018-10-31 16:39 ` Wei Liu
2018-10-31 16:45 ` Razvan Cojocaru
3 siblings, 1 reply; 11+ messages in thread
From: Wei Liu @ 2018-10-31 16:39 UTC (permalink / raw)
To: Razvan Cojocaru
Cc: tamas, wei.liu2, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich, xen-devel,
ian.jackson
On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote:
> Currently there is a subop for setting the memaccess of a page, but not
> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
> functionality.
>
> Both altp2m get/set mem access functions use the struct
> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
> been renamed from xen_hvm_altp2m_set_mem_access.
>
> Signed-off-by: Adrian Pop <apop@bitdefender.com>
> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
I'm not sure if this patch has been applied. If not I will do it during
my next sweep.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-31 16:39 ` Wei Liu
@ 2018-10-31 16:45 ` Razvan Cojocaru
2018-10-31 16:52 ` Wei Liu
0 siblings, 1 reply; 11+ messages in thread
From: Razvan Cojocaru @ 2018-10-31 16:45 UTC (permalink / raw)
To: Wei Liu
Cc: tamas, konrad.wilk, george.dunlap, andrew.cooper3, Adrian Pop,
tim, julien.grall, sstabellini, jbeulich, xen-devel, ian.jackson
On 10/31/18 6:39 PM, Wei Liu wrote:
> On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote:
>> Currently there is a subop for setting the memaccess of a page, but not
>> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
>> functionality.
>>
>> Both altp2m get/set mem access functions use the struct
>> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
>> been renamed from xen_hvm_altp2m_set_mem_access.
>>
>> Signed-off-by: Adrian Pop <apop@bitdefender.com>
>> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>
> I'm not sure if this patch has been applied. If not I will do it during
> my next sweep.
Hello Wei,
It's been applied:
https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=7559ab7830c3e1594cd73efd3f1acbb171036728
Thanks,
Razvan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page
2018-10-31 16:45 ` Razvan Cojocaru
@ 2018-10-31 16:52 ` Wei Liu
0 siblings, 0 replies; 11+ messages in thread
From: Wei Liu @ 2018-10-31 16:52 UTC (permalink / raw)
To: Razvan Cojocaru
Cc: tamas, Wei Liu, konrad.wilk, george.dunlap, andrew.cooper3,
Adrian Pop, tim, julien.grall, sstabellini, jbeulich, xen-devel,
ian.jackson
On Wed, Oct 31, 2018 at 06:45:49PM +0200, Razvan Cojocaru wrote:
> On 10/31/18 6:39 PM, Wei Liu wrote:
> > On Thu, Sep 27, 2018 at 10:58:54AM +0300, Razvan Cojocaru wrote:
> >> Currently there is a subop for setting the memaccess of a page, but not
> >> for consulting it. The new HVMOP_altp2m_get_mem_access adds this
> >> functionality.
> >>
> >> Both altp2m get/set mem access functions use the struct
> >> xen_hvm_altp2m_mem_access which has now dropped the `set' part and has
> >> been renamed from xen_hvm_altp2m_set_mem_access.
> >>
> >> Signed-off-by: Adrian Pop <apop@bitdefender.com>
> >> Signed-off-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
> >
> > I'm not sure if this patch has been applied. If not I will do it during
> > my next sweep.
>
> Hello Wei,
>
> It's been applied:
> https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=7559ab7830c3e1594cd73efd3f1acbb171036728
Thanks for letting me know.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2018-10-31 16:52 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-27 7:58 [PATCH V6] x86/altp2m: Add a subop for obtaining the mem access of a page Razvan Cojocaru
2018-10-03 13:14 ` Wei Liu
2018-10-04 12:10 ` Razvan Cojocaru
2018-10-04 16:56 ` Tamas K Lengyel
2018-10-08 7:59 ` Razvan Cojocaru
2018-10-16 16:26 ` George Dunlap
2018-10-16 16:31 ` George Dunlap
2018-10-17 7:43 ` Razvan Cojocaru
2018-10-31 16:39 ` Wei Liu
2018-10-31 16:45 ` Razvan Cojocaru
2018-10-31 16:52 ` Wei Liu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.