[Top][All Lists]

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

Re: [Qemu-block] [PATCH V2 1/2] block/nfs: tear down aio before nfs_clos

From: Peter Lieven
Subject: Re: [Qemu-block] [PATCH V2 1/2] block/nfs: tear down aio before nfs_close
Date: Fri, 13 Sep 2019 12:15:20 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0

Am 13.09.19 um 11:51 schrieb Max Reitz:
> On 10.09.19 17:41, Peter Lieven wrote:
>> nfs_close is a sync call from libnfs and has its own event
>> handler polling on the nfs FD. Avoid that both QEMU and libnfs
>> are intefering here.
>> CC: address@hidden
>> Signed-off-by: Peter Lieven <address@hidden>
>> ---
>>  block/nfs.c | 6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
> I’ve just seen that Kevin has already included the second patch (in its
> v1) in his pull request.
> So all that I can do is take this patch, which sounds good to me,
> especially since Ronnie has agreed that we should remove our FD handler
> there.
> (So I’ve rebased the patch on top of Kevin’s pull request, and I’ve
> taken it to my block branch.)

Thank you. After I discovered that we had this bug also before I added the 
nfs_umount call I thought

it would be good to have this patch first and the add the umount call because 
the fix should also go into

stable because in theory we could also run into trouble with just the *sync* 
nfs_clsoe call.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]