On Tue, Sep 24, 2024 at 2:44 AM Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
>
> On Mon, 23 Sept 2024 at 19:13, Joanne Koong <joannelkoong@xxxxxxxxx> wrote:
> >
> > Introduce the capability to dynamically configure the max pages limit
> > (FUSE_MAX_MAX_PAGES) through a sysctl. This allows system administrators
> > to dynamically set the maximum number of pages that can be used for
> > servicing requests in fuse.
>
> Applied (with a minor update, see below), thanks.
>
> > @@ -2077,6 +2085,7 @@ static void fuse_fs_cleanup(void)
> > {
> > unregister_filesystem(&fuse_fs_type);
> > unregister_fuseblk();
> > + fuse_sysctl_unregister();
>
> I moved this to the top of the function to make the order of the
> cleanups reverse that of the setups. I haven't tested this, but I
> guess it shouldn't make a difference, so this is just an aesthetic
> fix.
This update looks great to me. Thanks, Miklos!
>
> Thanks,
> Miklos
