dd95ba8937
- kvm-net-Fix-announce_self.patch [RHEL-73891] - kvm-migration-Add-helper-to-get-target-runstate.patch [RHEL-54296 RHEL-78397] - kvm-qmp-cont-Only-activate-disks-if-migration-completed.patch [RHEL-54296 RHEL-78397] - kvm-migration-block-Make-late-block-active-the-default.patch [RHEL-54296 RHEL-78397] - kvm-migration-block-Apply-late-block-active-behavior-to-.patch [RHEL-54296 RHEL-78397] - kvm-migration-block-Fix-possible-race-with-block_inactiv.patch [RHEL-54296 RHEL-78397] - kvm-migration-block-Rewrite-disk-activation.patch [RHEL-54296 RHEL-78397] - kvm-block-Add-active-field-to-BlockDeviceInfo.patch [RHEL-54296 RHEL-78397] - kvm-block-Allow-inactivating-already-inactive-nodes.patch [RHEL-54296 RHEL-78397] - kvm-block-Inactivate-external-snapshot-overlays-when-nec.patch [RHEL-54296 RHEL-78397] - kvm-migration-block-active-Remove-global-active-flag.patch [RHEL-54296 RHEL-78397] - kvm-block-Don-t-attach-inactive-child-to-active-node.patch [RHEL-54296 RHEL-78397] - kvm-block-Fix-crash-on-block_resize-on-inactive-node.patch [RHEL-54296 RHEL-78397] - kvm-block-Add-option-to-create-inactive-nodes.patch [RHEL-54296 RHEL-78397] - kvm-block-Add-blockdev-set-active-QMP-command.patch [RHEL-54296 RHEL-78397] - kvm-block-Support-inactive-nodes-in-blk_insert_bs.patch [RHEL-54296 RHEL-78397] - kvm-block-export-Don-t-ignore-image-activation-error-in-.patch [RHEL-54296 RHEL-78397] - kvm-block-Drain-nodes-before-inactivating-them.patch [RHEL-54296 RHEL-78397] - kvm-block-export-Add-option-to-allow-export-of-inactive-.patch [RHEL-54296 RHEL-78397] - kvm-nbd-server-Support-inactive-nodes.patch [RHEL-54296 RHEL-78397] - kvm-iotests-Add-filter_qtest.patch [RHEL-54296 RHEL-78397] - kvm-iotests-Add-qsd-migrate-case.patch [RHEL-54296 RHEL-78397] - kvm-iotests-Add-NBD-based-tests-for-inactive-nodes.patch [RHEL-54296 RHEL-78397] - Resolves: RHEL-73891 (No RARP packets on the destination after migration [rhel-9.6]) - Resolves: RHEL-54296 (Provide QMP command for block device reactivation after migration [rhel-9.5]) - Resolves: RHEL-78397 (backport fix for double migration of a paused VM (disk activation rewrite))
83 lines
2.5 KiB
Diff
83 lines
2.5 KiB
Diff
From ba42b9bf3f193bbc7f47d494bdc888e881539f4b Mon Sep 17 00:00:00 2001
|
|
From: Laurent Vivier <lvivier@redhat.com>
|
|
Date: Fri, 17 Jan 2025 12:17:08 +0100
|
|
Subject: [PATCH 01/23] net: Fix announce_self
|
|
MIME-Version: 1.0
|
|
Content-Type: text/plain; charset=UTF-8
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
RH-Author: Laurent Vivier <lvivier@redhat.com>
|
|
RH-MergeRequest: 338: net: Fix announce_self
|
|
RH-Jira: RHEL-73891
|
|
RH-Acked-by: Eugenio Pérez <eperezma@redhat.com>
|
|
RH-Acked-by: Cindy Lu <lulu@redhat.com>
|
|
RH-Commit: [1/1] dfee696c1c444af0ba2b2d3d8c7012385e84885c (lvivier/qemu-kvm-centos)
|
|
|
|
JIRA: https://issues.redhat.com/browse/RHEL-73891
|
|
|
|
b9ad513e1876 ("net: Remove receive_raw()") adds an iovec entry
|
|
in qemu_deliver_packet_iov() to add the virtio-net header
|
|
in the data when QEMU_NET_PACKET_FLAG_RAW is set but forgets
|
|
to increase the number of iovec entries in the array, so
|
|
receive_iov() will only send the first entry (the virtio-net
|
|
entry, full of 0) and no data. The packet will be discarded.
|
|
|
|
The only user of QEMU_NET_PACKET_FLAG_RAW is announce_self.
|
|
|
|
We can see the problem with tcpdump:
|
|
|
|
- QEMU parameters:
|
|
|
|
.. -monitor stdio \
|
|
-netdev bridge,id=netdev0,br=virbr0 \
|
|
-device virtio-net,mac=9a:2b:2c:2d:2e:2f,netdev=netdev0 \
|
|
|
|
- HMP command:
|
|
|
|
(qemu) announce_self
|
|
|
|
- TCP dump:
|
|
|
|
$ sudo tcpdump -nxi virbr0
|
|
|
|
without the fix:
|
|
|
|
<nothing>
|
|
|
|
with the fix:
|
|
|
|
ARP, Reverse Request who-is 9a:2b:2c:2d:2e:2f tell 9a:2b:2c:2d:2e:2f, length 46
|
|
0x0000: 0001 0800 0604 0003 9a2b 2c2d 2e2f 0000
|
|
0x0010: 0000 9a2b 2c2d 2e2f 0000 0000 0000 0000
|
|
0x0020: 0000 0000 0000 0000 0000 0000 0000
|
|
|
|
Reported-by: Xiaohui Li <xiaohli@redhat.com>
|
|
Bug: https://issues.redhat.com/browse/RHEL-73891
|
|
Fixes: b9ad513e1876 ("net: Remove receive_raw()")
|
|
Cc: akihiko.odaki@daynix.com
|
|
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
|
|
Reviewed-by: Akihiko Odaki <akihiko.odaki@daynix.com>
|
|
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
|
|
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
|
|
(cherry picked from commit 84dfdcbff33fff185528501be408c25c44499f32)
|
|
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
|
|
---
|
|
net/net.c | 1 +
|
|
1 file changed, 1 insertion(+)
|
|
|
|
diff --git a/net/net.c b/net/net.c
|
|
index fc1125111c..94f51b6e5f 100644
|
|
--- a/net/net.c
|
|
+++ b/net/net.c
|
|
@@ -828,6 +828,7 @@ static ssize_t qemu_deliver_packet_iov(NetClientState *sender,
|
|
iov_copy[0].iov_len = nc->vnet_hdr_len;
|
|
memcpy(&iov_copy[1], iov, iovcnt * sizeof(*iov));
|
|
iov = iov_copy;
|
|
+ iovcnt++;
|
|
}
|
|
|
|
if (nc->info->receive_iov) {
|
|
--
|
|
2.48.1
|
|
|