Hi Paul,
On 12/20/18 2:28 PM, Paul Durrant wrote:
-----Original Message-----
From: Petre Ovidiu PIRCALABU [mailto:[email protected]]
Sent: 20 December 2018 14:26
To: Paul Durrant <[email protected]>; [email protected]
Cc: Stefano Stabellini <[email protected]>; Wei Liu
<[email protected]>; Razvan Cojocaru <[email protected]>; Konrad
Rzeszutek Wilk <[email protected]>; George Dunlap
<[email protected]>; Andrew Cooper <[email protected]>; Ian
Jackson <[email protected]>; Tim (Xen.org) <[email protected]>; Julien
Grall <[email protected]>; Tamas K Lengyel <[email protected]>; Jan
Beulich <[email protected]>; Roger Pau Monne <[email protected]>
Subject: Re: [Xen-devel] [RFC PATCH 4/6] vm_event: Use slotted channels
for sync requests.
On Thu, 2018-12-20 at 12:05 +0000, Paul Durrant wrote:
The memory for the asynchronous ring and the synchronous channels
will
be allocated from domheap and mapped to the controlling domain
using the
foreignmemory_map_resource interface. Unlike the current
implementation,
the allocated pages are not part of the target DomU, so they will
not be
reclaimed when the vm_event domain is disabled.
Why re-invent the wheel here? The ioreq infrastructure already does
pretty much everything you need AFAICT.
Paul
I wanted preseve as much as possible from the existing vm_event DOMCTL
interface and add only the necessary code to allocate and map the
vm_event_pages.
That means we have two subsystems duplicating a lot of functionality though. It
would be much better to use ioreq server if possible than provide a
compatibility interface via DOMCTL.
Also, to my knowledge, the ioreq server is only supported for x86 hvm
targets. I didn't want to add an extra limitation to the vm_event
system.
I believe Julien is already porting it to ARM.
FWIW, yes I have a port of ioreq for Arm. Still cleaning-up the code.
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel
