diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2018-05-24 15:26:10 +0200 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2018-05-30 13:31:01 +0200 |
| commit | 1266c9b9f5fa05877b979eece5963a2bd99c3bfd (patch) | |
| tree | 348fa8501283cccf4a62ace8e262ca6bef6c1939 /tests/test-blockjob.c | |
| parent | 4a5f2779bad769184550869931937acd0707ec3b (diff) | |
| download | qemu-1266c9b9f5fa05877b979eece5963a2bd99c3bfd.zip qemu-1266c9b9f5fa05877b979eece5963a2bd99c3bfd.tar.gz qemu-1266c9b9f5fa05877b979eece5963a2bd99c3bfd.tar.bz2 | |
job: Add error message for failing jobs
So far we relied on job->ret and strerror() to produce an error message
for failed jobs. Not surprisingly, this tends to result in completely
useless messages.
This adds a Job.error field that can contain an error string for a
failing job, and a parameter to job_completed() that sets the field. As
a default, if NULL is passed, we continue to use strerror(job->ret).
All existing callers are changed to pass NULL. They can be improved in
separate patches.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Jeff Cody <jcody@redhat.com>
Diffstat (limited to 'tests/test-blockjob.c')
| -rw-r--r-- | tests/test-blockjob.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/test-blockjob.c b/tests/test-blockjob.c index e408d52..cb42f06 100644 --- a/tests/test-blockjob.c +++ b/tests/test-blockjob.c @@ -167,7 +167,7 @@ static void cancel_job_completed(Job *job, void *opaque) { CancelJob *s = opaque; s->completed = true; - job_completed(job, 0); + job_completed(job, 0, NULL); } static void cancel_job_complete(Job *job, Error **errp) |