diff options
| author | Emanuele Giuseppe Esposito | 2022-09-26 11:32:04 +0200 |
|---|---|---|
| committer | Kevin Wolf | 2022-10-07 12:11:41 +0200 |
| commit | 880eeec61329abc0aead900f0037fce91571b1ec (patch) | |
| tree | 684676e354ff54754203f40579d6b7eb969805c7 /include | |
| parent | block/mirror.c: use of job helpers in drivers (diff) | |
| download | qemu-880eeec61329abc0aead900f0037fce91571b1ec.tar.gz qemu-880eeec61329abc0aead900f0037fce91571b1ec.tar.xz qemu-880eeec61329abc0aead900f0037fce91571b1ec.zip | |
jobs: group together API calls under the same job lock
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>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions
