RE: [PATCH] net/virtio: remove address width limit for modern devices

2023-03-10 Thread Xia, Chenbo
> -Original Message- > From: David Marchand > Sent: Thursday, March 9, 2023 10:36 PM > To: dev@dpdk.org > Cc: Pei, Andy ; Maxime Coquelin > ; Xia, Chenbo > Subject: [PATCH] net/virtio: remove address width limit for modern devices > > Modern devices don&#x

RE: [PATCH] net/virtio: remove address width limit for modern devices

2023-03-09 Thread Pei, Andy
Hi David, The case is just want to enable ASAN to scan customer developed code. > -Original Message- > From: David Marchand > Sent: Thursday, March 9, 2023 11:08 PM > To: Maxime Coquelin > Cc: dev@dpdk.org; Pei, Andy ; Xia, Chenbo > > Subject: Re: [PATCH] net/v

RE: [PATCH] net/virtio: remove address width limit for modern devices

2023-03-09 Thread Pei, Andy
HI > -Original Message- > From: David Marchand > Sent: Thursday, March 9, 2023 10:36 PM > To: dev@dpdk.org > Cc: Pei, Andy ; Maxime Coquelin > ; Xia, Chenbo > Subject: [PATCH] net/virtio: remove address width limit for modern devices > > Modern devices don

Re: [PATCH] net/virtio: remove address width limit for modern devices

2023-03-09 Thread David Marchand
On Thu, Mar 9, 2023 at 3:59 PM Maxime Coquelin wrote: > > Hi David, > > On 3/9/23 15:36, David Marchand wrote: > > Modern devices don't have the same limitation as legacy devices, because > > vring addresses are not configured using a 32-bit register. > > > > Do we want to backport it? This is a b

Re: [PATCH] net/virtio: remove address width limit for modern devices

2023-03-09 Thread Maxime Coquelin
Hi David, On 3/9/23 15:36, David Marchand wrote: Modern devices don't have the same limitation as legacy devices, because vring addresses are not configured using a 32-bit register. Do we want to backport it? This is a bug in my opinion. Reviewed-by: Maxime Coquelin Thanks! Maxime Signe

[PATCH] net/virtio: remove address width limit for modern devices

2023-03-09 Thread David Marchand
Modern devices don't have the same limitation as legacy devices, because vring addresses are not configured using a 32-bit register. Signed-off-by: David Marchand --- drivers/net/virtio/virtio_pci.c | 28 1 file changed, 8 insertions(+), 20 deletions(-) diff --git a