* [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
@ 2020-10-14 11:44 Chen Qun
2020-10-14 15:56 ` Vladimir Sementsov-Ogievskiy
0 siblings, 1 reply; 6+ messages in thread
From: Chen Qun @ 2020-10-14 11:44 UTC (permalink / raw)
To: qemu-devel, qemu-trivial
Cc: fam, ganqixin, vsementsov, zhang.zhanghailiang, qemu-block,
quintela, Li Qiang, dgilbert, mreitz, stefanha, Euler Robot,
Chen Qun, jsnow, Laurent Vivier
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.robot@huawei.com>
Signed-off-by: Chen Qun <kuhn.chenqun@huawei.com>
---
v2->v3: Placing the declaration at the beginning of the block(Base on Max's suggestion).
Cc: Max Reitz <mreitz@redhat.com>
Cc: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Cc: Laurent Vivier <laurent@vivier.eu>
Cc: Li Qiang <liq3ea@gmail.com>
---
migration/block-dirty-bitmap.c | 21 +++++++++------------
1 file changed, 9 insertions(+), 12 deletions(-)
diff --git a/migration/block-dirty-bitmap.c b/migration/block-dirty-bitmap.c
index 5bef793ac0..114987961a 100644
--- a/migration/block-dirty-bitmap.c
+++ b/migration/block-dirty-bitmap.c
@@ -1071,18 +1071,15 @@ static int dirty_bitmap_load_header(QEMUFile *f, DBMLoadState *s,
return -EINVAL;
}
- if (!s->cancelled) {
- if (bitmap_alias_map) {
- bitmap_name = g_hash_table_lookup(bitmap_alias_map,
- s->bitmap_alias);
- if (!bitmap_name) {
- error_report("Error: Unknown bitmap alias '%s' on node "
- "'%s' (alias '%s')", s->bitmap_alias,
- s->bs->node_name, s->node_alias);
- cancel_incoming_locked(s);
- }
- } else {
- bitmap_name = s->bitmap_alias;
+ bitmap_name = s->bitmap_alias;
+ if (!s->cancelled && bitmap_alias_map) {
+ bitmap_name = g_hash_table_lookup(bitmap_alias_map,
+ s->bitmap_alias);
+ if (!bitmap_name) {
+ error_report("Error: Unknown bitmap alias '%s' on node "
+ "'%s' (alias '%s')", s->bitmap_alias,
+ s->bs->node_name, s->node_alias);
+ cancel_incoming_locked(s);
}
}
--
2.23.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
2020-10-14 11:44 [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning Chen Qun
@ 2020-10-14 15:56 ` Vladimir Sementsov-Ogievskiy
2020-10-20 7:36 ` Chenqun (kuhn)
0 siblings, 1 reply; 6+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2020-10-14 15:56 UTC (permalink / raw)
To: Chen Qun, qemu-devel, qemu-trivial
Cc: mreitz, stefanha, fam, eblake, jsnow, quintela, dgilbert,
zhang.zhanghailiang, ganqixin, qemu-block, Euler Robot,
Laurent Vivier, Li Qiang
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.robot@huawei.com>
> Signed-off-by: Chen Qun<kuhn.chenqun@huawei.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
2020-10-14 15:56 ` Vladimir Sementsov-Ogievskiy
@ 2020-10-20 7:36 ` Chenqun (kuhn)
2020-10-20 8:16 ` Laurent Vivier
0 siblings, 1 reply; 6+ messages in thread
From: Chenqun (kuhn) @ 2020-10-20 7:36 UTC (permalink / raw)
To: Vladimir Sementsov-Ogievskiy, qemu-devel, qemu-trivial
Cc: fam, ganqixin, Zhanghailiang, qemu-block, quintela, Li Qiang,
dgilbert, mreitz, stefanha, Euler Robot, jsnow, Laurent Vivier
Ping!
Hello,
Maybe this patch, some people have any other suggestions? Or, maybe missed to queue?
Thanks,
Chen Qun
> -----Original Message-----
> From: Vladimir Sementsov-Ogievskiy [mailto:vsementsov@virtuozzo.com]
> Sent: Wednesday, October 14, 2020 11:56 PM
> To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-devel@nongnu.org;
> qemu-trivial@nongnu.org
> Cc: mreitz@redhat.com; stefanha@redhat.com; fam@euphon.net;
> eblake@redhat.com; jsnow@redhat.com; quintela@redhat.com;
> dgilbert@redhat.com; Zhanghailiang <zhang.zhanghailiang@huawei.com>;
> ganqixin <ganqixin@huawei.com>; qemu-block@nongnu.org; Euler Robot
> <euler.robot@huawei.com>; Laurent Vivier <laurent@vivier.eu>; Li Qiang
> <liq3ea@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.robot@huawei.com>
> > Signed-off-by: Chen Qun<kuhn.chenqun@huawei.com>
>
> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>
> --
> Best regards,
> Vladimir
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
2020-10-20 7:36 ` Chenqun (kuhn)
@ 2020-10-20 8:16 ` Laurent Vivier
2020-10-20 10:39 ` Vladimir Sementsov-Ogievskiy
0 siblings, 1 reply; 6+ messages in thread
From: Laurent Vivier @ 2020-10-20 8:16 UTC (permalink / raw)
To: Chenqun (kuhn), Vladimir Sementsov-Ogievskiy, qemu-devel, qemu-trivial
Cc: fam, ganqixin, Zhanghailiang, qemu-block, quintela, Li Qiang,
dgilbert, mreitz, stefanha, Euler Robot, jsnow
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?).
Thanks,
Laurent
> Thanks,
> Chen Qun
>
>> -----Original Message-----
>> From: Vladimir Sementsov-Ogievskiy [mailto:vsementsov@virtuozzo.com]
>> Sent: Wednesday, October 14, 2020 11:56 PM
>> To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-devel@nongnu.org;
>> qemu-trivial@nongnu.org
>> Cc: mreitz@redhat.com; stefanha@redhat.com; fam@euphon.net;
>> eblake@redhat.com; jsnow@redhat.com; quintela@redhat.com;
>> dgilbert@redhat.com; Zhanghailiang <zhang.zhanghailiang@huawei.com>;
>> ganqixin <ganqixin@huawei.com>; qemu-block@nongnu.org; Euler Robot
>> <euler.robot@huawei.com>; Laurent Vivier <laurent@vivier.eu>; Li Qiang
>> <liq3ea@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.robot@huawei.com>
>>> Signed-off-by: Chen Qun<kuhn.chenqun@huawei.com>
>>
>> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>>
>> --
>> Best regards,
>> Vladimir
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
2020-10-20 8:16 ` Laurent Vivier
@ 2020-10-20 10:39 ` Vladimir Sementsov-Ogievskiy
2020-10-20 13:23 ` Eric Blake
0 siblings, 1 reply; 6+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2020-10-20 10:39 UTC (permalink / raw)
To: Laurent Vivier, Chenqun (kuhn), qemu-devel, qemu-trivial
Cc: mreitz, stefanha, fam, eblake, jsnow, quintela, dgilbert,
Zhanghailiang, ganqixin, qemu-block, Euler Robot, Li Qiang
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:vsementsov@virtuozzo.com]
>>> Sent: Wednesday, October 14, 2020 11:56 PM
>>> To: Chenqun (kuhn) <kuhn.chenqun@huawei.com>; qemu-devel@nongnu.org;
>>> qemu-trivial@nongnu.org
>>> Cc: mreitz@redhat.com; stefanha@redhat.com; fam@euphon.net;
>>> eblake@redhat.com; jsnow@redhat.com; quintela@redhat.com;
>>> dgilbert@redhat.com; Zhanghailiang <zhang.zhanghailiang@huawei.com>;
>>> ganqixin <ganqixin@huawei.com>; qemu-block@nongnu.org; Euler Robot
>>> <euler.robot@huawei.com>; Laurent Vivier <laurent@vivier.eu>; Li Qiang
>>> <liq3ea@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.robot@huawei.com>
>>>> Signed-off-by: Chen Qun<kuhn.chenqun@huawei.com>
>>>
>>> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
>>>
>>> --
>>> Best regards,
>>> Vladimir
>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning
2020-10-20 10:39 ` Vladimir Sementsov-Ogievskiy
@ 2020-10-20 13:23 ` Eric Blake
0 siblings, 0 replies; 6+ messages in thread
From: Eric Blake @ 2020-10-20 13:23 UTC (permalink / raw)
To: Vladimir Sementsov-Ogievskiy, Laurent Vivier, Chenqun (kuhn),
qemu-devel, qemu-trivial
Cc: fam, ganqixin, Zhanghailiang, qemu-block, quintela, Li Qiang,
dgilbert, mreitz, stefanha, Euler Robot, jsnow
[-- Attachment #1.1.1: Type: text/plain, Size: 755 bytes --]
On 10/20/20 5:39 AM, Vladimir Sementsov-Ogievskiy wrote:
> 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.
Yes, I'm planning on queuing this.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
[-- Attachment #1.1.2: OpenPGP_0xA7A16B4A2527436A_and_old_rev.asc --]
[-- Type: application/pgp-keys, Size: 103475 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-10-20 13:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-14 11:44 [PATCH v3] migration/block-dirty-bitmap: fix uninitialized variable warning Chen Qun
2020-10-14 15:56 ` Vladimir Sementsov-Ogievskiy
2020-10-20 7:36 ` Chenqun (kuhn)
2020-10-20 8:16 ` Laurent Vivier
2020-10-20 10:39 ` Vladimir Sementsov-Ogievskiy
2020-10-20 13:23 ` Eric Blake
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).