Re: [Virtio-fs] [PATCH-v3 0/1] virtiofs: FUSE_REMOVEMAPPING remove multiple entries in one call

[Peng Tao <tao.peng@linux.alibaba.com>]

On 2019/6/6 04:30, Vivek Goyal wrote:
> Hi Peng Tao,
> 
> I have squashed this patch and forget request wait patches into existing
> pathces and pushed latest patches here.
> 
> https://github.com/rhvgoyal/linux/commits/virtio-fs-dev-5.1
> 
> Please give it a try.
Both look good and work as expected. Thanks!

Cheers,
Tao

> 
> I have made more function name changes into existing patches. There were
> so many functions with name fuse_dax_free* that I was finding it hard
> to remember which function is doing what.
> 
> Thanks
> Vivek
> 
> On Wed, Jun 05, 2019 at 11:02:32AM +0800, Peng Tao wrote:
>> Hi Vivek,
>>
>> This works with corresponding qemu virtiofsd patch:
>> "[PATCH-v3] virtiofsd: make FUSE_REMOVEMAPPING support multiple entries".
>>
>> v1-v2: make fuse_removemapping_in count fuse_removemapping_one
>> v2->v3:
>> 1. fold in Vivek's cleanup and rename all forget_one to remove_one
>> 2. assert dmap->list is not used by anyone when adding to temporary list
>> 3. list_del() dmap->list when removing it from to_remove list
>>
>> Please add your SOB line as I don't see it with your cleanup and thus
>> did not add it to the patch when folding your change.
>>
>> Thanks,
>> Tao
>>
>> Peng Tao (1):
>>    virtiofs: FUSE_REMOVEMAPPING remove multiple entries in one call
>>
>>   fs/fuse/file.c            | 205 +++++++++++++++++++++++++++-----------
>>   fs/fuse/fuse_i.h          |   6 +-
>>   fs/fuse/inode.c           |   2 +-
>>   include/uapi/linux/fuse.h |   7 ++
>>   4 files changed, 160 insertions(+), 60 deletions(-)
>>
>> -- 
>> 2.17.1
>>

-- 
Into something rich and strange.