Re: [PATCH] migration: Unlock mutex in error case

2023-11-03 Thread Juan Quintela
Alex Bennée wrote: > Juan Quintela writes: > >> We were not unlocking bitmap mutex on the error case. >> Coverity discovered the problem. >> >> Fixes: a2326705e5 ("migration: Stop migration immediately in RDMA error >> paths") >> Signed-off-by: Juan Quintela >> --- >> migration/ram.c | 1 + >>

Re: [PATCH] migration: Unlock mutex in error case

2023-11-02 Thread Peter Xu
On Thu, Nov 02, 2023 at 10:12:45AM +0100, Juan Quintela wrote: > We were not unlocking bitmap mutex on the error case. > Coverity discovered the problem. > > Fixes: a2326705e5 ("migration: Stop migration immediately in RDMA error > paths") > Signed-off-by: Juan Quintela > --- > migration/ram.c

Re: [PATCH] migration: Unlock mutex in error case

2023-11-02 Thread Alex Bennée
Juan Quintela writes: > We were not unlocking bitmap mutex on the error case. > Coverity discovered the problem. > > Fixes: a2326705e5 ("migration: Stop migration immediately in RDMA error > paths") > Signed-off-by: Juan Quintela > --- > migration/ram.c | 1 + > 1 file changed, 1 insertion(+)

Re: [PATCH] migration: Unlock mutex in error case

2023-11-02 Thread Peter Maydell
On Thu, 2 Nov 2023 at 09:13, Juan Quintela wrote: > > We were not unlocking bitmap mutex on the error case. > Coverity discovered the problem. > > Fixes: a2326705e5 ("migration: Stop migration immediately in RDMA error > paths") > Signed-off-by: Juan Quintela CID 1523750. thanks -- PMM

[PATCH] migration: Unlock mutex in error case

2023-11-02 Thread Juan Quintela
We were not unlocking bitmap mutex on the error case. Coverity discovered the problem. Fixes: a2326705e5 ("migration: Stop migration immediately in RDMA error paths") Signed-off-by: Juan Quintela --- migration/ram.c | 1 + 1 file changed, 1 insertion(+) diff --git a/migration/ram.c b/migration/