49 lines
1.8 KiB
Diff
49 lines
1.8 KiB
Diff
From 09fe29d40b3d1e30e0e921e186a797c4da5ac583 Mon Sep 17 00:00:00 2001
|
|
From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= <berrange@redhat.com>
|
|
Date: Tue, 10 Jun 2025 13:36:57 +0100
|
|
Subject: [PATCH 20/31] hw/net/rtl8139: skip automatic zero-init of large array
|
|
MIME-Version: 1.0
|
|
Content-Type: text/plain; charset=UTF-8
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
RH-Author: Stefan Hajnoczi <stefanha@redhat.com>
|
|
RH-MergeRequest: 461: Solve -ftrivial-auto-var-init performance regression with QEMU_UNINITIALIZED
|
|
RH-Jira: RHEL-99887
|
|
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
RH-Commit: [19/30] 3c072f59b9a283b40327117585d9f01d32ecc081
|
|
|
|
The 'rtl8139_transmit_one' method has a 8k byte array used for
|
|
copying data between guest and host. Skip the automatic zero-init
|
|
of this array to eliminate the performance overhead in the I/O
|
|
hot path.
|
|
|
|
The 'txbuffer' will be fully initialized when reading PCI DMA
|
|
buffers.
|
|
|
|
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
|
|
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
Message-id: 20250610123709.835102-20-berrange@redhat.com
|
|
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
(cherry picked from commit 3ccc6489dd4925ddd1f3066bd3751389169cd7aa)
|
|
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
---
|
|
hw/net/rtl8139.c | 2 +-
|
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
|
diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c
|
|
index f2fe057535..a2732bf1c1 100644
|
|
--- a/hw/net/rtl8139.c
|
|
+++ b/hw/net/rtl8139.c
|
|
@@ -1818,7 +1818,7 @@ static int rtl8139_transmit_one(RTL8139State *s, int descriptor)
|
|
|
|
PCIDevice *d = PCI_DEVICE(s);
|
|
int txsize = s->TxStatus[descriptor] & 0x1fff;
|
|
- uint8_t txbuffer[0x2000];
|
|
+ QEMU_UNINITIALIZED uint8_t txbuffer[0x2000];
|
|
|
|
DPRINTF("+++ transmit reading %d bytes from host memory at 0x%08x\n",
|
|
txsize, s->TxAddr[descriptor]);
|
|
--
|
|
2.39.3
|
|
|