blob: 2fa1342e8a5e561664092b74cbc2acf4c02496d1 (
plain) (
blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
|
id = 2976
title = "attach-ns doesn't work correctly in SR-IOV, cannot attach NS to VF if it is attached to a PF"
state = "opened"
created_at = "2025-05-20T00:28:55.688Z"
closed_at = "n/a"
labels = []
url = "https://gitlab.com/qemu-project/qemu/-/issues/2976"
host-os = "n/a"
host-arch = "n/a"
qemu-version = "n/a"
guest-os = "n/a"
guest-arch = "n/a"
description = """We can't attach namespace to a VF (Secondary controller) unless it is not attached to a primary controller first
Lately in the commit https://github.com/qemu/qemu/commit/6ccca4b6bb9f994cc04e71004e1767a3476d2b23 the file qemu/hw/nvme/ctrl.c got changed -\\> in function "nvme_ns_attachment" -\\> line 6819 (At the time I'm writing the bug) which is the condation in attach ns to check if the namespace is attached "`if (nvme_ns(n, nsid)) {`"
This change will always result in checking the namespace attach to the PF even if we are trying to attach it to the VF."""
reproduce = """1. Enable a VF:
```
echo 1 > /sys/bus/pci/devices/0000:01:00.0/reset
sleep 1
echo 2 > /sys/bus/pci/devices/0000:01:00.0/sriov_numvfs
sleep 1
nvme virt-mgmt /dev/nvme0 -c 1 -r 1 -a 8 -n 1
nvme virt-mgmt /dev/nvme0 -c 1 -r 0 -a 8 -n 2
nvme virt-mgmt /dev/nvme0 -c 1 -a 9
sleep 1
```
2. attach namespace 1 to the PF (e.g. `vme attach-ns /dev/nvme0 -n1 -c0` )
3. try to attach it using the nvme_cli command from the PF (e.g. `nvme attach-ns /dev/nvme0 -n1 -c1`)"""
additional = "n/a"
|