Re: [PATCH] xfs: simplify two-level sysctl registration for xfs_table

From: Dave Chinner
Date: Sun Mar 12 2023 - 19:31:22 EST


On Fri, Mar 10, 2023 at 03:02:19PM -0800, Luis Chamberlain wrote:
> There is no need to declare two tables to just create directories,
> this can be easily be done with a prefix path with register_sysctl().
>
> Simplify this registration.
>
> Signed-off-by: Luis Chamberlain <mcgrof@xxxxxxxxxx>
> ---
>
> This is not clear to some so I've updated the docs for the sysctl
> registration here:
>
> https://lore.kernel.org/all/20230310223947.3917711-1-mcgrof@xxxxxxxxxx/T/#u
>
> fs/xfs/xfs_sysctl.c | 20 +-------------------
> 1 file changed, 1 insertion(+), 19 deletions(-)

Looks fine.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>

--
Dave Chinner
david@xxxxxxxxxxxxx