[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v8 30/31] job.h: split function pointers in JobDriver
From: |
Emanuele Giuseppe Esposito |
Subject: |
[PATCH v8 30/31] job.h: split function pointers in JobDriver |
Date: |
Thu, 3 Mar 2022 10:16:15 -0500 |
The job API will be handled separately in another serie.
Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
---
include/qemu/job.h | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/include/qemu/job.h b/include/qemu/job.h
index 6e67b6977f..c105b31076 100644
--- a/include/qemu/job.h
+++ b/include/qemu/job.h
@@ -169,6 +169,12 @@ typedef struct Job {
* Callbacks and other information about a Job driver.
*/
struct JobDriver {
+
+ /*
+ * These fields are initialized when this object is created,
+ * and are never changed afterwards
+ */
+
/** Derived Job struct size */
size_t instance_size;
@@ -184,9 +190,18 @@ struct JobDriver {
* aborted. If it returns zero, the job moves into the WAITING state. If it
* is the last job to complete in its transaction, all jobs in the
* transaction move from WAITING to PENDING.
+ *
+ * This callback must be run in the job's context.
*/
int coroutine_fn (*run)(Job *job, Error **errp);
+ /*
+ * Functions run without regard to the BQL that may run in any
+ * arbitrary thread. These functions do not need to be thread-safe
+ * because the caller ensures that they are invoked from one
+ * thread at time.
+ */
+
/**
* If the callback is not NULL, it will be invoked when the job transitions
* into the paused state. Paused jobs must not perform any asynchronous
@@ -201,6 +216,13 @@ struct JobDriver {
*/
void coroutine_fn (*resume)(Job *job);
+ /*
+ * Global state (GS) API. These functions run under the BQL.
+ *
+ * See include/block/block-global-state.h for more information about
+ * the GS API.
+ */
+
/**
* Called when the job is resumed by the user (i.e. user_paused becomes
* false). .user_resume is called before .resume.
--
2.31.1
- [PATCH v8 19/31] assertions for blockjob.h global state API, (continued)
- [PATCH v8 19/31] assertions for blockjob.h global state API, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 23/31] block/copy-before-write.h: global state API + assertions, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 25/31] block_int-common.h: split function pointers in BlockDriver, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 31/31] job.h: assertions in the callers of JobDriver function pointers, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 18/31] include/block/blockjob.h: global state API, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 20/31] include/sysemu/blockdev.h: global state API, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 26/31] block_int-common.h: assertions in the callers of BlockDriver function pointers, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 22/31] include/block/snapshot: global state API + assertions, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 21/31] assertions for blockdev.h global state API, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 28/31] block_int-common.h: assertions in the callers of BdrvChildClass function pointers, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 30/31] job.h: split function pointers in JobDriver,
Emanuele Giuseppe Esposito <=
- [PATCH v8 24/31] block/coroutines: I/O and "I/O or GS" API, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 27/31] block_int-common.h: split function pointers in BdrvChildClass, Emanuele Giuseppe Esposito, 2022/03/03
- [PATCH v8 29/31] block-backend-common.h: split function pointers in BlockDevOps, Emanuele Giuseppe Esposito, 2022/03/03
- Re: [PATCH v8 00/31] block layer: split block APIs in global state and I/O, Kevin Wolf, 2022/03/04