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

Re: [Xen-devel] [PATCH v2] arm/vm_event: get/set registers



On Fri, Sep 2, 2016 at 1:10 PM, Julien Grall <julien.grall@xxxxxxx> wrote:
>
>
> On 02/09/2016 19:47, Tamas K Lengyel wrote:
>>
>> On Fri, Sep 2, 2016 at 12:40 PM, Julien Grall <julien.grall@xxxxxxx>
>> wrote:
>>>
>>> On 02/09/2016 18:45, Andrew Cooper wrote:
>>>>
>>>>
>>>> On 02/09/16 18:37, Tamas K Lengyel wrote:
>>>>>
>>>>>
>>>>> On Tue, Aug 2, 2016 at 2:10 AM, Razvan Cojocaru
>>>>> <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>>>>>>
>>>>>>
>>>>>> On 08/01/2016 08:59 PM, Tamas K Lengyel wrote:
>>>>>>>
>>>>>>>
>>>>>>> Add support for getting/setting registers through vm_event on ARM.
>>>>>>> Only
>>>>>>> TTB/CR/R0/R1, PC and CPSR are sent as part of a request and only PC
>>>>>>> is
>>>>>>> set
>>>>>>> as part of a response. The set of registers can be expanded in the
>>>>>>> future to
>>>>>>> include other registers as well if necessary.
>>>>>>>
>>>>>>> Signed-off-by: Tamas K Lengyel <tamas.lengyel@xxxxxxxxxxxx>
>>>>>>> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
>>>>>>
>>>>>>
>>>>>> Acked-by: Razvan Cojocaru <rcojocaru@xxxxxxxxxxxxxxx>
>>>>>
>>>>>
>>>>> Patch ping.
>>>>
>>>>
>>>>
>>>> Requires an ARM ack.
>>>
>>>
>>>
>>> I don't think it requires an ack from Stefano and I, it touches only the
>>> vm_event subsystem.
>>>
>>> If you still want an ARM ack, then I will defer to Stefano.
>>
>>
>> Indeed it only touches the vm_event system so just wanted to double
>> check it's OK from your side as we had lengthy discussion about it. If
>> there are no objections a formal ack should not be necessary and it's
>> good to go.
>
>
> My objections are still there. Hence why I said I will defer to Stefano.

Fair enough.

Thanks,
Tamas

_______________________________________________
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®.