All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhang Chen <zhangckid@gmail.com>
To: qemu-devel <qemu-devel@nongnu.org>,
	"Eric Blake" <eblake@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	"'Michael S . Tsirkin'" <mst@redhat.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"'Daniel P . Berrangé'" <berrange@redhat.com>,
	"Jason Wang" <jasowangio@gmail.com>
Cc: Zhang Chen <zhangckid@gmail.com>
Subject: [PATCH V14 04/15] iothread: introduce iothread_unsafe_get_aio_context()
Date: Tue,  1 Sep 2026 00:00:25 +0800	[thread overview]
Message-ID: <20260831160038.221776-5-zhangckid@gmail.com> (raw)
In-Reply-To: <20260831160038.221776-1-zhangckid@gmail.com>

Add an explicitly unsafe accessor for legacy callers that cannot
provide a matching holder lifecycle, and convert blockdev.c to use it.

Keep iothread_get_aio_context() temporarily so that later caller
conversions remain independently buildable. It will be removed after
all production callers have migrated.

New code should use iothread_ref_and_get_aio_context().

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Zhang Chen <zhangckid@gmail.com>
---
 blockdev.c                |  2 +-
 include/system/iothread.h | 10 ++++++++++
 iothread.c                |  5 +++++
 3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/blockdev.c b/blockdev.c
index 6e86c6262f..baeab3a3e1 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -3683,7 +3683,7 @@ void qmp_x_blockdev_set_iothread(const char *node_name, StrOrNull *iothread,
             goto out;
         }
 
-        new_context = iothread_get_aio_context(obj);
+        new_context = iothread_unsafe_get_aio_context(obj);
     } else {
         new_context = qemu_get_aio_context();
     }
diff --git a/include/system/iothread.h b/include/system/iothread.h
index f6c5f95dd9..2ef8c1a1bb 100644
--- a/include/system/iothread.h
+++ b/include/system/iothread.h
@@ -68,6 +68,16 @@ char *iothread_get_id(IOThread *iothread);
 IOThread *iothread_by_id(const char *id);
 AioContext *iothread_get_aio_context(IOThread *iothread);
 
+/*
+ * Return @iothread's AioContext without registering a holder or taking a
+ * reference on @iothread.  The caller must ensure that the IOThread remains
+ * alive for as long as the returned AioContext is used.
+ *
+ * This API exists for legacy callers without a clear ref/unref lifecycle.  Do
+ * not use it in new code; use iothread_ref_and_get_aio_context() instead.
+ */
+AioContext *iothread_unsafe_get_aio_context(IOThread *iothread);
+
 /*
  * Register @holder and return @iothread's AioContext.  The holder is copied,
  * and a reference is taken on @iothread so that both the IOThread and its
diff --git a/iothread.c b/iothread.c
index 58dace351c..803d6ce3c3 100644
--- a/iothread.c
+++ b/iothread.c
@@ -443,6 +443,11 @@ void iothread_unref_and_put_aio_context(IOThread *iothread,
     iothread_unref(iothread, holder);
 }
 
+AioContext *iothread_unsafe_get_aio_context(IOThread *iothread)
+{
+    return iothread->ctx;
+}
+
 static int query_one_iothread(Object *object, void *opaque)
 {
     IOThreadInfoList ***tail = opaque;
-- 
2.55.0



  parent reply	other threads:[~2026-08-31 16:03 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-31 16:00 [PATCH V14 00/15] iothread: Support tracking and querying IOThread holder Zhang Chen
2026-08-31 16:00 ` [PATCH V14 01/15] qapi/misc: Fix missed query-iothreads items Zhang Chen
2026-08-31 16:00 ` [PATCH V14 02/15] iothread: introduce holder tracking Zhang Chen
2026-08-31 16:00 ` [PATCH V14 03/15] iothread: add holder-aware AioContext accessors Zhang Chen
2026-09-01  5:18   ` Markus Armbruster
2026-08-31 16:00 ` Zhang Chen [this message]
2026-09-01  5:30   ` [PATCH V14 04/15] iothread: introduce iothread_unsafe_get_aio_context() Markus Armbruster
2026-09-01 14:39     ` Zhang Chen
2026-09-02 10:46       ` Markus Armbruster
2026-08-31 16:00 ` [PATCH V14 05/15] block/export: track IOThread references Zhang Chen
2026-09-02 19:20   ` Stefan Hajnoczi
2026-08-31 16:00 ` [PATCH V14 06/15] monitor: track IOThread users with QOM paths Zhang Chen
2026-09-01  5:47   ` Markus Armbruster
2026-09-01 14:47     ` Zhang Chen
2026-09-02 11:25       ` Markus Armbruster
2026-09-02 11:53         ` Daniel P. Berrangé
2026-09-02 14:37           ` Markus Armbruster
2026-09-02 16:17             ` Daniel P. Berrangé
2026-09-04  8:33               ` Markus Armbruster
2026-09-04  8:57                 ` Daniel P. Berrangé
2026-09-04  9:37                   ` Zhang Chen
2026-09-02 19:35   ` Stefan Hajnoczi
2026-09-03  6:05     ` Markus Armbruster
2026-09-03 13:42       ` Stefan Hajnoczi
2026-09-03 14:52         ` Zhang Chen
2026-08-31 16:00 ` [PATCH V14 07/15] virtio-vq-mapping: track iothread-vq-mapping references using device path Zhang Chen
2026-08-31 16:00 ` [PATCH V14 08/15] virtio: track IOThread references for thread pinning Zhang Chen
2026-08-31 16:00 ` [PATCH V14 09/15] net/colo: track IOThread references using path-based holder Zhang Chen
2026-08-31 16:00 ` [PATCH V14 10/15] virtio-balloon: Update tracking iothread users with holder Zhang Chen
2026-08-31 16:00 ` [PATCH V14 11/15] vfio-user/proxy: Update tracking iothread users with holder name Zhang Chen
2026-08-31 16:00 ` [PATCH V14 12/15] xen-block: " Zhang Chen
2026-08-31 16:00 ` [PATCH V14 13/15] monitor/hmp: display IOThread holders Zhang Chen
2026-08-31 16:00 ` [PATCH V14 14/15] iothread: remove legacy iothread_get_aio_context() Zhang Chen
2026-09-01  5:35   ` Markus Armbruster
2026-09-01 14:39     ` Zhang Chen
2026-08-31 16:00 ` [PATCH V14 15/15] tests/unit/iothread: update AioContext ref/put helpers Zhang Chen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260831160038.221776-5-zhangckid@gmail.com \
    --to=zhangckid@gmail.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=eblake@redhat.com \
    --cc=jasowangio@gmail.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.