[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 5/7] qapi: add job-change
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[PATCH v2 5/7] qapi: add job-change |
Date: |
Wed, 26 Jun 2024 14:01:22 +0300 |
Add a new-style command job-change, doing same thing as
block-job-change. The aim is finally deprecate block-job-* APIs and
move to job-* APIs.
We add a new command to qapi/block-core.json, not to
qapi/job.json to avoid resolving json file including loops for now.
This all would be a lot simple to refactor when we finally drop
deprecated block-job-* APIs.
@type argument of the new command immediately becomes deprecated.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
---
job-qmp.c | 14 ++++++++++++++
qapi/block-core.json | 10 ++++++++++
2 files changed, 24 insertions(+)
diff --git a/job-qmp.c b/job-qmp.c
index c764bd3801..248e68f554 100644
--- a/job-qmp.c
+++ b/job-qmp.c
@@ -139,6 +139,20 @@ void qmp_job_dismiss(const char *id, Error **errp)
job_dismiss_locked(&job, errp);
}
+void qmp_job_change(JobChangeOptions *opts, Error **errp)
+{
+ Job *job;
+
+ JOB_LOCK_GUARD();
+ job = find_job_locked(opts->id, errp);
+
+ if (!job) {
+ return;
+ }
+
+ job_change_locked(job, opts, errp);
+}
+
/* Called with job_mutex held. */
static JobInfo *job_query_single_locked(Job *job, Error **errp)
{
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 660c7f4a48..9087ce300c 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -3104,6 +3104,16 @@
{ 'command': 'block-job-change',
'data': 'JobChangeOptions', 'boxed': true }
+##
+# @job-change:
+#
+# Change the block job's options.
+#
+# Since: 9.1
+##
+{ 'command': 'job-change',
+ 'data': 'JobChangeOptions', 'boxed': true }
+
##
# @BlockdevDiscardOptions:
#
--
2.34.1
- [PATCH v2 0/7] introduce job-change qmp command, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 4/7] blockjob: move change action implementation to job from block-job, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 7/7] iotests/mirror-change-copy-mode: switch to job-change command, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 3/7] qapi: block-job-change: make copy-mode parameter optional, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 5/7] qapi: add job-change,
Vladimir Sementsov-Ogievskiy <=
- [PATCH v2 6/7] qapi/block-core: derpecate block-job-change, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 2/7] blockjob: block_job_change_locked(): check job type, Vladimir Sementsov-Ogievskiy, 2024/06/26
- [PATCH v2 1/7] qapi: rename BlockJobChangeOptions to JobChangeOptions, Vladimir Sementsov-Ogievskiy, 2024/06/26