linux-kernel - Re: Patch "io_uring: fail links of cancelled timeouts" has been added to the 5.13-stable tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <YT8so5i420vp4LRZ@kroah.com>
Date:   Mon, 13 Sep 2021 12:49:07 +0200
From:   Greg KH <gregkh@...uxfoundation.org>
To:     linux-kernel@...r.kernel.org
Cc:     asml.silence@...il.com, axboe@...nel.dk,
        stable-commits@...r.kernel.org
Subject: Re: Patch "io_uring: fail links of cancelled timeouts" has been
 added to the 5.13-stable tree

On Mon, Sep 13, 2021 at 11:45:14AM +0200, gregkh@...uxfoundation.org wrote:
> 
> This is a note to let you know that I've just added the patch titled
> 
>     io_uring: fail links of cancelled timeouts
> 
> to the 5.13-stable tree which can be found at:
>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>      io_uring-fail-links-of-cancelled-timeouts.patch
> and it can be found in the queue-5.13 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@...r.kernel.org> know about it.
> 
> 
> >From 2ae2eb9dde18979b40629dd413b9adbd6c894cdf Mon Sep 17 00:00:00 2001
> From: Pavel Begunkov <asml.silence@...il.com>
> Date: Thu, 9 Sep 2021 13:56:27 +0100
> Subject: io_uring: fail links of cancelled timeouts
> 
> From: Pavel Begunkov <asml.silence@...il.com>
> 
> commit 2ae2eb9dde18979b40629dd413b9adbd6c894cdf upstream.
> 
> When we cancel a timeout we should mark it with REQ_F_FAIL, so
> linked requests are cancelled as well, but not queued for further
> execution.
> 
> Cc: stable@...r.kernel.org
> Signed-off-by: Pavel Begunkov <asml.silence@...il.com>
> Link: https://lore.kernel.org/r/fff625b44eeced3a5cae79f60e6acf3fbdf8f990.1631192135.git.asml.silence@gmail.com
> Signed-off-by: Jens Axboe <axboe@...nel.dk>
> Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> ---
>  fs/io_uring.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> --- a/fs/io_uring.c
> +++ b/fs/io_uring.c
> @@ -1307,6 +1307,8 @@ static void io_kill_timeout(struct io_ki
>  	struct io_timeout_data *io = req->async_data;
>  
>  	if (hrtimer_try_to_cancel(&io->timer) != -1) {
> +		if (status)
> +			req_set_fail(req);
>  		atomic_set(&req->ctx->cq_timeouts,
>  			atomic_read(&req->ctx->cq_timeouts) + 1);
>  		list_del_init(&req->timeout.list);
> 
> 
> Patches currently in stable-queue which might be from asml.silence@...il.com are
> 
> queue-5.13/io_uring-limit-fixed-table-size-by-rlimit_nofile.patch
> queue-5.13/io_uring-reexpand-under-reexpanded-iters.patch
> queue-5.13/io_uring-fail-links-of-cancelled-timeouts.patch
> queue-5.13/bio-fix-page-leak-bio_add_hw_page-failure.patch
> queue-5.13/io_uring-refactor-io_submit_flush_completions.patch

Breaks the build on 5.13.y so dropping it from there.

thanks,

greg k-h