Peter Xu <pet...@redhat.com> wrote:
> It's just a simple wrapper for rp_sem on either wait() or kick(), make it
> even clearer on how it is used. Prepared to be used even for other things.
>
> Reviewed-by: Fabiano Rosas <faro...@suse.de>
> Signed-off-by: Peter Xu <pet...@redhat.com>
Reviewed-by: Juan Quintela <quint...@redhat.com>
I agree with the idea, but I think that the problem is the name of the
semaphore.
> +void migration_rp_wait(MigrationState *s)
> +{
> + qemu_sem_wait(&s->rp_state.rp_sem);
I am not sure if it would be better to have the wrappers or just rename
If we rename the remaphore to migration_thread, this becomes:
qemu_sem_wait(&s->rp_state.return_path_ready);
qemu_sem_post(&s->rp_state.return_path_ready);
Or something similar?
