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:[email protected]]
Sent: Wednesday, October 14, 2020 11:56 PM
To: Chenqun (kuhn) <[email protected]>; [email protected];
[email protected]
Cc: [email protected]; [email protected]; [email protected];
[email protected]; [email protected]; [email protected];
[email protected]; Zhanghailiang <[email protected]>;
ganqixin <[email protected]>; [email protected]; Euler Robot
<[email protected]>; Laurent Vivier <[email protected]>; Li Qiang
<[email protected]>
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<[email protected]>
Signed-off-by: Chen Qun<[email protected]>

Reviewed-by: Vladimir Sementsov-Ogievskiy <[email protected]>

--
Best regards,
Vladimir



--
Best regards,
Vladimir

Reply via email to