Re: [dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-12-04 Thread Vlad Zolotarov
On 12/04/2017 12:54 PM, Ferruh Yigit wrote: On 12/1/2017 6:22 PM, Vlad Zolotarov wrote: On 12/01/2017 06:51 PM, Ferruh Yigit wrote: On 12/1/2017 2:17 PM, Vlad Zolotarov wrote: On 11/30/2017 09:29 PM, Ferruh Yigit wrote: remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. Flag seems added

Re: [dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-12-04 Thread Ferruh Yigit
On 12/1/2017 6:22 PM, Vlad Zolotarov wrote: > > > On 12/01/2017 06:51 PM, Ferruh Yigit wrote: >> On 12/1/2017 2:17 PM, Vlad Zolotarov wrote: >>> >>> On 11/30/2017 09:29 PM, Ferruh Yigit wrote: remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. Flag seems added with the patch that

Re: [dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-12-01 Thread Vlad Zolotarov
On 12/01/2017 06:51 PM, Ferruh Yigit wrote: On 12/1/2017 2:17 PM, Vlad Zolotarov wrote: On 11/30/2017 09:29 PM, Ferruh Yigit wrote: remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. Flag seems added with the patch that adds LRO support, and intention looks like giving a pointer to applica

Re: [dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-12-01 Thread Ferruh Yigit
On 12/1/2017 2:17 PM, Vlad Zolotarov wrote: > > > On 11/30/2017 09:29 PM, Ferruh Yigit wrote: >> remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. >> >> Flag seems added with the patch that adds LRO support, and intention >> looks like giving a pointer to application that library supports LRO.

Re: [dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-12-01 Thread Vlad Zolotarov
resending after registering with the new email domain ;) Please, see my comments below. On 12/01/2017 05:17 PM, Vlad Zolotarov wrote: On 11/30/2017 09:29 PM, Ferruh Yigit wrote: remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. Flag seems added with the patch that adds LRO support, and int

[dpdk-dev] [PATCH 1/7] ethdev: remove unused flag from header

2017-11-30 Thread Ferruh Yigit
remove RTE_ETHDEV_HAS_LRO_SUPPORT flag from header. Flag seems added with the patch that adds LRO support, and intention looks like giving a pointer to application that library supports LRO. Fixes: 8eecb3295aed ("ixgbe: add LRO support") Cc: vl...@cloudius-systems.com Signed-off-by: Ferruh Yigit