[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PULL 43/46] job: Add query-jobs QMP command
From: |
Kevin Wolf |
Subject: |
[Qemu-block] [PULL 43/46] job: Add query-jobs QMP command |
Date: |
Wed, 23 May 2018 15:11:52 +0200 |
This adds a minimal query-jobs implementation that shouldn't pose many
design questions. It can later be extended to expose more information,
and especially job-specific information.
Signed-off-by: Kevin Wolf <address@hidden>
---
qapi/job.json | 46 ++++++++++++++++++++++++++++++++++++++++++++++
include/qemu/job.h | 3 +++
job-qmp.c | 54 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
job.c | 2 +-
4 files changed, 104 insertions(+), 1 deletion(-)
diff --git a/qapi/job.json b/qapi/job.json
index b84dc6c820..970124de76 100644
--- a/qapi/job.json
+++ b/qapi/job.json
@@ -205,3 +205,49 @@
# Since: 2.13
##
{ 'command': 'job-finalize', 'data': { 'id': 'str' } }
+
+##
+# @JobInfo:
+#
+# Information about a job.
+#
+# @id: The job identifier
+#
+# @type: The kind of job that is being performed
+#
+# @status: Current job state/status
+#
+# @current-progress: Progress made until now. The unit is arbitrary and the
+# value can only meaningfully be used for the ratio of
+# @current-progress to @total-progress. The value is
+# monotonically increasing.
+#
+# @total-progress: Estimated @current-progress value at the completion of
+# the job. This value can arbitrarily change while the
+# job is running, in both directions.
+#
+# @error: If this field is present, the job failed; if it is
+# still missing in the CONCLUDED state, this indicates
+# successful completion.
+#
+# The value is a human-readable error message to describe
+# the reason for the job failure. It should not be parsed
+# by applications.
+#
+# Since: 2.13
+##
+{ 'struct': 'JobInfo',
+ 'data': { 'id': 'str', 'type': 'JobType', 'status': 'JobStatus',
+ 'current-progress': 'int', 'total-progress': 'int',
+ '*error': 'str' } }
+
+##
+# @query-jobs:
+#
+# Return information about jobs.
+#
+# Returns: a list with a @JobInfo for each active job
+#
+# Since: 2.13
+##
+{ 'command': 'query-jobs', 'returns': ['JobInfo'] }
diff --git a/include/qemu/job.h b/include/qemu/job.h
index 92d1d249fe..8c8badf75e 100644
--- a/include/qemu/job.h
+++ b/include/qemu/job.h
@@ -392,6 +392,9 @@ JobType job_type(const Job *job);
/** Returns the enum string for the JobType of a given Job. */
const char *job_type_str(const Job *job);
+/** Returns true if the job should not be visible to the management layer. */
+bool job_is_internal(Job *job);
+
/** Returns whether the job is scheduled for cancellation. */
bool job_is_cancelled(Job *job);
diff --git a/job-qmp.c b/job-qmp.c
index b2e18cfd9c..7f38f63336 100644
--- a/job-qmp.c
+++ b/job-qmp.c
@@ -132,3 +132,57 @@ void qmp_job_dismiss(const char *id, Error **errp)
job_dismiss(&job, errp);
aio_context_release(aio_context);
}
+
+static JobInfo *job_query_single(Job *job, Error **errp)
+{
+ JobInfo *info;
+ const char *errmsg = NULL;
+
+ assert(!job_is_internal(job));
+
+ if (job->ret < 0) {
+ errmsg = strerror(-job->ret);
+ }
+
+ info = g_new(JobInfo, 1);
+ *info = (JobInfo) {
+ .id = g_strdup(job->id),
+ .type = job_type(job),
+ .status = job->status,
+ .current_progress = job->progress_current,
+ .total_progress = job->progress_total,
+ .has_error = !!errmsg,
+ .error = g_strdup(errmsg),
+ };
+
+ return info;
+}
+
+JobInfoList *qmp_query_jobs(Error **errp)
+{
+ JobInfoList *head = NULL, **p_next = &head;
+ Job *job;
+
+ for (job = job_next(NULL); job; job = job_next(job)) {
+ JobInfoList *elem;
+ AioContext *aio_context;
+
+ if (job_is_internal(job)) {
+ continue;
+ }
+ elem = g_new0(JobInfoList, 1);
+ aio_context = job->aio_context;
+ aio_context_acquire(aio_context);
+ elem->value = job_query_single(job, errp);
+ aio_context_release(aio_context);
+ if (!elem->value) {
+ g_free(elem);
+ qapi_free_JobInfoList(head);
+ return NULL;
+ }
+ *p_next = elem;
+ p_next = &elem->next;
+ }
+
+ return head;
+}
diff --git a/job.c b/job.c
index 599a1042cf..f026661b0f 100644
--- a/job.c
+++ b/job.c
@@ -158,7 +158,7 @@ static int job_txn_apply(JobTxn *txn, int fn(Job *), bool
lock)
return rc;
}
-static bool job_is_internal(Job *job)
+bool job_is_internal(Job *job)
{
return (job->id == NULL);
}
--
2.13.6
- [Qemu-block] [PULL 27/46] job: Convert block_job_cancel_async() to Job, (continued)
- [Qemu-block] [PULL 27/46] job: Convert block_job_cancel_async() to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 39/46] job: Move progress fields to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 26/46] job: Move single job finalisation to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 29/46] job: Move .complete callback to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 25/46] job: Add job_event_*(), Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 35/46] job: Add job_yield(), Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 33/46] job: Move completion and cancellation to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 36/46] job: Add job_dismiss(), Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 37/46] job: Add job_is_ready(), Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 32/46] job: Move transactions to Job, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 43/46] job: Add query-jobs QMP command,
Kevin Wolf <=
- [Qemu-block] [PULL 40/46] job: Introduce qapi/job.json, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 45/46] iotests: Move qmp_to_opts() to VM, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 42/46] job: Add lifecycle QMP commands, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 44/46] blockjob: Remove BlockJob.driver, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 46/46] qemu-iotests: Test job-* with block jobs, Kevin Wolf, 2018/05/23
- [Qemu-block] [PULL 41/46] job: Add JOB_STATUS_CHANGE QMP event, Kevin Wolf, 2018/05/23
- Re: [Qemu-block] [Qemu-devel] [PULL 00/46] Block layer patches, no-reply, 2018/05/23
- Re: [Qemu-block] [Qemu-devel] [PULL 00/46] Block layer patches, Peter Maydell, 2018/05/24