aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCao jin <caoj.fnst@cn.fujitsu.com>2016-01-25 20:16:03 +0800
committerStefano Stabellini <sstabellini@kernel.org>2016-08-12 16:38:18 -0700
commitc4f68f0b52f4d84658ee977eb7e44a80857e0b50 (patch)
treeec79dccb0cec46a0b7fb1195b7ad523d89a5d3dd
parent28b874429ba16e71e0caa46453f3a3e31efb3c51 (diff)
downloadqemu-c4f68f0b52f4d84658ee977eb7e44a80857e0b50.zip
qemu-c4f68f0b52f4d84658ee977eb7e44a80857e0b50.tar.gz
qemu-c4f68f0b52f4d84658ee977eb7e44a80857e0b50.tar.bz2
Xen: fix converity warning of xen_pt_config_init()
emu_regs is a pointer, ARRAY_SIZE doesn't return what we expect. Since the remaining message is enough for debugging, so just remove it. Also tweaked the message a little. Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com> Signed-off-by: Stefano Stabellini <sstabellini@kernel.org> Acked-by: Stefano Stabellini <sstabellini@kernel.org>
-rw-r--r--hw/xen/xen_pt_config_init.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/hw/xen/xen_pt_config_init.c b/hw/xen/xen_pt_config_init.c
index 9869ffd..6f18366 100644
--- a/hw/xen/xen_pt_config_init.c
+++ b/hw/xen/xen_pt_config_init.c
@@ -2049,9 +2049,8 @@ void xen_pt_config_init(XenPCIPassthroughState *s, Error **errp)
for (j = 0; regs->size != 0; j++, regs++) {
xen_pt_config_reg_init(s, reg_grp_entry, regs, &err);
if (err) {
- error_append_hint(&err, "Failed to initialize %d/%zu"
- " reg 0x%x in grp_type = 0x%x (%d/%zu)",
- j, ARRAY_SIZE(xen_pt_emu_reg_grps[i].emu_regs),
+ error_append_hint(&err, "Failed to init register %d"
+ " offsets 0x%x in grp_type = 0x%x (%d/%zu)", j,
regs->offset, xen_pt_emu_reg_grps[i].grp_type,
i, ARRAY_SIZE(xen_pt_emu_reg_grps));
error_propagate(errp, err);