]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
drop fs-aio-make-io_cancel-generate-completions-again.patch from everywhere
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 Mar 2024 21:06:47 +0000 (21:06 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 Mar 2024 21:06:47 +0000 (21:06 +0000)
14 files changed:
queue-4.19/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-4.19/series
queue-5.10/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-5.10/series
queue-5.15/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-5.15/series
queue-5.4/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-5.4/series
queue-6.1/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-6.1/series
queue-6.6/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-6.6/series
queue-6.7/fs-aio-make-io_cancel-generate-completions-again.patch [deleted file]
queue-6.7/series

diff --git a/queue-4.19/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-4.19/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index 4841c17..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2131,14 +2131,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2169,14 +2166,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR, 0);
-       percpu_ref_put(&ctx->users);
index 0796eaa8294d560d0ce2bf371dfa4aa7be0c1077..13484f3a8946d2491e5df881733b1b65715bb72d 100644 (file)
@@ -12,6 +12,5 @@ gtp-fix-use-after-free-and-null-ptr-deref-in-gtp_newlink.patch
 wifi-nl80211-reject-iftype-change-with-mesh-id-change.patch
 btrfs-dev-replace-properly-validate-device-names.patch
 mmc-core-fix-emmc-initialization-with-1-bit-bus-connection.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 cachefiles-fix-memory-leak-in-cachefiles_add_cache.patch
 gpio-74x164-enable-output-pins-after-registers-are-r.patch
diff --git a/queue-5.10/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-5.10/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index 10f7324..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2121,14 +2121,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2159,14 +2156,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR, 0);
-       percpu_ref_put(&ctx->users);
index 12ad64369788759b4b36871f0228805da008527d..bd261a5fc8b25ac7207f0fd4043acdf4c3b16016 100644 (file)
@@ -30,7 +30,6 @@ mmc-sdhci-xenon-add-timeout-for-phy-init-complete.patch
 mmc-sdhci-xenon-fix-phy-init-clock-stability.patch
 riscv-add-caller_addrx-support.patch
 pmdomain-qcom-rpmhpd-fix-enabled_corner-aggregation.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 x86-cpu-intel-detect-tme-keyid-bits-before-setting-mtrr-mask-registers.patch
 mptcp-fix-possible-deadlock-in-subflow-diag.patch
 ext4-avoid-bb_free-and-bb_fragments-inconsistency-in-mb_free_blocks.patch
diff --git a/queue-5.15/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-5.15/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index c42b389..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2120,14 +2120,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2158,14 +2155,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR, 0);
-       percpu_ref_put(&ctx->users);
index a69bf905da965f572f4939a7d031747f3f282214..7a50e4632907c77cf997bfb06bbdd9e6bbe526dd 100644 (file)
@@ -57,7 +57,6 @@ mmc-sdhci-xenon-add-timeout-for-phy-init-complete.patch
 mmc-sdhci-xenon-fix-phy-init-clock-stability.patch
 riscv-add-caller_addrx-support.patch
 pmdomain-qcom-rpmhpd-fix-enabled_corner-aggregation.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 x86-cpu-intel-detect-tme-keyid-bits-before-setting-mtrr-mask-registers.patch
 mptcp-move-__mptcp_error_report-in-protocol.c.patch
 mptcp-process-pending-subflow-error-on-close.patch
diff --git a/queue-5.4/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-5.4/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index f6ac934..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2126,14 +2126,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2164,14 +2161,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR, 0);
-       percpu_ref_put(&ctx->users);
index eef1e58edcd699fd30f969fee4161675d95829a7..f9bef2290e9a7716210f59b62c4f68f2a22a0c8a 100644 (file)
@@ -19,7 +19,6 @@ btrfs-dev-replace-properly-validate-device-names.patch
 dmaengine-fsl-qdma-fix-soc-may-hang-on-16-byte-unaligned-read.patch
 dmaengine-fsl-qdma-init-irq-after-reg-initialization.patch
 mmc-core-fix-emmc-initialization-with-1-bit-bus-connection.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 x86-cpu-intel-detect-tme-keyid-bits-before-setting-mtrr-mask-registers.patch
 cachefiles-fix-memory-leak-in-cachefiles_add_cache.patch
 fs-hugetlb-fix-null-pointer-dereference-in-hugetlbs_fill_super.patch
diff --git a/queue-6.1/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-6.1/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index 18eb258..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2141,14 +2141,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2179,14 +2176,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR);
-       percpu_ref_put(&ctx->users);
index 326e39202647d6c0b724b54e286ca448026e6b96..851191a6ed1feeb49300ea1588b33bb3ce1b065a 100644 (file)
@@ -92,7 +92,6 @@ mmc-sdhci-xenon-fix-phy-init-clock-stability.patch
 riscv-add-caller_addrx-support.patch
 efivarfs-request-at-most-512-bytes-for-variable-names.patch
 pmdomain-qcom-rpmhpd-fix-enabled_corner-aggregation.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 x86-e820-don-t-reserve-setup_rng_seed-in-e820.patch
 x86-cpu-intel-detect-tme-keyid-bits-before-setting-mtrr-mask-registers.patch
 mptcp-fix-data-races-on-local_id.patch
diff --git a/queue-6.6/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-6.6/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index 1969e81..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2119,14 +2119,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2157,14 +2154,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR);
-       percpu_ref_put(&ctx->users);
index 0aba89632139fb8917b48983cc2d024c177893f1..2e0857edc993d69840404bd7c8e91d5c0ac8e9c5 100644 (file)
@@ -100,7 +100,6 @@ kbuild-add-wa-fatal-warnings-to-as-instr-invocation.patch
 iommufd-fix-iopt_access_list_id-overwrite-bug.patch
 efivarfs-request-at-most-512-bytes-for-variable-names.patch
 pmdomain-qcom-rpmhpd-fix-enabled_corner-aggregation.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 mm-debug_vm_pgtable-fix-bug_on-with-pud-advanced-test.patch
 x86-e820-don-t-reserve-setup_rng_seed-in-e820.patch
 x86-cpu-intel-detect-tme-keyid-bits-before-setting-mtrr-mask-registers.patch
diff --git a/queue-6.7/fs-aio-make-io_cancel-generate-completions-again.patch b/queue-6.7/fs-aio-make-io_cancel-generate-completions-again.patch
deleted file mode 100644 (file)
index 1969e81..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-From 54cbc058d86beca3515c994039b5c0f0a34f53dd Mon Sep 17 00:00:00 2001
-From: Bart Van Assche <bvanassche@acm.org>
-Date: Thu, 15 Feb 2024 12:47:39 -0800
-Subject: fs/aio: Make io_cancel() generate completions again
-
-From: Bart Van Assche <bvanassche@acm.org>
-
-commit 54cbc058d86beca3515c994039b5c0f0a34f53dd upstream.
-
-The following patch accidentally removed the code for delivering
-completions for cancelled reads and writes to user space: "[PATCH 04/33]
-aio: remove retry-based AIO"
-(https://lore.kernel.org/all/1363883754-27966-5-git-send-email-koverstreet@google.com/)
->From that patch:
-
--      if (kiocbIsCancelled(iocb)) {
--              ret = -EINTR;
--              aio_complete(iocb, ret, 0);
--              /* must not access the iocb after this */
--              goto out;
--      }
-
-This leads to a leak in user space of a struct iocb. Hence this patch
-that restores the code that reports to user space that a read or write
-has been cancelled successfully.
-
-Fixes: 41003a7bcfed ("aio: remove retry-based AIO")
-Cc: Christoph Hellwig <hch@lst.de>
-Cc: Avi Kivity <avi@scylladb.com>
-Cc: Sandeep Dhavale <dhavale@google.com>
-Cc: Jens Axboe <axboe@kernel.dk>
-Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-Cc: Kent Overstreet <kent.overstreet@linux.dev>
-Cc: stable@vger.kernel.org
-Signed-off-by: Bart Van Assche <bvanassche@acm.org>
-Link: https://lore.kernel.org/r/20240215204739.2677806-3-bvanassche@acm.org
-Signed-off-by: Christian Brauner <brauner@kernel.org>
-Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
----
- fs/aio.c |   27 +++++++++++----------------
- 1 file changed, 11 insertions(+), 16 deletions(-)
-
---- a/fs/aio.c
-+++ b/fs/aio.c
-@@ -2119,14 +2119,11 @@ COMPAT_SYSCALL_DEFINE3(io_submit, compat
- #endif
- /* sys_io_cancel:
-- *    Attempts to cancel an iocb previously passed to io_submit.  If
-- *    the operation is successfully cancelled, the resulting event is
-- *    copied into the memory pointed to by result without being placed
-- *    into the completion queue and 0 is returned.  May fail with
-- *    -EFAULT if any of the data structures pointed to are invalid.
-- *    May fail with -EINVAL if aio_context specified by ctx_id is
-- *    invalid.  May fail with -EAGAIN if the iocb specified was not
-- *    cancelled.  Will fail with -ENOSYS if not implemented.
-+ *    Attempts to cancel an iocb previously passed to io_submit(). If the
-+ *    operation is successfully cancelled 0 is returned. May fail with
-+ *    -EFAULT if any of the data structures pointed to are invalid. May
-+ *    fail with -EINVAL if aio_context specified by ctx_id is invalid. Will
-+ *    fail with -ENOSYS if not implemented.
-  */
- SYSCALL_DEFINE3(io_cancel, aio_context_t, ctx_id, struct iocb __user *, iocb,
-               struct io_event __user *, result)
-@@ -2157,14 +2154,12 @@ SYSCALL_DEFINE3(io_cancel, aio_context_t
-       }
-       spin_unlock_irq(&ctx->ctx_lock);
--      if (!ret) {
--              /*
--               * The result argument is no longer used - the io_event is
--               * always delivered via the ring buffer. -EINPROGRESS indicates
--               * cancellation is progress:
--               */
--              ret = -EINPROGRESS;
--      }
-+      /*
-+       * The result argument is no longer used - the io_event is always
-+       * delivered via the ring buffer.
-+       */
-+      if (ret == 0 && kiocb->rw.ki_flags & IOCB_AIO_RW)
-+              aio_complete_rw(&kiocb->rw, -EINTR);
-       percpu_ref_put(&ctx->users);
index 20055bf71d441766bbe8fd0572d7239b2610ce33..1b5a25ca8f1bfc3de7a260f3bdca34f0d44407c7 100644 (file)
@@ -113,7 +113,6 @@ iommufd-fix-protection-fault-in-iommufd_test_syz_conv_iova.patch
 efivarfs-request-at-most-512-bytes-for-variable-names.patch
 pmdomain-arm-fix-null-dereference-on-scmi_perf_domain-removal.patch
 pmdomain-qcom-rpmhpd-fix-enabled_corner-aggregation.patch
-fs-aio-make-io_cancel-generate-completions-again.patch
 fprobe-fix-to-allocate-entry_data_size-buffer-with-rethook-instances.patch
 mm-debug_vm_pgtable-fix-bug_on-with-pud-advanced-test.patch
 mm-vmscan-fix-a-bug-calling-wakeup_kswapd-with-a-wrong-zone-index.patch