Re: [PATCH 0/2] block-backend: Retain permissions after migration

2022-01-10 Thread Peng Liang via
On 11/25/2021 9:53 PM, Hanna Reitz wrote: > Hi, > > Peng Liang has reported an issue regarding migration of raw images here: > https://lists.nongnu.org/archive/html/qemu-block/2021-11/msg00673.html > > It turns out that after migrating, all permissions are shared when the

Re: [PATCH 1/2] block-backend: Retain permissions after migration

2021-11-26 Thread Peng Liang via
5f7772c4d0cf32f4e779fcd5a69ae4dae24aeebf >("block-backend: Defer shared_perm tightening migration >completion") > Reported-by: Peng Liang > Signed-off-by: Hanna Reitz > --- > block/block-backend.c | 11 +++ > 1 file changed, 11 insertions(+) > Thanks for your patch! Tested-by: Peng Liang

Questions about losing the write lock of raw-format disks after migration

2021-11-24 Thread Peng Liang via
Hi folks, When we test migration with raw-format disk, we found that the QEMU process in the dst will lose the write lock after migration. However, the QEMU process in the dst will still hold the write lock for qcow2-format disk. After reading some block layer's code, I found that the first

Re: [PATCH] block/mirror: fix core when using iothreads

2021-02-01 Thread Peng Liang
On 9/23/2020 10:50 PM, John Snow wrote: > On 8/26/20 9:19 AM, Peng Liang wrote: >> We found an issue when doing block-commit with iothreads, which tries to >> dereference a NULL pointer. >> > > I'm clearing out my patch backlog. I am a bit out of the loop on block

Re: [PATCH v4] blockjob: Fix crash with IOthread when block commit after snapshot

2021-02-01 Thread Peng Liang
Hi, I encountered the problem months ago too. Could we move the creation of the block job (block_job_create) before appending the new bs to mirror_top_bs (bdrv_append) as I wrote in [*]? I found that after bdrv_append, qemu will use mirror_top_bs to do write. And when writing, qemu will use

[PATCH] block/nbd: Fix crash when cancelling drive mirror

2020-12-30 Thread Peng Liang
1e8, envp=0x7ffe4cf8f4e0) at ../softmmu/main.c:50 Reported-by: Euler Robot Signed-off-by: Peng Liang --- block/nbd.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/block/nbd.c b/block/nbd.c index 42536702b6f9..b22a9ecbbcd0 100644 --- a/block/nbd.c +++ b/block/nb

[PATCH] block/mirror: fix core when using iothreads

2020-08-26 Thread Peng Liang
| (s->job is not NULL until 3!) 3. bs_opaque->job = s;| Just moving step 2 & 3 before 1 can avoid this. Reported-by: Euler Robot Signed-off-by: Peng Liang --- block/mirror.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-)