When using a VDPA device, the following situations are
also acceptable: the hardware MAC address is not 0,
and the MAC address in the QEMU command line is 0.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/hw/net/virtio-net.c b
While the hardware MAC address is 0 and the MAC address in
the QEMU command line is also 0, this configuration is
acceptable.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index
When using a VDPA device, it is important to ensure that the MAC
address is correctly set. The MAC address in the hardware should
match the MAC address from the QEMU command line. This is a recommended
configuration and will allow the system to boot.
Signed-off-by: Cindy Lu
---
hw/net/virtio
o-net-pci,netdev=vhost-vdpa0\
Signed-off-by: Cindy Lu
---
include/net/net.h | 1 +
net/vhost-vdpa.c | 4
qapi/net.json | 5 +
3 files changed, 10 insertions(+)
diff --git a/include/net/net.h b/include/net/net.h
index c8f679761b..e00651a97b 100644
--- a/include/net/net.h
,vhostdev=/dev/vhost-vdpa-0,id=vhost-vdpa0,macstrickcheck=true\
-device virtio-net-pci,netdev=vhost-vdpa0\
tested by ConnectX-6 Dx device
change in v3
1. add a new parameter to enable the check and keep the old behavior
2. adjust the comment and make it more clear
Cindy Lu (4):
vhost_vdpa : Add a
On Wed, 9 Oct 2024 at 16:30, Jason Wang wrote:
>
> On Mon, Sep 30, 2024 at 12:01 AM Cindy Lu wrote:
> >
> > While the hardware MAC address is 0 and the MAC address in
> > the QEMU command line is also 0, this configuration is
> > acceptable.
> >
> > Si
On Wed, 9 Oct 2024 at 16:30, Jason Wang wrote:
>
> On Mon, Sep 30, 2024 at 12:01 AM Cindy Lu wrote:
> >
> > When using a VDPA device, the following situations are
> > also acceptable: the hardware MAC address is not 0,
> > and the MAC address in the QEMU command li
When using a VDPA device, it is important to ensure that the MAC
address is correctly set.
There are only three acceptable situations for MAC setup; any other
configuration will fail to boot.
tested by ConnectX-6 Dx device
Cindy Lu (3):
virtio_net: Add the check for vdpa's mac ad
When using a VDPA device, the following situations are
also acceptable: the hardware MAC address is not 0,
and the MAC address in the QEMU command line is 0.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw/net/virtio-net.c b
While the hardware MAC address is 0 and the MAC address in
the QEMU command line is also 0, this configuration is
acceptable.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index
When using a VDPA device, it is important to ensure that the MAC
address is correctly set. The MAC address in the hardware should
match the MAC address from the QEMU command line. This is a recommended
configuration and will allow the system to boot.
Signed-off-by: Cindy Lu
---
hw/net/virtio
On Tue, 6 Aug 2024 at 21:30, Michael S. Tsirkin wrote:
>
> On Tue, Aug 06, 2024 at 08:58:01AM +0800, Cindy Lu wrote:
> > When using a VDPA device, it is important to ensure that
> > the MAC address in the hardware matches the MAC address
> > from the QEMU command line.
On Wed, 7 Aug 2024 at 10:36, Jason Wang wrote:
> On Tue, Aug 6, 2024 at 5:44 PM Cindy Lu wrote:
> >
> > On Tue, 6 Aug 2024 at 11:07, Jason Wang wrote:
> > >
> > > On Tue, Aug 6, 2024 at 8:58 AM Cindy Lu wrote:
> > > >
> > > >
x27;t
have this kind of verification. I will double check this
Thanks
Cindy
>
> On Tue, Aug 6, 2024 at 12:45 PM Cindy Lu wrote:
> >
> > On Tue, 6 Aug 2024 at 11:07, Jason Wang wrote:
> > >
> > > On Tue, Aug 6, 2024 at 8:58 AM Cindy Lu wrote:
> > > &
On Tue, 6 Aug 2024 at 11:09, Jason Wang wrote:
>
> On Tue, Aug 6, 2024 at 8:58 AM Cindy Lu wrote:
> >
> > The vdpa device with MAC address 0 should not boot.
> > So remove the check here
> >
> > Signed-off-by: Cindy Lu
>
> Please describe what issue
On Tue, 6 Aug 2024 at 11:07, Jason Wang wrote:
>
> On Tue, Aug 6, 2024 at 8:58 AM Cindy Lu wrote:
> >
> > When using a VDPA device, it is important to ensure that
> > the MAC address in the hardware matches the MAC address
> > from the QEMU command line.
> >
fix use of a released vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index 592fdaa10f..dc31a37ec0 100644
--- a/hw/virtio/virtio-pci.c
+++ b/hw/virtio/v
When using a VDPA device, it is important to ensure that
the MAC address in the hardware matches the MAC address
from the QEMU command line.
This will allow the device to boot.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 33 +
1 file changed, 29 insertions
The vdpa device with MAC address 0 should not boot.
So remove the check here
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 13 -
1 file changed, 13 deletions(-)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index c144ae2e78..8a7c743ad3 100644
--- a/hw/net/virtio-net.c
When using a VDPA device, this is another acceptable situations
The hardware MAC address is not 0, and the MAC address in the QEMU command line
is 0.
This is also acceptable
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 13 -
1 file changed, 12 insertions(+), 1 deletion
On Mon, 22 Jul 2024 at 15:24, Jason Wang wrote:
>
> Hi Cindy
>
> On Fri, Jul 19, 2024 at 1:25 PM Cindy Lu wrote:
> >
> > The crash was reported in MAC OS and NixOS, here is the link for this
bug
> > https://gitlab.com/qemu-project/qemu/-/issues/2334
> >
cff94d ("virtio-pci: fix use of a released vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index 592fdaa10f..dc31a37ec0 100644
--- a/hw/virtio/vir
On Tue, 2 Jul 2024 at 15:09, Jason Wang wrote:
>
> On Tue, Jul 2, 2024 at 10:00 AM Cindy Lu wrote:
> >
> > The crash was reported in MAC OS and NixOS, here is the link for this bug
> > https://gitlab.com/qemu-project/qemu/-/issues/2334
> > https://gitlab.com/q
On Tue, 16 Jul 2024 at 13:37, Jason Wang wrote:
>
> On Tue, Jul 16, 2024 at 9:14 AM Cindy Lu wrote:
> >
> > When using a VDPA device, it is important to ensure that the MAC address
> > in the hardware matches the MAC address from the QEMU command line.
> >
>
On Tue, 16 Jul 2024 at 09:56, Lei Yang wrote:
>
> Hi Cindy
>
> If needed, QE can help test this MR before merging into the master branch.
>
> Best Regards
> Lei
>
sure， Really thanks for your help
thanks
cindy
>
> On Tue, Jul 16, 2024 at 9:14 AM Cindy Lu wrote:
&g
addresses are not 0.
2. The hardware MAC address is not 0, and the MAC address in the QEMU command
line is 0.
In this situation, the hardware MAC address will overwrite the QEMU command
line address.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 43 +--
1
On Tue, 9 Jul 2024 at 10:47, Jason Wang wrote:
>
> On Tue, Jul 9, 2024 at 10:41 AM Cindy Lu wrote:
> >
> > On Tue, 9 Jul 2024 at 10:33, Jason Wang wrote:
> > >
> > > On Tue, Jul 9, 2024 at 10:27 AM Cindy Lu wrote:
> > > >
> > > > When
On Tue, 9 Jul 2024 at 10:33, Jason Wang wrote:
>
> On Tue, Jul 9, 2024 at 10:27 AM Cindy Lu wrote:
> >
> > When using VDPA device, we should verify whether the MAC address in the
> > hardware matches the MAC address from the QEMU command line. If not,
> > we w
When using VDPA device, we should verify whether the MAC address in the
hardware matches the MAC address from the QEMU command line. If not,
we will need to update the related information.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c | 15 +++
1 file changed, 11 insertions(+), 4
Add the return value check for kvm_virtio_pci_vector_use_one().
Since the return value of function virtio_pci_set_vector() is void,
we can do nothing here. So just add the error message here.
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 4 +++-
1 file changed, 3 insertions(+), 1
check for kvm_virtio_pci_vector_use_one().
Since the return value of function virtio_pci_set_vector() is void,
just add the error message here.
Cindy Lu (2):
virtio-pci: Fix the use of an uninitialized irqfd
virtio-pci: Add check the return of kvm_virtio_pci_vector_use_one
hw/virtio/vir
t
../qemu-9.0.0/util/qemu-thread-posix.c:541
13 0x7c8185bcd1cf in ??? () at /usr/lib/libc.so.6
14 0x7c8185c4e504 in clone () at /usr/lib/libc.so.6
Fixes: 2ce6cff94d ("virtio-pci: fix use of a released vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/vir
On Thu, Jun 27, 2024 at 4:55 PM Michael S. Tsirkin wrote:
>
> On Thu, Jun 27, 2024 at 04:40:33PM +0800, Cindy Lu wrote:
> > On Wed, Jun 26, 2024 at 3:45 PM Michael S. Tsirkin wrote:
> > >
> > > On Wed, Jun 26, 2024 at 10:44:31AM +0800, Cindy Lu wrote:
> > >
On Wed, Jun 26, 2024 at 3:45 PM Michael S. Tsirkin wrote:
>
> On Wed, Jun 26, 2024 at 10:44:31AM +0800, Cindy Lu wrote:
> > The crash was reported in MAC OS and NixOS, here is the link for this bug
> > https://gitlab.com/qemu-project/qemu/-/issues/2334
> > https://gitl
) at /usr/lib/libc.so.6
14 0x7c8185c4e504 in clone () at /usr/lib/libc.so.6
Fixes: 2ce6cff94d ("virtio-pci: fix use of a released vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git
On Mon, Jun 17, 2024 at 6:38 PM Philippe Mathieu-Daudé
wrote:
>
> Hi Cindy,
>
> On 17/6/24 11:55, Cindy Lu wrote:
> > The crash was reported in MAC OS and NixOS, here is the link for this bug
> > https://gitlab.com/qemu-project/qemu/-/issues/2334
> > https://gitlab.
leased vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index b1d02f4b3d..502aad28b2 100644
--- a/hw/virtio/virtio-pci.c
+++ b/hw/virti
On Wed, May 29, 2024 at 9:54 PM Fiona Ebner wrote:
>
> Hi,
>
> Am 22.05.24 um 07:10 schrieb Cindy Lu:
> > The crash was reported in MAC OS and NixOS, here is the link for this bug
> > https://gitlab.com/qemu-project/qemu/-/issues/2334
> > https://gitlab.com/q
: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 18 ++
1 file changed, 2 insertions(+), 16 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index b1d02f4b3d..8a9d0f6ef5 100644
--- a/hw/virtio/virtio-pci.c
+++ b/hw/virtio/virtio-pci.c
es: 2ce6cff94d ("virtio-pci: fix use of a released vector")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index b1d02f4b3d..a7faee5
On Thu, May 9, 2024 at 4:18 AM Michael Tokarev wrote:
>
> 15.04.2024 12:34, Michael S. Tsirkin wrote:
> > From: Cindy Lu
> >
> > During the booting process of the non-standard image, the behavior of the
> > called function in qemu is as follows:
> >
> >
On Wed, Apr 17, 2024 at 2:38 AM Michael S. Tsirkin wrote:
>
> On Tue, Apr 16, 2024 at 02:14:57PM +0100, Peter Maydell wrote:
> > On Tue, 16 Apr 2024 at 13:41, Cindy Lu wrote:
> > >
> > > On Tue, Apr 16, 2024 at 8:30 PM Peter Maydell
> > > wrote:
>
On Tue, Apr 16, 2024 at 11:21 AM Jason Wang wrote:
>
> On Mon, Apr 15, 2024 at 6:41 PM Cindy Lu wrote:
> >
> > On Mon, Apr 15, 2024 at 5:34 PM Michael S. Tsirkin wrote:
> > >
> > > From: Cindy Lu
> > >
> > > During the booting process of th
On Tue, Apr 16, 2024 at 8:22 PM Peter Maydell wrote:
>
> On Tue, 16 Apr 2024 at 12:50, Peter Maydell wrote:
> >
> > On Tue, 16 Apr 2024 at 12:05, Cindy Lu wrote:
> > >
> > > On Tue, Apr 16, 2024 at 6:01 PM Peter Maydell
> > > wrote:
> > >
On Tue, Apr 16, 2024 at 7:50 PM Peter Maydell wrote:
>
> On Tue, 16 Apr 2024 at 12:05, Cindy Lu wrote:
> >
> > On Tue, Apr 16, 2024 at 6:01 PM Peter Maydell
> > wrote:
> > > Here we pass that through to kvm_virtio_pci_vector_use_one().
> > > In kvm_vir
On Tue, Apr 16, 2024 at 8:30 PM Peter Maydell wrote:
>
> On Tue, 16 Apr 2024 at 13:29, Cindy Lu wrote:
> >
> > In function kvm_virtio_pci_vector_use_one(), in the undo label,
> > the function will get the vector incorrectly while using
> > VIRTIO_CONFIG_IRQ_IDX
&
: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 19 +++
1 file changed, 3 insertions(+), 16 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index b138fa127a..565bdb0897 100644
--- a/hw/virtio/virtio-pci.c
+++ b/hw/virtio/virtio-pci.c
On Tue, Apr 16, 2024 at 6:01 PM Peter Maydell wrote:
>
> On Mon, 15 Apr 2024 at 11:52, Michael S. Tsirkin wrote:
> >
> > From: Cindy Lu
> >
> > During the booting process of the non-standard image, the behavior of the
> > called function in qemu is as foll
On Mon, Apr 15, 2024 at 5:34 PM Michael S. Tsirkin wrote:
>
> From: Cindy Lu
>
> During the booting process of the non-standard image, the behavior of the
> called function in qemu is as follows:
>
> 1. vhost_net_stop() was triggered by guest image. This w
On Mon, Apr 15, 2024 at 5:34 PM Michael S. Tsirkin wrote:
>
> From: Cindy Lu
>
> During the booting process of the non-standard image, the behavior of the
> called function in qemu is as follows:
>
> 1. vhost_net_stop() was triggered by guest image. This w
On Mon, Apr 15, 2024 at 4:32 PM Michael S. Tsirkin wrote:
>
> On Fri, Apr 12, 2024 at 02:26:55PM +0800, Cindy Lu wrote:
> > During the booting process of the non-standard image, the behavior of the
> > called function in qemu is as follows:
> >
> > 1. vhost_net_stop(
r);
if (ret < 0) {
return;
}
...
So I move the setting vector into the function
virtio_pci_set_and_change_vector()
the other part are the same .
the sanity test is passed and the qemu qtest is also passed
Thanks
Cindy
On Fri, Apr 12, 2024 at 2:28 PM Cindy Lu wrote:
>
> During th
) at
pthread_create.c:442
23 0x7fc871512630 in clone3 () at
../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
(gdb)
Fixes: f9a09ca3ea ("vhost: add support for configure interrupt")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 43 ++
) at
pthread_create.c:442
23 0x7fc871512630 in clone3 () at
../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
(gdb)
Fixes: f9a09ca3ea ("vhost: add support for configure interrupt")
Cc: qemu-sta...@nongnu.org
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 29 +
On Thu, Apr 11, 2024 at 1:56 PM Michael S. Tsirkin wrote:
>
> On Thu, Apr 11, 2024 at 12:11:30PM +0800, Cindy Lu wrote:
> > During the booting process of the non-standard image, the behavior of the
> > called function in qemu is as follows:
> >
> > 1. vhost_net_stop(
) at
pthread_create.c:442
23 0x7fc871512630 in clone3 () at
../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
(gdb)
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
On Thu, Apr 11, 2024 at 12:18 AM Michael S. Tsirkin wrote:
>
> On Thu, Apr 11, 2024 at 12:12:00AM +0800, Cindy Lu wrote:
> > During the booting process of the non-standard image, the behavior of the
> > called function in qemu is as follows:
> >
> > 1. vhost_net
) at
pthread_create.c:442
23 0x7fc871512630 in clone3 () at
../sysdeps/unix/sysv/linux/x86_64/clone3.S:81
(gdb)
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
On Wed, Apr 10, 2024 at 1:48 PM Jason Wang wrote:
>
> On Wed, Apr 10, 2024 at 1:29 PM Cindy Lu wrote:
> >
> > When the guest triggers vhost_stop and then virtio_reset, the vector will
> > the
> > IRQFD for this vector will be released and change to VIRTIO_NO_VEC
On Wed, Apr 10, 2024 at 1:36 PM Jason Wang wrote:
>
> On Wed, Apr 10, 2024 at 1:29 PM Cindy Lu wrote:
> >
> > When the guest triggers vhost_stop and then virtio_reset, the vector will
> > the
> > IRQFD for this vector will be released and change to VIRTIO_NO_VEC
et_stop(),
So this step will not lost during this process.
Change from V1
1.add the check for if using irqfd
2.remove the check for bool recovery, irqfd's user is enough to check status
Cindy Lu (1):
virtio-pci: Fix the crash that the vector was used after releas
640f8d731 in kvm_vcpu_thread_fn (arg=0x560642f205e0) at
../accel/kvm/kvm-accel-ops.c:51
21 0x5606411949f4 in qemu_thread_start (args=0x560642f292b0) at
../util/qemu-thread-posix.c:541
22 0x7fc87148cdcd in start_thread (arg=) at
pthread_create.c:442
23 0x7fc871512630 in clone3 () at
On Mon, Apr 8, 2024 at 12:59 PM Jason Wang wrote:
>
> On Sun, Apr 7, 2024 at 3:00 PM Cindy Lu wrote:
> >
> > On Sun, Apr 7, 2024 at 12:20 PM Jason Wang wrote:
> > >
> > > On Tue, Apr 2, 2024 at 11:02 PM Cindy Lu wrote:
> > > >
> > >
On Sun, Apr 7, 2024 at 12:20 PM Jason Wang wrote:
>
> On Tue, Apr 2, 2024 at 11:02 PM Cindy Lu wrote:
> >
> > When the guest calls virtio_stop and then virtio_reset,
>
> Guests could not call those functions directly, it is triggered by for
> example writing to some
0 is_masked 1
[msix_fire_vector_notifier] 112 called vector 0 is_masked 0
To fix this, we need to call the function "kvm_virtio_pci_vector_use_one()"
when the vector changes back from VIRTIO_NO_VECTOR
Signed-off-by: Cindy Lu
Cindy Lu (1):
virtio-pci: Fix the crash when the v
e_one()"
when the vector changes back from VIRTIO_NO_VECTOR
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 31 ---
1 file changed, 28 insertions(+), 3 deletions(-)
diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c
index e433879542..45f3ab38c3 10064
On Thu, Mar 28, 2024 at 12:12 PM Jason Wang wrote:
>
> On Wed, Mar 27, 2024 at 5:33 PM Cindy Lu wrote:
> >
> > On Wed, Mar 27, 2024 at 5:12 PM Jason Wang wrote:
> > >
> > > On Wed, Mar 27, 2024 at 4:28 PM Cindy Lu wrote:
> > > >
> >
On Thu, Mar 28, 2024 at 12:14 PM Jason Wang wrote:
>
> On Wed, Mar 27, 2024 at 5:44 PM Cindy Lu wrote:
> >
> > On Wed, Mar 27, 2024 at 5:13 PM Jason Wang wrote:
> > >
> > > On Wed, Mar 27, 2024 at 5:12 PM Jason Wang wrote:
> > > >
> &g
On Wed, Mar 27, 2024 at 5:13 PM Jason Wang wrote:
>
> On Wed, Mar 27, 2024 at 5:12 PM Jason Wang wrote:
> >
> > On Wed, Mar 27, 2024 at 4:28 PM Cindy Lu wrote:
> > >
> > > On Wed, Mar 27, 2024 at 3:54 PM Jason Wang wrote:
> > > >
> &g
On Wed, Mar 27, 2024 at 5:12 PM Jason Wang wrote:
>
> On Wed, Mar 27, 2024 at 4:28 PM Cindy Lu wrote:
> >
> > On Wed, Mar 27, 2024 at 3:54 PM Jason Wang wrote:
> > >
> > > On Wed, Mar 27, 2024 at 2:03 PM Cindy Lu wrote:
> > > >
> >
On Wed, Mar 27, 2024 at 3:54 PM Jason Wang wrote:
>
> On Wed, Mar 27, 2024 at 2:03 PM Cindy Lu wrote:
> >
> > On Wed, Mar 27, 2024 at 11:05 AM Jason Wang wrote:
> > >
> > > Hi Cindy:
> > >
> > > On Wed, Mar 27, 2024 at 9:29 AM Cindy Lu
On Wed, Mar 27, 2024 at 11:05 AM Jason Wang wrote:
>
> Hi Cindy:
>
> On Wed, Mar 27, 2024 at 9:29 AM Cindy Lu wrote:
> >
> > we need a crash in Non-standard image, here is the jira for this
> > https://issues.redhat.com/browse/RHEL-28522
> > The root cause of
will set to true.
Signed-off-by: Cindy Lu
---
hw/net/virtio-net.c| 5 -
hw/virtio/virtio.c | 1 +
include/hw/virtio/virtio.h | 1 +
3 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index 80c56f0cfc..3b487864a8 100644
---
If config_irq_enabled is not true, it means that configure interrupt is
not supported. Therefore, the config vector will not be handled during
the interrupt process.
Signed-off-by: Cindy Lu
---
hw/virtio/virtio-pci.c | 41 +++--
1 file changed, 23 insertions
device Vyatta using, it doesn't support configure interrupts at all, So
we plan to disable the configure interrupts in unsupported device
Signed-off-by: Cindy Lu
Cindy Lu (2):
virtio-net: disable the configure interrupt for not support device
virtio-pci: check if the configure interrupt
Jason Wang 于2024年2月7日周三 11:17写道：
>
> On Tue, Feb 6, 2024 at 4:31 PM Stefano Garzarella wrote:
> >
> > On Tue, Feb 06, 2024 at 10:47:40AM +0800, Jason Wang wrote:
> > >On Mon, Feb 5, 2024 at 6:51 PM Stefano Garzarella
> > >wrote:
> > >>
> > >> On Fri, Feb 02, 2024 at 02:25:21PM +0100, Kevin Wolf
3:30PM +0800, Cindy Lu wrote:
> > Hi All
> > There is the RFC to support the IOMMUFD in vdpa device
> > any comments are welcome
> > Thanks
> > Cindy
>
> Any plans to work on this or should I consider this abandoned?
>
>
> > Cindy Lu (7):
> > vhos
On Mon, Jun 5, 2023 at 1:41 PM Michael S. Tsirkin wrote:
>
> On Fri, May 05, 2023 at 02:29:23PM +0800, Cindy Lu wrote:
> > On Fri, May 5, 2023 at 11:29 AM Jason Wang wrote:
> > >
> > > Hi Cindy
> > >
> > > On Wed, May 3, 2023 at 5:13 PM Cindy Lu
To support vIOMMU in vdpa, need to exposed the function
vhost_dev_has_iommu, vdpa will use this function to check
if vIOMMU enable.
Signed-off-by: Cindy Lu
---
hw/virtio/vhost.c | 2 +-
include/hw/virtio/vhost.h | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw
vhost_vdpa_dev_start, if the SVQ and IOMMU enable at the same time
the function will return fail.
2. Skip the iova_max check vhost_vdpa_listener_skipped_section(). While
MR is IOMMU, move this check to vhost_vdpa_iommu_map_notify()
Verified in vp_vdpa and vdpa_sim_net driver
Signed-off-by: Cindy Lu
---
hw
The unmap ioctl doesn't accept a full 64-bit span. So need to
add check for the section's size in vhost_vdpa_listener_region_del().
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/hw/virtio/vh
changes in V16
remove the batch mode operation in vIOMMU MR
Cindy Lu (4):
vhost: expose function vhost_dev_has_iommu()
vhost_vdpa: fix the input in trace_vhost_vdpa_listener_region_del()
vhost-vdpa: Add check for full 64-bit in region delete
vhost-vdpa: Add support for vIOMMU.
hw/virtio/vhost
In trace_vhost_vdpa_listener_region_del, the value for llend
should change to int128_get64(int128_sub(llend, int128_one()))
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
On Fri, May 5, 2023 at 11:29 AM Jason Wang wrote:
>
> Hi Cindy
>
> On Wed, May 3, 2023 at 5:13 PM Cindy Lu wrote:
> >
> > Hi All
> > There is the RFC to support the IOMMUFD in vdpa device
> > any comments are welcome
> > Thanks
> > Cindy
>
&g
egacy iotlb message if iommufd enable
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 56 ++
include/hw/virtio/vhost-vdpa.h | 24 +++
2 files changed, 74 insertions(+), 6 deletions(-)
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-v
Add a new option for iommufd, The usage is
-object iommufd,id=iommufd0 \
-device virtio-net-pci,netdev=vhost-vdpa1,disable-legacy=on,disable-modern=off\
-netdev
type=vhost-vdpa,vhostdev=/dev/vhost-vdpa-0,id=vhost-vdpa1,iommufd=iommufd0\
...
Signed-off-by: Cindy Lu
---
qapi/net.json | 1
f this is the new asid, get the new ioas_id and attach it to iommufd.
save this information in vdpa_iommufd.
c. Use the ioas_id for mapping
The unmap logic is the same
Signed-off-by: Cindy Lu
---
hw/virtio/meson.build | 2 +-
hw/virtio/vhost-vdpa-iommufd.c
Add 3 new UAPI
VHOST_VDPA_SET_IOMMU_FD: this to bind the vdpa device to iommufd
VDPA_DEVICE_ATTACH_IOMMUFD_AS: attach new ioas to iommufd
VDPA_DEVICE_DTTACH_IOMMUFD_AS: detach all the ioas from iommufd
Signed-off-by: Cindy Lu
---
linux-headers/linux/vhost.h | 72
Add the check for object iommufd, if the iommfd enabled.
pass the information to vhost_vdpa.
vhost_vdpa dev start will check this bool and connect
to the iommufd
Signed-off-by: Cindy Lu
---
net/vhost-vdpa.c | 31 +--
1 file changed, 17 insertions(+), 14 deletions
Add support for iommufd, init the vdpa_iommufd in vdpa_start
in this step, driver will bind to the iommufd device
and attach the default ASID(asid 0) to iommufd
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/hw/virtio
Hi All
There is the RFC to support the IOMMUFD in vdpa device
any comments are welcome
Thanks
Cindy
Cindy Lu (7):
vhost: introduce new UAPI to support IOMMUFD
qapi: support iommufd in vdpa
virtio : add a ptr for vdpa_iommufd in VirtIODevice
net/vhost-vdpa: Add the check for iommufd
write
Signed-off-by: Cindy Lu
---
include/hw/virtio/vhost-vdpa.h | 23 +++
include/hw/virtio/virtio.h | 5 +
2 files changed, 28 insertions(+)
diff --git a/include/hw/virtio/vhost-vdpa.h b/include/hw/virtio/vhost-vdpa.h
index 7997f09a8d..309d4ffc70 100644
--- a
On Fri, Mar 24, 2023 at 10:49 AM Jason Wang wrote:
>
> On Thu, Mar 23, 2023 at 4:41 PM Cindy Lu wrote:
> >
> > On Thu, Mar 23, 2023 at 11:47 AM Jason Wang wrote:
> > >
> > > On Tue, Mar 21, 2023 at 10:24 PM Cindy Lu wrote:
> > > >
> > >
On Thu, Mar 23, 2023 at 11:47 AM Jason Wang wrote:
>
> On Tue, Mar 21, 2023 at 10:24 PM Cindy Lu wrote:
> >
> > 1. The vIOMMU support will make vDPA can work in IOMMU mode. This
> > will fix security issues while using the no-IOMMU mode.
> > To support this featur
In trace_vhost_vdpa_listener_region_del, the value for llend
should change to int128_get64(int128_sub(llend, int128_one()))
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c
vhost_vdpa_dev_start, if the SVQ and IOMMU enable at the same time
the function will return fail.
2. Skip the iova_max check vhost_vdpa_listener_skipped_section(). While
MR is IOMMU, move this check to vhost_vdpa_iommu_map_notify()
Verified in vp_vdpa and vdpa_sim_net driver
Signed-off-by: Cindy Lu
---
hw
To support vIOMMU in vdpa, need to exposed the function
vhost_dev_has_iommu, vdpa will use this function to check
if vIOMMU enable.
Signed-off-by: Cindy Lu
---
hw/virtio/vhost.c | 2 +-
include/hw/virtio/vhost.h | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw
The unmap ioctl doesn't accept a full 64-bit span. So need to
add check for the section's size in vhost_vdpa_listener_region_del().
Signed-off-by: Cindy Lu
---
hw/virtio/vhost-vdpa.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/hw/virtio/vh
crash found in test
changes in V12
Address the comments, squash patch 1 into the next patch
improve the code style issue
changes in V13
fail to start if IOMMU and svq enable at same time
improve the code style issue
changes in V14
Address the comments
changes in V15
Address the comments
Cindy
On Tue, Mar 21, 2023 at 11:21 AM Jason Wang wrote:
>
> On Tue, Mar 21, 2023 at 12:20 AM Cindy Lu wrote:
> >
> > 1. The vIOMMU support will make vDPA can work in IOMMU mode. This
> > will fix security issues while using the no-IOMMU mode.
> > To support this featur
1 - 100 of 545 matches
Mail list logo
