Re: [PATCH 0/5] fuse: over-io-uring fixes

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

 



On Fri, Jan 24, 2025 at 1:00 AM Luis Henriques <luis@xxxxxxxxxx> wrote:
>
> Hi Bernd,
>
> On Thu, Jan 23 2025, Bernd Schubert wrote:
>
> > This is a list of fixes that came up from review of Luis
> > and smatch run from Dan.
> > I didn't put in commit id in the "Fixes:" line, as it is
> > fuse-io-uring is in linux next only and might get rebases
> > with new IDs.
>
> Thank you for this, Bernd.  And sorry for the extra work -- I should have
> sent these patches myself instead of simply sending review comments. :-(

Sorry for the delay, I haven't gotten to looking at the latter
io-uring patches (10 to 17) yet in v10.
If there are any minor fixes, Bernd do you prefer that we send review
comments or that we send patches on top of the for-next tree? Thanks
for your hard work on this io-uring series.


Thanks,
Joanne

>
> Anyway, they all look good, and probably they should simply be squashed
> into the respective patches they are fixing.  If they are kept separately,
> feel free to add my
>
> Reviewed-by: Luis Henriques <luis@xxxxxxxxxx>
>
> Cheers,
> --
> Luís
>
> > Signed-off-by: Bernd Schubert <bschubert@xxxxxxx>
> > ---
> > Bernd Schubert (5):
> >       fuse: Fix copy_from_user error return code in fuse_uring_commit
> >       fuse: Remove an err= assignment and move a comment
> >       fuse: prevent disabling io-uring on active connections
> >       fuse: Remove unneeded include in fuse_dev_i.h
> >       fuse: Fix the struct fuse_args->in_args array size
> >
> >  fs/fuse/dev_uring.c  | 23 ++++++++++++-----------
> >  fs/fuse/fuse_dev_i.h |  1 -
> >  fs/fuse/fuse_i.h     |  2 +-
> >  3 files changed, 13 insertions(+), 13 deletions(-)
> > ---
> > base-commit: a5ca7ba2e604b5d4eb54e1e2746851fdd5d9e98f
> > change-id: 20250123-fuse-uring-for-6-14-incremental-to-v10-b6b916b77720
> >
> > Best regards,
> > --
> > Bernd Schubert <bschubert@xxxxxxx>
> >





[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux