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

Re: [Xen-devel] [PATCH v10 3/6] x86/ioreq server: Add device model wrappers for new DMOP





On 4/3/2017 5:28 PM, Wei Liu wrote:
On Mon, Apr 03, 2017 at 09:13:20AM +0100, Paul Durrant wrote:
-----Original Message-----
From: Yu Zhang [mailto:yu.c.zhang@xxxxxxxxxxxxxxx]
Sent: 02 April 2017 13:24
To: xen-devel@xxxxxxxxxxxxx
Cc: zhiyuan.lv@xxxxxxxxx; Paul Durrant <Paul.Durrant@xxxxxxxxxx>; Ian
Jackson <Ian.Jackson@xxxxxxxxxx>; Wei Liu <wei.liu2@xxxxxxxxxx>
Subject: [PATCH v10 3/6] x86/ioreq server: Add device model wrappers for
new DMOP

A new device model wrapper is added for the newly introduced
DMOP - XEN_DMOP_map_mem_type_to_ioreq_server.

Since currently this DMOP only supports the emulation of write
operations, attempts to trigger the DMOP with values other than
XEN_DMOP_IOREQ_MEM_ACCESS_WRITE or 0(to unmap the ioreq server)
shall fail. The wrapper shall be updated once read operations
are also to be emulated in the future.

Also note currently this DMOP only supports one memory type,
and can be extended in the future to map multiple memory types
to multiple ioreq servers, e.g. mapping HVMMEM_ioreq_serverX to
ioreq server X, This wrapper shall be updated when such change
is made.

Signed-off-by: Yu Zhang <yu.c.zhang@xxxxxxxxxxxxxxx>
Reviewed-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

...with one observation...

---
Cc: Paul Durrant <paul.durrant@xxxxxxxxxx>
Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
---
  tools/libs/devicemodel/core.c                   | 25
+++++++++++++++++++++++++
  tools/libs/devicemodel/include/xendevicemodel.h | 18
++++++++++++++++++
  tools/libs/devicemodel/libxendevicemodel.map    |  1 +
  tools/libxc/include/xenctrl_compat.h            |  5 +++++
  tools/libxc/xc_devicemodel_compat.c             | 17 +++++++++++++++++
This is new code so I don't think you really want compat code for this, do you?
Correct. No compat code is needed.

Thank you, Paul & Wei.
Oh. I see, the compat code is only for existing ones. We do not need this.
What about the libxendevicemodel.map? I updated this file to build pass for the libxc, with compat code not changed, maybe we shall not add the new op to libxendevicemodel.map either.

I can send a V11 of this series - but must be quick to catch up the code freeze date. :) Or with other patches received "Reviewed by", we can just drop the useless code of this patch.
Any suggestions?


B.R.
Yu

Wei.



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

 


Rackspace

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