From mboxrd@z Thu Jan 1 00:00:00 1970 References: <20190605030233.19567-1-tao.peng@linux.alibaba.com> <20190605203016.GC22236@redhat.com> From: Peng Tao Message-ID: <3df1985d-7ba8-c435-d6c3-24c16d61124f@linux.alibaba.com> Date: Thu, 6 Jun 2019 14:51:53 +0800 MIME-Version: 1.0 In-Reply-To: <20190605203016.GC22236@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Virtio-fs] [PATCH-v3 0/1] virtiofs: FUSE_REMOVEMAPPING remove multiple entries in one call List-Id: Development discussions about virtio-fs List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vivek Goyal Cc: virtio-fs@redhat.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.