From nobody Fri May 17 12:14:37 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1598119871; cv=none; d=zohomail.com; s=zohoarc; b=Ji17NqjmiO384p+ARRFOy1BMG5g/9atPSXpkZvW8lDfhx1iA5PTtnLIvjRX/j3m01u4IbUKdyESypPADVZbfF3I46sr3GLweiUBqSsJ9LEGl01xP/BvNnQ2Mkk6qcjK9Xlq37tjOwJmnovGV/uHfULM4pYFg+S+d6NZgBNLARpg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1598119871; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=x+8JWtN3sZgMYgY5lTuOwjZoF/npzERcdc45xzdTzrg=; b=RXJuabxQKyW1RxtGAWIliqO1kTl2MPGo/kiLuAygKiPpHSRocjyfSFLsM70PNd9UffP2285FRSggvaxcW7omdLu6NKjiO0WW9YMGVDDtT/lMu/vrpBLQrcBzYdhKtsdkWW7pN4QW/Zyru3omDWGns/zrYCozW8IMsKE+Y9awdEI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1598119871347285.5886447092189; Sat, 22 Aug 2020 11:11:11 -0700 (PDT) Received: from localhost ([::1]:59906 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1k9Xyv-00083V-T4 for importer@patchew.org; Sat, 22 Aug 2020 14:11:09 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47428) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1k9Xxo-0006Jr-1P; Sat, 22 Aug 2020 14:10:00 -0400 Received: from mail-wr1-x444.google.com ([2a00:1450:4864:20::444]:44547) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1k9Xxm-0007bC-DN; Sat, 22 Aug 2020 14:09:59 -0400 Received: by mail-wr1-x444.google.com with SMTP id c15so4788090wrs.11; Sat, 22 Aug 2020 11:09:56 -0700 (PDT) Received: from localhost.localdomain (121.red-81-40-121.staticip.rima-tde.net. [81.40.121.121]) by smtp.gmail.com with ESMTPSA id 126sm13713876wme.42.2020.08.22.11.09.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 22 Aug 2020 11:09:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=x+8JWtN3sZgMYgY5lTuOwjZoF/npzERcdc45xzdTzrg=; b=uH07j2BNVo6I8qEQK6TibF1OJeeKpre4MprFO3ipTxbUJi3/qBAEKFZF3si+6wuI3e ybT1EPS27JV/Tjt/Vap2iXaypCF6tXt4LYHgFYKUuA+tUp1mWmcO4nEceRsEx9O6kykS z9A7w48KmtD1lClRHKPe/n6qAaqGhBKaQw0DNV+JCyfPdki5Bv+M4gEzwB4FzlItZVNz 7Epz9n8oIQlVMgA42qrlauiEmYjN/3pdE1aBzTReLoNjeTGzq/4ujgxwHsC/YqgMhnxi sAwR6j+yxi6ZuL89pRPVBsukB+9Sl2dkR951Wm/6YArJD4X0UeyRoSNnX7K2bp8YVvI/ 9ZEA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; bh=x+8JWtN3sZgMYgY5lTuOwjZoF/npzERcdc45xzdTzrg=; b=GQUdU1YMTHvai6DWWkWu3+mMdGeOVkBo8IPDPf5yvYnlvPDy6tT4CXKLjvKlch11Pe efMCdna9SeoHou7ujrv3d+0X8ktYZQ9IMc/pr2O9FaUINFKg27NZENIP+v6ogjikl2f7 jfGOyUuvv0O+cok+TFdO/W2BJgrdRXUgTTlW/LN6r2Su7ntClp4da8QczGYqf/4z3xe6 PD3buOEmvAyhdsqOMa9/P0xlTH56KLB4xBgiqkCOKqOWkeZjgouUA8ZARlKmohL0B8su DsFcOo4C44jpwYF7GDdpCJEP/jyQtPsXHVV3sxjbd77A9WnchuiMbCvg8wtUxMRFWVGZ vh4w== X-Gm-Message-State: AOAM531It/Qhtgm0OMX73ETabJau5AM9yIpLcL83l1I3xANq3FYeGt+A Ha3bJXERZhz755ya9lMAzQaIJuoHWYQ= X-Google-Smtp-Source: ABdhPJwvbjLS0UM1zzO7gqKcBghK3m1OZV6oMEQfa7Dbub1UNPxehDcbX60KVvvijD5JA/TMWBuVgw== X-Received: by 2002:a5d:480b:: with SMTP id l11mr7537457wrq.85.1598119795372; Sat, 22 Aug 2020 11:09:55 -0700 (PDT) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PATCH v2 1/2] util/hexdump: Convert to take a void pointer argument Date: Sat, 22 Aug 2020 20:09:49 +0200 Message-Id: <20200822180950.1343963-2-f4bug@amsat.org> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200822180950.1343963-1-f4bug@amsat.org> References: <20200822180950.1343963-1-f4bug@amsat.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2a00:1450:4864:20::444; envelope-from=philippe.mathieu.daude@gmail.com; helo=mail-wr1-x444.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: 0 X-Spam_score: 0.0 X-Spam_bar: / X-Spam_report: (0.0 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FORGED_FROMDOMAIN=1, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , qemu-block@nongnu.org, Li Zhijian , qemu-trivial@nongnu.org, Jason Wang , Alistair Francis , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Zhang Chen , qemu-arm@nongnu.org, qemu-ppc@nongnu.org, Gerd Hoffmann , "Edgar E. Iglesias" , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Most uses of qemu_hexdump() do not take an array of char as input, forcing use of cast. Since we can use this helper to dump any kind of buffer, use a pointer to void argument instead. Signed-off-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Alistair Francis Reviewed-by: Edgar E. Iglesias Reviewed-by: Li Qiang Reviewed-by: Stefano Garzarella --- Since v1: - renamed argument 'bufptr' (Peter Maydell) --- include/qemu-common.h | 3 ++- hw/dma/xlnx_dpdma.c | 2 +- hw/net/fsl_etsec/etsec.c | 2 +- hw/sd/sd.c | 2 +- hw/usb/redirect.c | 2 +- net/colo-compare.c | 12 ++++++------ net/net.c | 2 +- util/hexdump.c | 4 +++- 8 files changed, 16 insertions(+), 13 deletions(-) diff --git a/include/qemu-common.h b/include/qemu-common.h index bb9496bd80f..6834883822f 100644 --- a/include/qemu-common.h +++ b/include/qemu-common.h @@ -138,7 +138,8 @@ int os_parse_cmd_args(int index, const char *optarg); * Hexdump a buffer to a file. An optional string prefix is added to every= line */ =20 -void qemu_hexdump(const char *buf, FILE *fp, const char *prefix, size_t si= ze); +void qemu_hexdump(const void *bufptr, FILE *fp, + const char *prefix, size_t size); =20 /* * helper to parse debug environment variables diff --git a/hw/dma/xlnx_dpdma.c b/hw/dma/xlnx_dpdma.c index b40c897de2c..d75a8069426 100644 --- a/hw/dma/xlnx_dpdma.c +++ b/hw/dma/xlnx_dpdma.c @@ -388,7 +388,7 @@ static void xlnx_dpdma_dump_descriptor(DPDMADescriptor = *desc) { if (DEBUG_DPDMA) { qemu_log("DUMP DESCRIPTOR:\n"); - qemu_hexdump((char *)desc, stdout, "", sizeof(DPDMADescriptor)); + qemu_hexdump(desc, stdout, "", sizeof(DPDMADescriptor)); } } =20 diff --git a/hw/net/fsl_etsec/etsec.c b/hw/net/fsl_etsec/etsec.c index 7035cf4eb97..c817a28decd 100644 --- a/hw/net/fsl_etsec/etsec.c +++ b/hw/net/fsl_etsec/etsec.c @@ -357,7 +357,7 @@ static ssize_t etsec_receive(NetClientState *nc, =20 #if defined(HEX_DUMP) fprintf(stderr, "%s receive size:%zd\n", nc->name, size); - qemu_hexdump((void *)buf, stderr, "", size); + qemu_hexdump(buf, stderr, "", size); #endif /* Flush is unnecessary as are already in receiving path */ etsec->need_flush =3D false; diff --git a/hw/sd/sd.c b/hw/sd/sd.c index fad9cf1ee7a..190e4cf1232 100644 --- a/hw/sd/sd.c +++ b/hw/sd/sd.c @@ -1781,7 +1781,7 @@ send_response: } =20 #ifdef DEBUG_SD - qemu_hexdump((const char *)response, stderr, "Response", rsplen); + qemu_hexdump(response, stderr, "Response", rsplen); #endif =20 return rsplen; diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c index 417a60a2e68..09edb0d81c0 100644 --- a/hw/usb/redirect.c +++ b/hw/usb/redirect.c @@ -240,7 +240,7 @@ static void usbredir_log_data(USBRedirDevice *dev, cons= t char *desc, if (dev->debug < usbredirparser_debug_data) { return; } - qemu_hexdump((char *)data, stderr, desc, len); + qemu_hexdump(data, stderr, desc, len); } =20 /* diff --git a/net/colo-compare.c b/net/colo-compare.c index 2c20de1537d..550272b3baa 100644 --- a/net/colo-compare.c +++ b/net/colo-compare.c @@ -494,9 +494,9 @@ sec: g_queue_push_head(&conn->secondary_list, spkt); =20 if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump((char *)ppkt->data, stderr, + qemu_hexdump(ppkt->data, stderr, "colo-compare ppkt", ppkt->size); - qemu_hexdump((char *)spkt->data, stderr, + qemu_hexdump(spkt->data, stderr, "colo-compare spkt", spkt->size); } =20 @@ -535,9 +535,9 @@ static int colo_packet_compare_udp(Packet *spkt, Packet= *ppkt) trace_colo_compare_udp_miscompare("primary pkt size", ppkt->size); trace_colo_compare_udp_miscompare("Secondary pkt size", spkt->size= ); if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump((char *)ppkt->data, stderr, "colo-compare pri pkt= ", + qemu_hexdump(ppkt->data, stderr, "colo-compare pri pkt", ppkt->size); - qemu_hexdump((char *)spkt->data, stderr, "colo-compare sec pkt= ", + qemu_hexdump(spkt->data, stderr, "colo-compare sec pkt", spkt->size); } return -1; @@ -578,9 +578,9 @@ static int colo_packet_compare_icmp(Packet *spkt, Packe= t *ppkt) trace_colo_compare_icmp_miscompare("Secondary pkt size", spkt->size); if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump((char *)ppkt->data, stderr, "colo-compare pri pkt= ", + qemu_hexdump(ppkt->data, stderr, "colo-compare pri pkt", ppkt->size); - qemu_hexdump((char *)spkt->data, stderr, "colo-compare sec pkt= ", + qemu_hexdump(spkt->data, stderr, "colo-compare sec pkt", spkt->size); } return -1; diff --git a/net/net.c b/net/net.c index bbaedb3c7a6..f3e5d533fd7 100644 --- a/net/net.c +++ b/net/net.c @@ -636,7 +636,7 @@ static ssize_t qemu_send_packet_async_with_flags(NetCli= entState *sender, =20 #ifdef DEBUG_NET printf("qemu_send_packet_async:\n"); - qemu_hexdump((const char *)buf, stdout, "net", size); + qemu_hexdump(buf, stdout, "net", size); #endif =20 if (sender->link_down || !sender->peer) { diff --git a/util/hexdump.c b/util/hexdump.c index f879ff0ad6b..053f05d4160 100644 --- a/util/hexdump.c +++ b/util/hexdump.c @@ -16,8 +16,10 @@ #include "qemu/osdep.h" #include "qemu-common.h" =20 -void qemu_hexdump(const char *buf, FILE *fp, const char *prefix, size_t si= ze) +void qemu_hexdump(const void *bufptr, FILE *fp, + const char *prefix, size_t size) { + const char *buf =3D bufptr; unsigned int b, len, i, c; =20 for (b =3D 0; b < size; b +=3D 16) { --=20 2.26.2 From nobody Fri May 17 12:14:37 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1598119974; cv=none; d=zohomail.com; s=zohoarc; b=MVD9cVewYAcqMEjlFyb9qBcHUcsOAVpIjrzkgnHSmsuFyEW86F1AQQQfjxhVMckYE7h02A8/U/kC+/jTSXRk90vz3s2lhZ6R6FVhVvvaWWbMAUToiry90HHv6dF71Z1SEjpIiY/39gksK5tsGDFJiZ77mnv3Q++mbsMQR9xNWFE= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1598119974; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=8sK4imuwnDEg9EWUUe6vAsAHrNHBEVzHJOZxgM7BTmw=; b=HtX8yIXYto5a8UM2KEupFjY/Tf0+JRKwPTKMCCt48xS/XTI6RHbF5rrBz6XxGbwVohYQVjamd2/hIfAf5Xhh8Z/Qb8ai1bwhwUooBukEc2hIg9Jth9vn8vzsBMl+3/okVnquh0MOWun0fIidBRH2WLyLUn5gXCn7UX0wHpk5dus= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1598119974020769.9985184641455; Sat, 22 Aug 2020 11:12:54 -0700 (PDT) Received: from localhost ([::1]:36532 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1k9Y0a-0001ZT-Pi for importer@patchew.org; Sat, 22 Aug 2020 14:12:52 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47446) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1k9Xxq-0006PF-9G; Sat, 22 Aug 2020 14:10:02 -0400 Received: from mail-wm1-x344.google.com ([2a00:1450:4864:20::344]:34959) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1k9Xxo-0007bg-6X; Sat, 22 Aug 2020 14:10:01 -0400 Received: by mail-wm1-x344.google.com with SMTP id o21so4684140wmc.0; Sat, 22 Aug 2020 11:09:58 -0700 (PDT) Received: from localhost.localdomain (121.red-81-40-121.staticip.rima-tde.net. [81.40.121.121]) by smtp.gmail.com with ESMTPSA id 126sm13713876wme.42.2020.08.22.11.09.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 22 Aug 2020 11:09:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=8sK4imuwnDEg9EWUUe6vAsAHrNHBEVzHJOZxgM7BTmw=; b=f69LWHTY1qtdmyV4frOoM3Vq10A9SSVr0QT8i0x6HDNCwpnE+RNslUePBzCmlXVQkL WJuxGp8I2mZPPGIs8gyl+aQqHNa6hTV9d1jpY/6BdDLySjHaKsmKUE7juYor9hdyJAtY JCYrz5Lhe8TUCG0fSzX9ttfpSmmN4EteSK++hHhZZ41k8PErXXLlXPx0ENK0t1yySM/0 ChT+8YoonANxQduuWaOV57YotaVjZLEVhByAe7v844cw5pXiDS7urHWn8axjw7CNwrYn 2vdWNthhyIFy4fHP8R4AbiFjwA07mXk8OJVBVzoO0DzQEWIpQatOo98m4ya3fwyKpOJx P+ng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; bh=8sK4imuwnDEg9EWUUe6vAsAHrNHBEVzHJOZxgM7BTmw=; b=iE3qecR84W80/z57Z0Xk/DFP2h6DavVxB86VBF7lW35XLgg7yJSf5AXqKsXkr+h/+B 7NaSd/D6i/kgMiz6BNMsSHKdY7ym2GhiaoxDJNiOx9gfZXyrwdqPYw5kV0oI+vwq6DQw fcL9qsVHckcCUTb3ZhHQ18vLgD3M1UR+J+yTSSc+u2Dkae59BE9Y5MVI61qqNrSRLA9L cSkqZSYX82sCx5WDO094KQhbHExv3Sb1hrhp0E+V16cVQf+TSQP4QNCSwzR0ZbQvFBIZ AodvjlPTAIL3doyprJlZMa32P/FZ1oD6ScRAiArZXgeODmNWh80fNHzn3gT8sXicUt2b XjJQ== X-Gm-Message-State: AOAM530Redwvqr/bXUTbtlBLaaAuMbvVdtB5dfqfPI4okK1dpk2q7Nik qMBz4U8e3EWGWVvFg+XclUFgiH6j0rE= X-Google-Smtp-Source: ABdhPJwgF2+cBqYe2jSlu3NUifAvWEtahVVv/o4J6rY6T+25OiE2lV4b7lDFqSzxVXcmElsiVVbVCQ== X-Received: by 2002:a1c:f416:: with SMTP id z22mr8217984wma.62.1598119797002; Sat, 22 Aug 2020 11:09:57 -0700 (PDT) From: =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Subject: [PATCH v2 2/2] util/hexdump: Reorder qemu_hexdump() arguments Date: Sat, 22 Aug 2020 20:09:50 +0200 Message-Id: <20200822180950.1343963-3-f4bug@amsat.org> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200822180950.1343963-1-f4bug@amsat.org> References: <20200822180950.1343963-1-f4bug@amsat.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2a00:1450:4864:20::344; envelope-from=philippe.mathieu.daude@gmail.com; helo=mail-wm1-x344.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: 0 X-Spam_score: 0.0 X-Spam_bar: / X-Spam_report: (0.0 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FORGED_FROMDOMAIN=1, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Peter Maydell , qemu-block@nongnu.org, Li Zhijian , qemu-trivial@nongnu.org, Jason Wang , Alistair Francis , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , Zhang Chen , qemu-arm@nongnu.org, qemu-ppc@nongnu.org, Gerd Hoffmann , "Edgar E. Iglesias" , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , David Gibson Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) qemu_hexdump()'s pointer to the buffer and length of the buffer are closely related arguments but are widely separated in the argument list order (also, the format of function prototypes is usually to have the FILE* argument coming first). Reorder the arguments as "fp, prefix, buf, size" which is more logical. Suggested-by: Peter Maydell Signed-off-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Edgar E. Iglesias Reviewed-by: Stefano Garzarella --- include/qemu-common.h | 4 ++-- hw/dma/xlnx_dpdma.c | 2 +- hw/net/fsl_etsec/etsec.c | 2 +- hw/net/fsl_etsec/rings.c | 2 +- hw/sd/sd.c | 2 +- hw/usb/redirect.c | 2 +- net/colo-compare.c | 24 ++++++++++++------------ net/net.c | 2 +- util/hexdump.c | 4 ++-- util/iov.c | 2 +- 10 files changed, 23 insertions(+), 23 deletions(-) diff --git a/include/qemu-common.h b/include/qemu-common.h index 6834883822f..9cfd62669bf 100644 --- a/include/qemu-common.h +++ b/include/qemu-common.h @@ -138,8 +138,8 @@ int os_parse_cmd_args(int index, const char *optarg); * Hexdump a buffer to a file. An optional string prefix is added to every= line */ =20 -void qemu_hexdump(const void *bufptr, FILE *fp, - const char *prefix, size_t size); +void qemu_hexdump(FILE *fp, const char *prefix, + const void *bufptr, size_t size); =20 /* * helper to parse debug environment variables diff --git a/hw/dma/xlnx_dpdma.c b/hw/dma/xlnx_dpdma.c index d75a8069426..967548abd31 100644 --- a/hw/dma/xlnx_dpdma.c +++ b/hw/dma/xlnx_dpdma.c @@ -388,7 +388,7 @@ static void xlnx_dpdma_dump_descriptor(DPDMADescriptor = *desc) { if (DEBUG_DPDMA) { qemu_log("DUMP DESCRIPTOR:\n"); - qemu_hexdump(desc, stdout, "", sizeof(DPDMADescriptor)); + qemu_hexdump(stdout, "", desc, sizeof(DPDMADescriptor)); } } =20 diff --git a/hw/net/fsl_etsec/etsec.c b/hw/net/fsl_etsec/etsec.c index c817a28decd..c5edb25dc9f 100644 --- a/hw/net/fsl_etsec/etsec.c +++ b/hw/net/fsl_etsec/etsec.c @@ -357,7 +357,7 @@ static ssize_t etsec_receive(NetClientState *nc, =20 #if defined(HEX_DUMP) fprintf(stderr, "%s receive size:%zd\n", nc->name, size); - qemu_hexdump(buf, stderr, "", size); + qemu_hexdump(stderr, "", buf, size); #endif /* Flush is unnecessary as are already in receiving path */ etsec->need_flush =3D false; diff --git a/hw/net/fsl_etsec/rings.c b/hw/net/fsl_etsec/rings.c index 337a55fc957..628648a9c37 100644 --- a/hw/net/fsl_etsec/rings.c +++ b/hw/net/fsl_etsec/rings.c @@ -269,7 +269,7 @@ static void process_tx_bd(eTSEC *etsec, =20 #if defined(HEX_DUMP) qemu_log("eTSEC Send packet size:%d\n", etsec->tx_buffer_len); - qemu_hexdump(etsec->tx_buffer, stderr, "", etsec->tx_buffer_le= n); + qemu_hexdump(stderr, "", etsec->tx_buffer, etsec->tx_buffer_le= n); #endif /* ETSEC_RING_DEBUG */ =20 if (etsec->first_bd.flags & BD_TX_TOEUN) { diff --git a/hw/sd/sd.c b/hw/sd/sd.c index 190e4cf1232..6508939b1f4 100644 --- a/hw/sd/sd.c +++ b/hw/sd/sd.c @@ -1781,7 +1781,7 @@ send_response: } =20 #ifdef DEBUG_SD - qemu_hexdump(response, stderr, "Response", rsplen); + qemu_hexdump(stderr, "Response", response, rsplen); #endif =20 return rsplen; diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c index 09edb0d81c0..48f38d33912 100644 --- a/hw/usb/redirect.c +++ b/hw/usb/redirect.c @@ -240,7 +240,7 @@ static void usbredir_log_data(USBRedirDevice *dev, cons= t char *desc, if (dev->debug < usbredirparser_debug_data) { return; } - qemu_hexdump(data, stderr, desc, len); + qemu_hexdump(stderr, desc, data, len); } =20 /* diff --git a/net/colo-compare.c b/net/colo-compare.c index 550272b3baa..4a5ed642e9a 100644 --- a/net/colo-compare.c +++ b/net/colo-compare.c @@ -494,10 +494,10 @@ sec: g_queue_push_head(&conn->secondary_list, spkt); =20 if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump(ppkt->data, stderr, - "colo-compare ppkt", ppkt->size); - qemu_hexdump(spkt->data, stderr, - "colo-compare spkt", spkt->size); + qemu_hexdump(stderr, "colo-compare ppkt", + ppkt->data, ppkt->size); + qemu_hexdump(stderr, "colo-compare spkt", + spkt->data, spkt->size); } =20 colo_compare_inconsistency_notify(s); @@ -535,10 +535,10 @@ static int colo_packet_compare_udp(Packet *spkt, Pack= et *ppkt) trace_colo_compare_udp_miscompare("primary pkt size", ppkt->size); trace_colo_compare_udp_miscompare("Secondary pkt size", spkt->size= ); if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump(ppkt->data, stderr, "colo-compare pri pkt", - ppkt->size); - qemu_hexdump(spkt->data, stderr, "colo-compare sec pkt", - spkt->size); + qemu_hexdump(stderr, "colo-compare pri pkt", + ppkt->data, ppkt->size); + qemu_hexdump(stderr, "colo-compare sec pkt", + spkt->data, spkt->size); } return -1; } else { @@ -578,10 +578,10 @@ static int colo_packet_compare_icmp(Packet *spkt, Pac= ket *ppkt) trace_colo_compare_icmp_miscompare("Secondary pkt size", spkt->size); if (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))= { - qemu_hexdump(ppkt->data, stderr, "colo-compare pri pkt", - ppkt->size); - qemu_hexdump(spkt->data, stderr, "colo-compare sec pkt", - spkt->size); + qemu_hexdump(stderr, "colo-compare pri pkt", + ppkt->data, ppkt->size); + qemu_hexdump(stderr, "colo-compare sec pkt", + spkt->data, spkt->size); } return -1; } else { diff --git a/net/net.c b/net/net.c index f3e5d533fd7..7a2a0fb5ac6 100644 --- a/net/net.c +++ b/net/net.c @@ -636,7 +636,7 @@ static ssize_t qemu_send_packet_async_with_flags(NetCli= entState *sender, =20 #ifdef DEBUG_NET printf("qemu_send_packet_async:\n"); - qemu_hexdump(buf, stdout, "net", size); + qemu_hexdump(stdout, "net", buf, size); #endif =20 if (sender->link_down || !sender->peer) { diff --git a/util/hexdump.c b/util/hexdump.c index 053f05d4160..0b4662e701d 100644 --- a/util/hexdump.c +++ b/util/hexdump.c @@ -16,8 +16,8 @@ #include "qemu/osdep.h" #include "qemu-common.h" =20 -void qemu_hexdump(const void *bufptr, FILE *fp, - const char *prefix, size_t size) +void qemu_hexdump(FILE *fp, const char *prefix, + const void *bufptr, size_t size) { const char *buf =3D bufptr; unsigned int b, len, i, c; diff --git a/util/iov.c b/util/iov.c index 45ef3043eec..ae61d696aae 100644 --- a/util/iov.c +++ b/util/iov.c @@ -237,7 +237,7 @@ void iov_hexdump(const struct iovec *iov, const unsigne= d int iov_cnt, size =3D size > limit ? limit : size; buf =3D g_malloc(size); iov_to_buf(iov, iov_cnt, 0, buf, size); - qemu_hexdump(buf, fp, prefix, size); + qemu_hexdump(fp, prefix, buf, size); g_free(buf); } =20 --=20 2.26.2