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 V12 04/15] iothread: introduce iothread_unsafe_get_aio_context()
Date: Sat, 15 Aug 2026 01:48:18 +0800	[thread overview]
Message-ID: <20260814174830.1691540-5-zhangckid@gmail.com> (raw)
In-Reply-To: <20260814174830.1691540-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().

Signed-off-by: Zhang Chen <zhangckid@gmail.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.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 64be852d1c..3afa296aa0 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.54.0



  parent reply	other threads:[~2026-08-14 17:49 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-14 17:48 [PATCH V12 00/15] iothread: Support tracking and querying IOThread holders Zhang Chen
2026-08-14 17:48 ` [PATCH V12 01/15] qapi/misc: Fix missed query-iothreads items Zhang Chen
2026-08-14 17:48 ` [PATCH V12 02/15] iothread: introduce holder tracking Zhang Chen
2026-08-14 17:48 ` [PATCH V12 03/15] iothread: track users with holder name Zhang Chen
2026-08-18 20:31   ` Stefan Hajnoczi
2026-08-14 17:48 ` Zhang Chen [this message]
2026-08-14 17:48 ` [PATCH V12 05/15] block/export: track IOThread references Zhang Chen
2026-08-18 20:49   ` Stefan Hajnoczi
2026-08-14 17:48 ` [PATCH V12 06/15] monitor: track IOThread users with QOM paths Zhang Chen
2026-08-14 17:48 ` [PATCH V12 07/15] virtio-vq-mapping: track iothread-vq-mapping references using device path Zhang Chen
2026-08-14 17:48 ` [PATCH V12 08/15] virtio: track IOThread references for thread pinning Zhang Chen
2026-08-14 17:48 ` [PATCH V12 09/15] net/colo: track IOThread references using path-based holder Zhang Chen
2026-08-14 17:48 ` [PATCH V12 10/15] virtio-balloon: Update tracking iothread users with holder Zhang Chen
2026-08-14 17:48 ` [PATCH V12 11/15] vfio-user/proxy: Update tracking iothread users with holder name Zhang Chen
2026-08-14 17:48 ` [PATCH V12 12/15] xen-block: " Zhang Chen
2026-08-14 17:48 ` [PATCH V12 13/15] monitor/hmp: display IOThread holders Zhang Chen
2026-08-14 17:48 ` [PATCH V12 14/15] iothread: remove legacy iothread_get_aio_context() Zhang Chen
2026-08-14 17:48 ` [PATCH V12 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=20260814174830.1691540-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.