Re: [Qemu-devel] [PATCH 3/6] vhost-user-test: add back memfd check

2018-03-16 Thread Paolo Bonzini
On 16/03/2018 15:17, Marc-André Lureau wrote:
> Hi Paolo,
> 
> Could you pick the remaining fixes acked by Maxime?

Ok, queued them now.

Paolo

> thanks
> 
> On Fri, Feb 23, 2018 at 3:23 PM, Maxime Coquelin
>  wrote:
>>
>>
>> On 02/15/2018 10:25 PM, Marc-André Lureau wrote:
>>>
>>> This revert commit fb68096da3d35e64c88cd610c1fa42766c58e92a, and
>>> modify test_read_guest_mem() to use different chardev names, when
>>> using memfd (_test_server_free(), where the chardev is removed, runs
>>> in idle).
>>>
>>> Signed-off-by: Marc-André Lureau
>>> ---
>>>   tests/vhost-user-test.c | 93
>>> +++--
>>>   1 file changed, 66 insertions(+), 27 deletions(-)
>>
>>
>> Looks good to me:
>> Acked-by: Maxime Coquelin 
>>
> 




Re: [Qemu-devel] [PATCH 3/6] vhost-user-test: add back memfd check

2018-03-16 Thread Marc-André Lureau
Hi Paolo,

Could you pick the remaining fixes acked by Maxime?

thanks

On Fri, Feb 23, 2018 at 3:23 PM, Maxime Coquelin
 wrote:
>
>
> On 02/15/2018 10:25 PM, Marc-André Lureau wrote:
>>
>> This revert commit fb68096da3d35e64c88cd610c1fa42766c58e92a, and
>> modify test_read_guest_mem() to use different chardev names, when
>> using memfd (_test_server_free(), where the chardev is removed, runs
>> in idle).
>>
>> Signed-off-by: Marc-André Lureau
>> ---
>>   tests/vhost-user-test.c | 93
>> +++--
>>   1 file changed, 66 insertions(+), 27 deletions(-)
>
>
> Looks good to me:
> Acked-by: Maxime Coquelin 
>

-- 
Marc-André Lureau



Re: [Qemu-devel] [PATCH 3/6] vhost-user-test: add back memfd check

2018-02-23 Thread Maxime Coquelin



On 02/15/2018 10:25 PM, Marc-André Lureau wrote:

This revert commit fb68096da3d35e64c88cd610c1fa42766c58e92a, and
modify test_read_guest_mem() to use different chardev names, when
using memfd (_test_server_free(), where the chardev is removed, runs
in idle).

Signed-off-by: Marc-André Lureau
---
  tests/vhost-user-test.c | 93 +++--
  1 file changed, 66 insertions(+), 27 deletions(-)


Looks good to me:
Acked-by: Maxime Coquelin 

Thanks,
Maxime