diff options
author | Emanuele Giuseppe Esposito <eesposit@redhat.com> | 2022-09-26 05:31:54 -0400 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2022-10-07 12:11:41 +0200 |
commit | 55c5a25a0363f153d8875a60001342eb6fe6e4f5 (patch) | |
tree | 36e046ad659f66fc2fa588aa6d44dae31a296838 /io/channel-command.c | |
parent | 2ffc10d53b6ab57ab228359709f8703b0b010430 (diff) | |
download | qemu-55c5a25a0363f153d8875a60001342eb6fe6e4f5.zip qemu-55c5a25a0363f153d8875a60001342eb6fe6e4f5.tar.gz qemu-55c5a25a0363f153d8875a60001342eb6fe6e4f5.tar.bz2 |
job.c: make job_mutex and job_lock/unlock() public
job mutex will be used to protect the job struct elements and list,
replacing AioContext locks.
Right now use a shared lock for all jobs, in order to keep things
simple. Once the AioContext lock is gone, we can introduce per-job
locks.
To simplify the switch from aiocontext to job lock, introduce
*nop* lock/unlock functions and macros.
We want to always call job_lock/unlock outside the AioContext locks,
and not vice-versa, otherwise we might get a deadlock. This is not
straightforward to do, and that's why we start with nop functions.
Once everything is protected by job_lock/unlock, we can change the nop into
an actual mutex and remove the aiocontext lock.
Since job_mutex is already being used, add static
real_job_{lock/unlock} for the existing usage.
Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
Message-Id: <20220926093214.506243-2-eesposit@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'io/channel-command.c')
0 files changed, 0 insertions, 0 deletions