20.10.2020 11:16, Laurent Vivier wrote:
Le 20/10/2020 à 09:36, Chenqun (kuhn) a écrit :
Ping!

Hello,

   Maybe this patch, some people have any other suggestions?   Or, maybe missed 
to queue?

Hi,

As we've seen by reviewing previous versions, this change is not really
trivial, so I will not take it via the trivial queue.

Perhaps via block I/O queue (Stefan?), Dirty Bitmaps (John?) or
Migration (Juan?).


Correct would be Eric's bitmaps tree.


Thanks,
Chen Qun

-----Original Message-----
From: Vladimir Sementsov-Ogievskiy [mailto:vsement...@virtuozzo.com]
Sent: Wednesday, October 14, 2020 11:56 PM
To: Chenqun (kuhn) <kuhn.chen...@huawei.com>; qemu-devel@nongnu.org;
qemu-triv...@nongnu.org
Cc: mre...@redhat.com; stefa...@redhat.com; f...@euphon.net;
ebl...@redhat.com; js...@redhat.com; quint...@redhat.com;
dgilb...@redhat.com; Zhanghailiang <zhang.zhanghaili...@huawei.com>;
ganqixin <ganqi...@huawei.com>; qemu-bl...@nongnu.org; Euler Robot
<euler.ro...@huawei.com>; Laurent Vivier <laur...@vivier.eu>; Li Qiang
<liq...@gmail.com>
Subject: Re: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable
warning

14.10.2020 14:44, Chen Qun wrote:
A default value is provided for the variable 'bitmap_name' to avoid compiler
warning.

The compiler show warning:
migration/block-dirty-bitmap.c:1090:13: warning: ‘bitmap_name’
may be used uninitialized in this function [-Wmaybe-uninitialized]
         g_strlcpy(s->bitmap_name, bitmap_name,
sizeof(s->bitmap_name));

^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Reported-by: Euler Robot<euler.ro...@huawei.com>
Signed-off-by: Chen Qun<kuhn.chen...@huawei.com>

Reviewed-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>

--
Best regards,
Vladimir



--
Best regards,
Vladimir

Reply via email to