Re: [PATCH v2 15/15] io_uring: port to struct kmem_cache_args

[Mike Rapoport <rppt@kernel.org>]

On Tue, Sep 03, 2024 at 04:20:56PM +0200, Christian Brauner wrote:
> Port req_cachep to struct kmem_cache_args.
> 
> Signed-off-by: Christian Brauner <brauner@kernel.org>

Reviewed-by: Mike Rapoport (Microsoft) <rppt@kernel.org>

> ---
>  io_uring/io_uring.c | 14 ++++++++------
>  1 file changed, 8 insertions(+), 6 deletions(-)
> 
> diff --git a/io_uring/io_uring.c b/io_uring/io_uring.c
> index 3942db160f18..d9d721d1424e 100644
> --- a/io_uring/io_uring.c
> +++ b/io_uring/io_uring.c
> @@ -3638,6 +3638,11 @@ SYSCALL_DEFINE2(io_uring_setup, u32, entries,
>  
>  static int __init io_uring_init(void)
>  {
> +	struct kmem_cache_args kmem_args = {
> +		.useroffset = offsetof(struct io_kiocb, cmd.data),
> +		.usersize = sizeof_field(struct io_kiocb, cmd.data),
> +	};
> +
>  #define __BUILD_BUG_VERIFY_OFFSET_SIZE(stype, eoffset, esize, ename) do { \
>  	BUILD_BUG_ON(offsetof(stype, ename) != eoffset); \
>  	BUILD_BUG_ON(sizeof_field(stype, ename) != esize); \
> @@ -3722,12 +3727,9 @@ static int __init io_uring_init(void)
>  	 * range, and HARDENED_USERCOPY will complain if we haven't
>  	 * correctly annotated this range.
>  	 */
> -	req_cachep = kmem_cache_create_usercopy("io_kiocb",
> -				sizeof(struct io_kiocb), 0,
> -				SLAB_HWCACHE_ALIGN | SLAB_PANIC |
> -				SLAB_ACCOUNT | SLAB_TYPESAFE_BY_RCU,
> -				offsetof(struct io_kiocb, cmd.data),
> -				sizeof_field(struct io_kiocb, cmd.data), NULL);
> +	req_cachep = kmem_cache_create("io_kiocb", sizeof(struct io_kiocb), &kmem_args,
> +				SLAB_HWCACHE_ALIGN | SLAB_PANIC | SLAB_ACCOUNT |
> +				SLAB_TYPESAFE_BY_RCU);
>  	io_buf_cachep = KMEM_CACHE(io_buffer,
>  					  SLAB_HWCACHE_ALIGN | SLAB_PANIC | SLAB_ACCOUNT);
>  
> 
> -- 
> 2.45.2
> 

-- 
Sincerely yours,
Mike.