DPDK-dev Archive on lore.kernel.org
 help / color / Atom feed
* Re: [dpdk-dev] [PATCH v1 1/5] vhost: fix add a missing include
       [not found] <1558861560-114683-1-git-send-email-noae@mellanox.com>
@ 2019-05-26 10:22 ` Matan Azrad
  2019-06-18  6:24   ` Noa Ezra
  0 siblings, 1 reply; 3+ messages in thread
From: Matan Azrad @ 2019-05-26 10:22 UTC (permalink / raw)
  To: Noa Ezra, dev, maxime.coquelin; +Cc: Noa Ezra, stable



 From: Noa Ezra
> Add a missing include with the defines for vhost-user driver features.
> 
> Fixes: 5fbb3941da9f ("vhost: introduce driver features related APIs")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Noa Ezra <noae@mellanox.com>

Reviewed-by: Matan Azrad <matan@mellanox.com>
For all the series.

> ---
>  lib/librte_vhost/rte_vhost.h | 1 +
>  1 file changed, 1 insertion(+)
>  mode change 100644 => 100755 lib/librte_vhost/rte_vhost.h
> 
> diff --git a/lib/librte_vhost/rte_vhost.h b/lib/librte_vhost/rte_vhost.h old
> mode 100644 new mode 100755 index 0226b3e..338e47c
> --- a/lib/librte_vhost/rte_vhost.h
> +++ b/lib/librte_vhost/rte_vhost.h
> @@ -23,6 +23,7 @@
>  /* These are not C++-aware. */
>  #include <linux/vhost.h>
>  #include <linux/virtio_ring.h>
> +#include <linux/virtio_net.h>
> 
>  #define RTE_VHOST_USER_CLIENT		(1ULL << 0)
>  #define RTE_VHOST_USER_NO_RECONNECT	(1ULL << 1)
> --
> 1.8.3.1


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

* Re: [dpdk-dev] [PATCH v1 1/5] vhost: fix add a missing include
  2019-05-26 10:22 ` [dpdk-dev] [PATCH v1 1/5] vhost: fix add a missing include Matan Azrad
@ 2019-06-18  6:24   ` Noa Ezra
  2019-06-18  7:33     ` Maxime Coquelin
  0 siblings, 1 reply; 3+ messages in thread
From: Noa Ezra @ 2019-06-18  6:24 UTC (permalink / raw)
  To: Matan Azrad, dev, maxime.coquelin; +Cc: stable

Hi,
Does anyone else want to review this patch?

Thanks,
Noa.

-----Original Message-----
From: Matan Azrad 
Sent: Sunday, May 26, 2019 1:23 PM
To: Noa Ezra <noae@mellanox.com>; dev@dpdk.org; maxime.coquelin@redhat.com
Cc: Noa Ezra <noae@mellanox.com>; stable@dpdk.org
Subject: RE: [PATCH v1 1/5] vhost: fix add a missing include



 From: Noa Ezra
> Add a missing include with the defines for vhost-user driver features.
> 
> Fixes: 5fbb3941da9f ("vhost: introduce driver features related APIs")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Noa Ezra <noae@mellanox.com>

Reviewed-by: Matan Azrad <matan@mellanox.com> For all the series.

> ---
>  lib/librte_vhost/rte_vhost.h | 1 +
>  1 file changed, 1 insertion(+)
>  mode change 100644 => 100755 lib/librte_vhost/rte_vhost.h
> 
> diff --git a/lib/librte_vhost/rte_vhost.h 
> b/lib/librte_vhost/rte_vhost.h old mode 100644 new mode 100755 index 
> 0226b3e..338e47c
> --- a/lib/librte_vhost/rte_vhost.h
> +++ b/lib/librte_vhost/rte_vhost.h
> @@ -23,6 +23,7 @@
>  /* These are not C++-aware. */
>  #include <linux/vhost.h>
>  #include <linux/virtio_ring.h>
> +#include <linux/virtio_net.h>
> 
>  #define RTE_VHOST_USER_CLIENT		(1ULL << 0)
>  #define RTE_VHOST_USER_NO_RECONNECT	(1ULL << 1)
> --
> 1.8.3.1


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

* Re: [dpdk-dev] [PATCH v1 1/5] vhost: fix add a missing include
  2019-06-18  6:24   ` Noa Ezra
@ 2019-06-18  7:33     ` Maxime Coquelin
  0 siblings, 0 replies; 3+ messages in thread
From: Maxime Coquelin @ 2019-06-18  7:33 UTC (permalink / raw)
  To: Noa Ezra, Matan Azrad, dev; +Cc: stable



On 6/18/19 8:24 AM, Noa Ezra wrote:
> Hi,
> Does anyone else want to review this patch?

I do, but it seems that the other 4 patches of the series aren't on
dev@dpdk.org, is that expected?

> Thanks,
> Noa.
> 
> -----Original Message-----
> From: Matan Azrad
> Sent: Sunday, May 26, 2019 1:23 PM
> To: Noa Ezra <noae@mellanox.com>; dev@dpdk.org; maxime.coquelin@redhat.com
> Cc: Noa Ezra <noae@mellanox.com>; stable@dpdk.org
> Subject: RE: [PATCH v1 1/5] vhost: fix add a missing include
> 
> 
> 
>   From: Noa Ezra
>> Add a missing include with the defines for vhost-user driver features.
>>
>> Fixes: 5fbb3941da9f ("vhost: introduce driver features related APIs")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Noa Ezra <noae@mellanox.com>
> 
> Reviewed-by: Matan Azrad <matan@mellanox.com> For all the series.

For this patch only:
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime

> 
>> ---
>>   lib/librte_vhost/rte_vhost.h | 1 +
>>   1 file changed, 1 insertion(+)
>>   mode change 100644 => 100755 lib/librte_vhost/rte_vhost.h
>>
>> diff --git a/lib/librte_vhost/rte_vhost.h
>> b/lib/librte_vhost/rte_vhost.h old mode 100644 new mode 100755 index
>> 0226b3e..338e47c
>> --- a/lib/librte_vhost/rte_vhost.h
>> +++ b/lib/librte_vhost/rte_vhost.h
>> @@ -23,6 +23,7 @@
>>   /* These are not C++-aware. */
>>   #include <linux/vhost.h>
>>   #include <linux/virtio_ring.h>
>> +#include <linux/virtio_net.h>
>>
>>   #define RTE_VHOST_USER_CLIENT		(1ULL << 0)
>>   #define RTE_VHOST_USER_NO_RECONNECT	(1ULL << 1)
>> --
>> 1.8.3.1
> 

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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1558861560-114683-1-git-send-email-noae@mellanox.com>
2019-05-26 10:22 ` [dpdk-dev] [PATCH v1 1/5] vhost: fix add a missing include Matan Azrad
2019-06-18  6:24   ` Noa Ezra
2019-06-18  7:33     ` Maxime Coquelin

DPDK-dev Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dpdk-dev/0 dpdk-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dpdk-dev dpdk-dev/ https://lore.kernel.org/dpdk-dev \
		dev@dpdk.org dpdk-dev@archiver.kernel.org
	public-inbox-index dpdk-dev


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/ public-inbox