DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v1 0/2] small fixes
@ 2020-03-07 13:22 Xiaolong Ye
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment Xiaolong Ye
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Xiaolong Ye @ 2020-03-07 13:22 UTC (permalink / raw)
  To: maxime.coquelin; +Cc: dev, Xiaolong Ye

This series contains small fixes for virtio and vhost.

Xiaolong Ye (2):
  net/virtio: fix for out of date comment
  vhost: remove unused variable

 drivers/net/virtio/virtio_ethdev.c | 6 +++---
 lib/librte_vhost/vhost.h           | 1 -
 2 files changed, 3 insertions(+), 4 deletions(-)

-- 
2.17.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment
  2020-03-07 13:22 [dpdk-dev] [PATCH v1 0/2] small fixes Xiaolong Ye
@ 2020-03-07 13:22 ` Xiaolong Ye
  2020-04-09 14:20   ` Maxime Coquelin
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable Xiaolong Ye
  2020-04-10 14:44 ` [dpdk-dev] [PATCH v1 0/2] small fixes Maxime Coquelin
  2 siblings, 1 reply; 6+ messages in thread
From: Xiaolong Ye @ 2020-03-07 13:22 UTC (permalink / raw)
  To: maxime.coquelin, Tiwei Bie, Zhihong Wang; +Cc: dev, Xiaolong Ye, stable

Fix the comment that's incorrect as the code evolves.

Fixes: 9470427c88e1 ("net/virtio: do not store PCI device pointer at shared memory")
Cc: stable@dpdk.org

Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
---
 drivers/net/virtio/virtio_ethdev.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
index f9d0ea70d..35203940a 100644
--- a/drivers/net/virtio/virtio_ethdev.c
+++ b/drivers/net/virtio/virtio_ethdev.c
@@ -466,7 +466,7 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t vtpci_queue_idx)
 	}
 
 	if (!vtpci_packed_queue(hw) && !rte_is_power_of_2(vq_size)) {
-		PMD_INIT_LOG(ERR, "split virtqueue size is not powerof 2");
+		PMD_INIT_LOG(ERR, "split virtqueue size is not power of 2");
 		return -EINVAL;
 	}
 
@@ -588,8 +588,8 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t vtpci_queue_idx)
 		hw->cvq = cvq;
 	}
 
-	/* For virtio_user case (that is when hw->dev is NULL), we use
-	 * virtual address. And we need properly set _offset_, please see
+	/* For virtio_user case (that is when hw->virtio_user_dev is not NULL),
+	 * we use virtual address. And we need properly set _offset_, please see
 	 * VIRTIO_MBUF_DATA_DMA_ADDR in virtqueue.h for more information.
 	 */
 	if (!hw->virtio_user_dev)
-- 
2.17.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable
  2020-03-07 13:22 [dpdk-dev] [PATCH v1 0/2] small fixes Xiaolong Ye
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment Xiaolong Ye
@ 2020-03-07 13:22 ` Xiaolong Ye
  2020-04-09 14:21   ` Maxime Coquelin
  2020-04-10 14:44 ` [dpdk-dev] [PATCH v1 0/2] small fixes Maxime Coquelin
  2 siblings, 1 reply; 6+ messages in thread
From: Xiaolong Ye @ 2020-03-07 13:22 UTC (permalink / raw)
  To: maxime.coquelin, Tiwei Bie, Zhihong Wang; +Cc: dev, Xiaolong Ye, stable

VHOST_FEATURES has been removed in previous refactor.

Fixes: 0917f9d1f059 ("vhost: use new APIs to handle features")
Cc: stable@dpdk.org

Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
---
 lib/librte_vhost/vhost.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/librte_vhost/vhost.h b/lib/librte_vhost/vhost.h
index 2087d1400..507dbf214 100644
--- a/lib/librte_vhost/vhost.h
+++ b/lib/librte_vhost/vhost.h
@@ -543,7 +543,6 @@ extern int vhost_data_log_level;
 #define PRINT_PACKET(device, addr, size, header) do {} while (0)
 #endif
 
-extern uint64_t VHOST_FEATURES;
 #define MAX_VHOST_DEVICE	1024
 extern struct virtio_net *vhost_devices[MAX_VHOST_DEVICE];
 
-- 
2.17.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment Xiaolong Ye
@ 2020-04-09 14:20   ` Maxime Coquelin
  0 siblings, 0 replies; 6+ messages in thread
From: Maxime Coquelin @ 2020-04-09 14:20 UTC (permalink / raw)
  To: Xiaolong Ye, Tiwei Bie, Zhihong Wang; +Cc: dev, stable



On 3/7/20 2:22 PM, Xiaolong Ye wrote:
> Fix the comment that's incorrect as the code evolves.
> 
> Fixes: 9470427c88e1 ("net/virtio: do not store PCI device pointer at shared memory")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
> ---
>  drivers/net/virtio/virtio_ethdev.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c
> index f9d0ea70d..35203940a 100644
> --- a/drivers/net/virtio/virtio_ethdev.c
> +++ b/drivers/net/virtio/virtio_ethdev.c
> @@ -466,7 +466,7 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t vtpci_queue_idx)
>  	}
>  
>  	if (!vtpci_packed_queue(hw) && !rte_is_power_of_2(vq_size)) {
> -		PMD_INIT_LOG(ERR, "split virtqueue size is not powerof 2");
> +		PMD_INIT_LOG(ERR, "split virtqueue size is not power of 2");
>  		return -EINVAL;
>  	}
>  
> @@ -588,8 +588,8 @@ virtio_init_queue(struct rte_eth_dev *dev, uint16_t vtpci_queue_idx)
>  		hw->cvq = cvq;
>  	}
>  
> -	/* For virtio_user case (that is when hw->dev is NULL), we use
> -	 * virtual address. And we need properly set _offset_, please see
> +	/* For virtio_user case (that is when hw->virtio_user_dev is not NULL),
> +	 * we use virtual address. And we need properly set _offset_, please see
>  	 * VIRTIO_MBUF_DATA_DMA_ADDR in virtqueue.h for more information.
>  	 */
>  	if (!hw->virtio_user_dev)
> 

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable Xiaolong Ye
@ 2020-04-09 14:21   ` Maxime Coquelin
  0 siblings, 0 replies; 6+ messages in thread
From: Maxime Coquelin @ 2020-04-09 14:21 UTC (permalink / raw)
  To: Xiaolong Ye, Tiwei Bie, Zhihong Wang; +Cc: dev, stable



On 3/7/20 2:22 PM, Xiaolong Ye wrote:
> VHOST_FEATURES has been removed in previous refactor.
> 
> Fixes: 0917f9d1f059 ("vhost: use new APIs to handle features")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Xiaolong Ye <xiaolong.ye@intel.com>
> ---
>  lib/librte_vhost/vhost.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/lib/librte_vhost/vhost.h b/lib/librte_vhost/vhost.h
> index 2087d1400..507dbf214 100644
> --- a/lib/librte_vhost/vhost.h
> +++ b/lib/librte_vhost/vhost.h
> @@ -543,7 +543,6 @@ extern int vhost_data_log_level;
>  #define PRINT_PACKET(device, addr, size, header) do {} while (0)
>  #endif
>  
> -extern uint64_t VHOST_FEATURES;
>  #define MAX_VHOST_DEVICE	1024
>  extern struct virtio_net *vhost_devices[MAX_VHOST_DEVICE];
>  
> 


Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dpdk-dev] [PATCH v1 0/2] small fixes
  2020-03-07 13:22 [dpdk-dev] [PATCH v1 0/2] small fixes Xiaolong Ye
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment Xiaolong Ye
  2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable Xiaolong Ye
@ 2020-04-10 14:44 ` Maxime Coquelin
  2 siblings, 0 replies; 6+ messages in thread
From: Maxime Coquelin @ 2020-04-10 14:44 UTC (permalink / raw)
  To: Xiaolong Ye; +Cc: dev



On 3/7/20 2:22 PM, Xiaolong Ye wrote:
> This series contains small fixes for virtio and vhost.
> 
> Xiaolong Ye (2):
>   net/virtio: fix for out of date comment
>   vhost: remove unused variable
> 
>  drivers/net/virtio/virtio_ethdev.c | 6 +++---
>  lib/librte_vhost/vhost.h           | 1 -
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 
Applied to dpdk-next-virtio/master.

Thanks,
Maxime


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-04-10 14:44 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-07 13:22 [dpdk-dev] [PATCH v1 0/2] small fixes Xiaolong Ye
2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 1/2] net/virtio: fix for out of date comment Xiaolong Ye
2020-04-09 14:20   ` Maxime Coquelin
2020-03-07 13:22 ` [dpdk-dev] [PATCH v1 2/2] vhost: remove unused variable Xiaolong Ye
2020-04-09 14:21   ` Maxime Coquelin
2020-04-10 14:44 ` [dpdk-dev] [PATCH v1 0/2] small fixes Maxime Coquelin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).