Re: [Xen-devel] [PATCH v8 for-4.12 00/17] Argo: hypervisor-mediated interdomain communication

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 06.02.19 at 14:53, <jgross@xxxxxxxx> wrote:
> On 06/02/2019 14:45, Jan Beulich wrote:
>>>>> On 06.02.19 at 09:54, <christopher.w.clark@xxxxxxxxx> wrote:
>>> Version eight of this series:
>>>
>>> Note: This version may not address the currently open discussion on the
>>> ARM hypercall argument convention and type selection for hypercall
>>> parameters.
>>>
>>> * Range check applied to numeric args in native hypercall entry
>>>   (ref: the above open discussion)
>>>
>>> * Revises the compat ABI and implementation
>>>     - avoids duplication of hypercall op implementations via
>>>       forwarding to native for ops other than sendv
>>>     - register op uses an always-64-bit fixed width pfn type
>>>       for consistent ABI as well as compat reuse of the native op
>>>     - tested communication between VMs on x86-64 host with:
>>>       32-bit PV, 32-bit HVM and 64-bit PV guests
>>>
>>> * Applies list_first_entry_or_null macro in multiple loops to
>>>   replace previous use of a list foreach to address review feedback
>>>
>>> * Removed stale comments from the public header
>>>
>>> New to this series:
>>>
>>> * Adds an initial version of a design document for Argo
>>>     - based on work previously sent to the mailing list, covers
>>>       the implementation's granular locking
>>>
>>> * Adds a SUPPORT.md section for the feature and Experimental statement
>>>
>>> Christopher Clark (17):
>>>   argo: Introduce the Kconfig option to govern inclusion of Argo
>>>   argo: introduce the argo_op hypercall boilerplate
>>>   argo: define argo_dprintk for subsystem debugging
>>>   argo: init, destroy and soft-reset, with enable command line opt
>>>   errno: add POSIX error codes EMSGSIZE, ECONNREFUSED to the ABI
>>>   xen/arm: introduce guest_handle_for_field()
>>>   argo: implement the register op
>>>   argo: implement the unregister op
>>>   argo: implement the sendv op; evtchn: expose send_guest_global_virq
>>>   argo: implement the notify op
>>>   xsm, argo: XSM control for argo register
>>>   xsm, argo: XSM control for argo message send operation
>>>   xsm, argo: XSM control for any access to argo by a domain
>>>   xsm, argo: notify: don't describe rings that cannot be sent to
>>>   MAINTAINERS: add new section for Argo and self as maintainer
>>>   SUPPORT.md : add new entry for the Argo feature
>>>   docs, argo: add design document for Argo
>> 
>> Where necessary and not already present
>> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
>> 
>> Jürgen, for this to be committed, your Rab would be needed, assuming
>> you're still comfortable with this going in this late.
> 
> What about the ARM hypercall parameters? Is this settled?

My interpretation of Stefano's latest response was "yes, it is".

> If yes or if this question is solved this week:
> 
> Release-acked-by: Juergen Gross <jgross@xxxxxxxx>

Thanks, I'll ask for last minute objections on irc and commit
later this afternoon if I hear none.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel