880eeec613
Now that the API offers also _locked() functions, take advantage of it and give also the caller control to take the lock and call _locked functions. This makes sense especially when we have for loops, because it makes no sense to have: for(job = job_next(); ...) where each job_next() takes the lock internally. Instead we want JOB_LOCK_GUARD(); for(job = job_next_locked(); ...) In addition, protect also direct field accesses, by either creating a new critical section or widening the existing ones. Note: at this stage, job_{lock/unlock} and job lock guard macros are *nop*. Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru> Message-Id: <20220926093214.506243-12-eesposit@redhat.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
220 lines
5.3 KiB
C
220 lines
5.3 KiB
C
/*
|
|
* QMP interface for background jobs
|
|
*
|
|
* Copyright (c) 2011 IBM Corp.
|
|
* Copyright (c) 2012, 2018 Red Hat, Inc.
|
|
*
|
|
* Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
* of this software and associated documentation files (the "Software"), to deal
|
|
* in the Software without restriction, including without limitation the rights
|
|
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
* copies of the Software, and to permit persons to whom the Software is
|
|
* furnished to do so, subject to the following conditions:
|
|
*
|
|
* The above copyright notice and this permission notice shall be included in
|
|
* all copies or substantial portions of the Software.
|
|
*
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
|
* THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
* THE SOFTWARE.
|
|
*/
|
|
|
|
#include "qemu/osdep.h"
|
|
#include "qemu/job.h"
|
|
#include "qapi/qapi-commands-job.h"
|
|
#include "qapi/error.h"
|
|
#include "trace/trace-root.h"
|
|
|
|
/*
|
|
* Get a job using its ID and acquire its AioContext.
|
|
* Called with job_mutex held.
|
|
*/
|
|
static Job *find_job_locked(const char *id,
|
|
AioContext **aio_context,
|
|
Error **errp)
|
|
{
|
|
Job *job;
|
|
|
|
*aio_context = NULL;
|
|
|
|
job = job_get_locked(id);
|
|
if (!job) {
|
|
error_setg(errp, "Job not found");
|
|
return NULL;
|
|
}
|
|
|
|
*aio_context = job->aio_context;
|
|
aio_context_acquire(*aio_context);
|
|
|
|
return job;
|
|
}
|
|
|
|
void qmp_job_cancel(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_cancel(job);
|
|
job_user_cancel_locked(job, true, errp);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
void qmp_job_pause(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_pause(job);
|
|
job_user_pause_locked(job, errp);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
void qmp_job_resume(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_resume(job);
|
|
job_user_resume_locked(job, errp);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
void qmp_job_complete(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_complete(job);
|
|
job_complete_locked(job, errp);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
void qmp_job_finalize(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_finalize(job);
|
|
job_ref_locked(job);
|
|
job_finalize_locked(job, errp);
|
|
|
|
/*
|
|
* Job's context might have changed via job_finalize (and job_txn_apply
|
|
* automatically acquires the new one), so make sure we release the correct
|
|
* one.
|
|
*/
|
|
aio_context = job->aio_context;
|
|
job_unref_locked(job);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
void qmp_job_dismiss(const char *id, Error **errp)
|
|
{
|
|
AioContext *aio_context;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
job = find_job_locked(id, &aio_context, errp);
|
|
|
|
if (!job) {
|
|
return;
|
|
}
|
|
|
|
trace_qmp_job_dismiss(job);
|
|
job_dismiss_locked(&job, errp);
|
|
aio_context_release(aio_context);
|
|
}
|
|
|
|
/* Called with job_mutex held. */
|
|
static JobInfo *job_query_single_locked(Job *job, Error **errp)
|
|
{
|
|
JobInfo *info;
|
|
uint64_t progress_current;
|
|
uint64_t progress_total;
|
|
|
|
assert(!job_is_internal(job));
|
|
progress_get_snapshot(&job->progress, &progress_current,
|
|
&progress_total);
|
|
|
|
info = g_new(JobInfo, 1);
|
|
*info = (JobInfo) {
|
|
.id = g_strdup(job->id),
|
|
.type = job_type(job),
|
|
.status = job->status,
|
|
.current_progress = progress_current,
|
|
.total_progress = progress_total,
|
|
.has_error = !!job->err,
|
|
.error = job->err ? \
|
|
g_strdup(error_get_pretty(job->err)) : NULL,
|
|
};
|
|
|
|
return info;
|
|
}
|
|
|
|
JobInfoList *qmp_query_jobs(Error **errp)
|
|
{
|
|
JobInfoList *head = NULL, **tail = &head;
|
|
Job *job;
|
|
|
|
JOB_LOCK_GUARD();
|
|
|
|
for (job = job_next_locked(NULL); job; job = job_next_locked(job)) {
|
|
JobInfo *value;
|
|
AioContext *aio_context;
|
|
|
|
if (job_is_internal(job)) {
|
|
continue;
|
|
}
|
|
aio_context = job->aio_context;
|
|
aio_context_acquire(aio_context);
|
|
value = job_query_single_locked(job, errp);
|
|
aio_context_release(aio_context);
|
|
if (!value) {
|
|
qapi_free_JobInfoList(head);
|
|
return NULL;
|
|
}
|
|
QAPI_LIST_APPEND(tail, value);
|
|
}
|
|
|
|
return head;
|
|
}
|