> -----Original Message-----
> From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of
> Ross Lagerwall
> Sent: 18 October 2017 15:04
> To: Xen-devel <xen-devel@lists.xen.org>
> Cc: Stefano Stabellini <sstabell...@kernel.org>; Wei Liu
> <wei.l...@citrix.com>; Konrad Rzeszutek Wilk <konrad.w...@oracle.com>;
> George Dunlap <george.dun...@citrix.com>; Andrew Cooper
> <andrew.coop...@citrix.com>; Ian Jackson <ian.jack...@citrix.com>; Tim
> (Xen.org) <t...@xen.org>; Ross Lagerwall <ross.lagerw...@citrix.com>; Jan
> Beulich <jbeul...@suse.com>
> Subject: [Xen-devel] [PATCH v1 3/5] xen: Provide
> XEN_DMOP_pin_memory_cacheattr
>
> Provide XEN_DMOP_pin_memory_cacheattr to allow a deprivileged QEMU
> to
> pin the caching type of RAM after moving the VRAM. It is equivalent to
> XEN_DOMCTL_pin_memory_cacheattr.
>
> Signed-off-by: Ross Lagerwall <ross.lagerw...@citrix.com>
> ---
> xen/arch/x86/hvm/dm.c | 12 ++++++++++++
> xen/include/public/hvm/dm_op.h | 14 ++++++++++++++
> xen/include/xlat.lst | 1 +
> 3 files changed, 27 insertions(+)
>
> diff --git a/xen/arch/x86/hvm/dm.c b/xen/arch/x86/hvm/dm.c
> index 432a863..eebcbcc 100644
> --- a/xen/arch/x86/hvm/dm.c
> +++ b/xen/arch/x86/hvm/dm.c
> @@ -21,6 +21,7 @@
>
> #include <asm/hap.h>
> #include <asm/hvm/ioreq.h>
> +#include <asm/hvm/cacheattr.h>
> #include <asm/shadow.h>
>
> #include <xsm/xsm.h>
> @@ -656,6 +657,16 @@ static int dm_op(const struct dmop_args *op_args)
> break;
> }
>
> + case XEN_DMOP_pin_memory_cacheattr:
> + {
> + const struct xen_dm_op_pin_memory_cacheattr *data =
> + &op.u.pin_memory_cacheattr;
> +
You need to check data->pad is 0 here.
Paul
> + rc = hvm_set_mem_pinned_cacheattr(d, data->start, data->end,
> + data->type);
> + break;
> + }
> +
> default:
> rc = -EOPNOTSUPP;
> break;
> @@ -686,6 +697,7 @@ CHECK_dm_op_inject_event;
> CHECK_dm_op_inject_msi;
> CHECK_dm_op_remote_shutdown;
> CHECK_dm_op_add_to_physmap;
> +CHECK_dm_op_pin_memory_cacheattr;
>
> int compat_dm_op(domid_t domid,
> unsigned int nr_bufs,
> diff --git a/xen/include/public/hvm/dm_op.h
> b/xen/include/public/hvm/dm_op.h
> index 88aace7..11bb386 100644
> --- a/xen/include/public/hvm/dm_op.h
> +++ b/xen/include/public/hvm/dm_op.h
> @@ -381,6 +381,19 @@ struct xen_dm_op_add_to_physmap {
> uint64_aligned_t gpfn; /* GPFN in domid where the GMFN should appear.
> */
> };
>
> +/*
> + * XEN_DMOP_pin_memory_cacheattr : Pin caching type of RAM space.
> + * Identical to XEN_DOMCTL_pin_mem_cacheattr.
> + */
> +#define XEN_DMOP_pin_memory_cacheattr 18
> +
> +struct xen_dm_op_pin_memory_cacheattr {
> + uint64_aligned_t start; /* Start gfn. */
> + uint64_aligned_t end; /* End gfn. */
> + uint32_t type; /* XEN_DOMCTL_MEM_CACHEATTR_* */
> + uint32_t pad;
> +};
> +
> struct xen_dm_op {
> uint32_t op;
> uint32_t pad;
> @@ -403,6 +416,7 @@ struct xen_dm_op {
> map_mem_type_to_ioreq_server;
> struct xen_dm_op_remote_shutdown remote_shutdown;
> struct xen_dm_op_add_to_physmap add_to_physmap;
> + struct xen_dm_op_pin_memory_cacheattr pin_memory_cacheattr;
> } u;
> };
>
> diff --git a/xen/include/xlat.lst b/xen/include/xlat.lst
> index d40bac6..fffb308 100644
> --- a/xen/include/xlat.lst
> +++ b/xen/include/xlat.lst
> @@ -65,6 +65,7 @@
> ? dm_op_inject_msi hvm/dm_op.h
> ? dm_op_ioreq_server_range hvm/dm_op.h
> ? dm_op_modified_memory hvm/dm_op.h
> +? dm_op_pin_memory_cacheattr hvm/dm_op.h
> ? dm_op_remote_shutdown hvm/dm_op.h
> ? dm_op_set_ioreq_server_state hvm/dm_op.h
> ? dm_op_set_isa_irq_level hvm/dm_op.h
> --
> 2.9.5
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> https://lists.xen.org/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
