Re: [for-4.18][PATCH 2/2] xen/public: arch-arm: Update list of supported hypercalls

[Michal Orzel <michal.orzel@xxxxxxx>]

Hi Julien,

On 06/10/2023 12:11, Julien Grall wrote:
> 
> 
> Hi Michal,
> 
> On 06/10/2023 08:51, Michal Orzel wrote:
>> The list is out of date and does not specify all the hypercalls/sub-ops
>> we support, so update it.
>>
>> Signed-off-by: Michal Orzel <michal.orzel@xxxxxxx>
>> ---
>>   xen/include/public/arch-arm.h | 21 +++++++++++++++++++++
>>   1 file changed, 21 insertions(+)
>>
>> diff --git a/xen/include/public/arch-arm.h b/xen/include/public/arch-arm.h
>> index 6a4467e8f5d1..263b1e182eb3 100644
>> --- a/xen/include/public/arch-arm.h
>> +++ b/xen/include/public/arch-arm.h
>> @@ -104,6 +104,7 @@
>>    *   Exactly these sub-operations are supported:
>>    *    * HVMOP_set_param
>>    *    * HVMOP_get_param
>> + *    * HVMOP_guest_request_vm_event
>>    *
>>    *  HYPERVISOR_grant_table_op
>>    *   All generic sub-operations
>> @@ -116,6 +117,26 @@
>>    *  HYPERVISOR_argo_op
>>    *   All generic sub-operations
>>    *
>> + *  HYPERVISOR_hypfs_op
>> + *   All generic sub-operations
>> + *
>> + *  HYPERVISOR_platform_op
>> + *   Exactly these sub-operations are supported:
>> + *    * XENPF_settime64
>> + *
>> + *  HYPERVISOR_vm_assist
>> + *   All generic sub-operations
>> + *
>> + *  HYPERVISOR_dm_op
>> + *   All ioreq server sub-operations are supported as well as:
> 
> I interpret this sentence as all the hypercall with 'ioreq' in their
> name is supported. But, for Arm, we at least don't support
> XEN_DMOP_map_mem_type_to_ioreq_server.
What I meant is that we support all sub-ops from ioreq_server_dm_op (ioreq.c).
XEN_DMOP_map_mem_type_to_ioreq_server is only accessible on x86 and not part of 
ioreq.c

~Michal