Re: [dpdk-dev] [PATCH] kni: fix build with Linux 5.9

2020-09-09 Thread Ferruh Yigit
On 9/9/2020 12:59 PM, Min Hu (Connor) wrote: > Build error seen with Linux kernel 5.9. > > Build error: > kernel/linux/kni/kni_dev.h:104:30: > error: passing argument 1 of ‘get_user_pages_remote’ from > incompatible pointer type [-Werror=incompatible-pointer-types] > ret = get_us

[dpdk-dev] [PATCH] kni: fix build with Linux 5.9

2020-09-09 Thread Min Hu (Connor)
Build error seen with Linux kernel 5.9. Build error: kernel/linux/kni/kni_dev.h:104:30: error: passing argument 1 of ‘get_user_pages_remote’ from incompatible pointer type [-Werror=incompatible-pointer-types] ret = get_user_pages_remote(tsk, tsk->mm, iova, 1, get_user_page

[dpdk-dev] [PATCH] kni: fix build with Linux 5.9

2020-08-17 Thread Ferruh Yigit
Starting from Linux 5.9 'get_user_pages_remote()' API doesn't get 'struct task_struct' parameter: commit 64019a2e467a ("mm/gup: remove task_struct pointer for all gup code") The change reflected to the KNI with version check. Signed-off-by: Ferruh Yigit --- kernel/linux/kni/compat.h | 4

[dpdk-dev] [PATCH] kni: fix build with Linux 5.6

2020-02-12 Thread Ferruh Yigit
With the following Linux commit a new parameter 'txqueue' has been added to 'ndo_tx_timeout' ndo: commit 0290bd291cc0 ("netdev: pass the stuck queue to the timeout handler") The change reflected to the KNI with version check. Cc: sta...@dpdk.org Signed-off-by: Ferruh Yigit --- kernel/linux/kni

Re: [dpdk-dev] [PATCH] kni: fix build with 4.9.x

2019-11-28 Thread David Marchand
On Thu, Nov 28, 2019 at 2:43 PM Andrew Rybchenko wrote: > On 11/28/19 3:15 PM, Ferruh Yigit wrote: > > The 'get_user_pages_remote()' API is updated in kernel 4.10.0 [1], > > but the check added as > 4.9.0, > > this logic is broken for kernels 4.9.x, because they justify > >> 4.9.0 check but have t

Re: [dpdk-dev] [PATCH] kni: fix build with 4.9.x

2019-11-28 Thread Andrew Rybchenko
On 11/28/19 3:15 PM, Ferruh Yigit wrote: > The 'get_user_pages_remote()' API is updated in kernel 4.10.0 [1], > but the check added as > 4.9.0, > this logic is broken for kernels 4.9.x, because they justify >> 4.9.0 check but have the old API. > Fixing the check as >= 4.10.0 > > [1] > commit 5b56d4

[dpdk-dev] [PATCH] kni: fix build with 4.9.x

2019-11-28 Thread Ferruh Yigit
The 'get_user_pages_remote()' API is updated in kernel 4.10.0 [1], but the check added as > 4.9.0, this logic is broken for kernels 4.9.x, because they justify > 4.9.0 check but have the old API. Fixing the check as >= 4.10.0 [1] commit 5b56d49fc31d ("mm: add locked parameter to get_user_pages_re

[dpdk-dev] [PATCH] kni: fix build on RHEL8

2019-05-26 Thread michael . luo
From: Luo Gaoliang Build error was seen on RHEL8 when CONFIG_RTE_KNI_KMOD_ETHTOOL is enabled. Build error log: /root/kni/dpdk-19.05/kernel/linux/kni/ethtool/igb/kcompat.h:3928:25: error: 'const struct net_device_ops' has no member named 'extended' #define ndo_set_vf_vlan extended.ndo_set_vf_vla

Re: [dpdk-dev] [PATCH] kni: fix build for ndo_fdb_add in Linux 5.1

2019-04-09 Thread Rami Rosen
On Mon, Apr 08, 2019 at 03:22:26PM +0100, Ferruh Yigit wrote: > Build error seen with Linux kernel 5.1 and > when CONFIG_RTE_KNI_KMOD_ETHTOOL is enabled. > > Build error: > .../dpdk/build/build/kernel/linux/kni/igb_main.c:2352:18: > error: initialization of ... from incompatible pointer type ...

Re: [dpdk-dev] [PATCH] kni: fix build for ndo_fdb_add in Linux 5.1

2019-04-08 Thread Bruce Richardson
On Mon, Apr 08, 2019 at 03:22:26PM +0100, Ferruh Yigit wrote: > Build error seen with Linux kernel 5.1 and > when CONFIG_RTE_KNI_KMOD_ETHTOOL is enabled. > > Build error: > .../dpdk/build/build/kernel/linux/kni/igb_main.c:2352:18: > error: initialization of ... from incompatible pointer type ...

[dpdk-dev] [PATCH] kni: fix build for ndo_fdb_add in Linux 5.1

2019-04-08 Thread Ferruh Yigit
Build error seen with Linux kernel 5.1 and when CONFIG_RTE_KNI_KMOD_ETHTOOL is enabled. Build error: .../dpdk/build/build/kernel/linux/kni/igb_main.c:2352:18: error: initialization of ... from incompatible pointer type ... [-Werror=incompatible-pointer-types] .ndo_fdb_add = igb_ndo_fdb_ad

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2019-01-14 Thread Thomas Monjalon
21/12/2018 18:02, Ferruh Yigit: > On 12/21/2018 4:27 PM, David Zeng wrote: > > Signed-off-by: David Zeng > > Acked-by: Ferruh Yigit Applied on top of another fix: http://git.dpdk.org/dpdk/commit/?id=4c820b22 and renamed: kni: fix build on RHEL8 for arm and Power9 Thanks

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2018-12-21 Thread Ferruh Yigit
On 12/21/2018 4:27 PM, David Zeng wrote: > Signed-off-by: David Zeng Acked-by: Ferruh Yigit

[dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2018-12-21 Thread David Zeng
Signed-off-by: David Zeng --- kernel/linux/kni/compat.h | 9 - kernel/linux/kni/ethtool/igb/kcompat.h | 4 +++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/kernel/linux/kni/compat.h b/kernel/linux/kni/compat.h index 5aadebb..3c575c7 100644 --- a/kernel/lin

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2018-12-21 Thread Ferruh Yigit
On 12/21/2018 4:15 PM, David Zeng wrote: > Signed-off-by: David Zeng Acked-by: Ferruh Yigit

[dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2018-12-21 Thread David Zeng
Thanks, Ferruh!

[dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT

2018-12-21 Thread David Zeng
Signed-off-by: David Zeng --- kernel/linux/kni/compat.h | 9 - kernel/linux/kni/ethtool/igb/kcompat.h | 4 +++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/kernel/linux/kni/compat.h b/kernel/linux/kni/compat.h index 5aadebb..3c575c7 100644 --- a/kernel/lin

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9

2018-12-21 Thread Ferruh Yigit
On 12/21/2018 6:03 AM, David Zeng wrote: > Signed-off-by: David Zeng > --- > kernel/linux/kni/compat.h | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/kernel/linux/kni/compat.h b/kernel/linux/kni/compat.h > index 5aadebb..1afa3b8 100644 > --- a/kernel/linux/kni/co

[dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9

2018-12-20 Thread David Zeng
Signed-off-by: David Zeng --- kernel/linux/kni/compat.h | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/kernel/linux/kni/compat.h b/kernel/linux/kni/compat.h index 5aadebb..1afa3b8 100644 --- a/kernel/linux/kni/compat.h +++ b/kernel/linux/kni/compat.h @@ -102,8 +102,14

Re: [dpdk-dev] [PATCH] kni: Fix build on RHEL 8

2018-12-19 Thread Ferruh Yigit
On 11/27/2018 1:38 PM, Ferruh Yigit wrote: > On 11/27/2018 11:13 AM, Mohammed Gamal wrote: >> On Tue, 2018-11-27 at 10:55 +, Ferruh Yigit wrote: >>> On 11/26/2018 8:45 PM, Mohammed Gamal wrote: As RHEL8 beta has been released and is based on recent upstream kernels which don't us

Re: [dpdk-dev] [PATCH] kni: Fix build on RHEL 8

2018-11-27 Thread Ferruh Yigit
On 11/27/2018 11:13 AM, Mohammed Gamal wrote: > On Tue, 2018-11-27 at 10:55 +, Ferruh Yigit wrote: >> On 11/26/2018 8:45 PM, Mohammed Gamal wrote: >>> As RHEL8 beta has been released and is based on recent upstream >>> kernels >>> which don't use ndo_change_mtu_rh74 function. Fix the build so t

Re: [dpdk-dev] [PATCH] kni: Fix build on RHEL 8

2018-11-27 Thread Mohammed Gamal
On Tue, 2018-11-27 at 10:55 +, Ferruh Yigit wrote: > On 11/26/2018 8:45 PM, Mohammed Gamal wrote: > > As RHEL8 beta has been released and is based on recent upstream > > kernels > > which don't use ndo_change_mtu_rh74 function. Fix the build so that > > it > > doesn't fail because the function

Re: [dpdk-dev] [PATCH] kni: Fix build on RHEL 8

2018-11-27 Thread Ferruh Yigit
On 11/26/2018 8:45 PM, Mohammed Gamal wrote: > As RHEL8 beta has been released and is based on recent upstream kernels > which don't use ndo_change_mtu_rh74 function. Fix the build so that it > doesn't fail because the function is missing > > Signed-off-by: Mohammed Gamal > --- > kernel/linux/kn

[dpdk-dev] [PATCH] kni: Fix build on RHEL 8

2018-11-26 Thread Mohammed Gamal
As RHEL8 beta has been released and is based on recent upstream kernels which don't use ndo_change_mtu_rh74 function. Fix the build so that it doesn't fail because the function is missing Signed-off-by: Mohammed Gamal --- kernel/linux/kni/compat.h | 3 ++- 1 file changed, 2 insertions(+), 1 dele

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
27/10/2018 00:42, Ferruh Yigit: > On 10/26/2018 10:56 PM, Thomas Monjalon wrote: > > 26/10/2018 23:40, Thomas Monjalon: > >> The atomic functions smp_load_acquire() and smp_store_release() > >> were introduced in Linux 3.14. Older kernels miss the functions: > >> > >> kni_fifo.h:19:2: error: > >>

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Ferruh Yigit
On 10/26/2018 10:56 PM, Thomas Monjalon wrote: > 26/10/2018 23:40, Thomas Monjalon: >> The atomic functions smp_load_acquire() and smp_store_release() >> were introduced in Linux 3.14. Older kernels miss the functions: >> >> kni_fifo.h:19:2: error: >> implicit declaration of function ‘smp_load

Re: [dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
26/10/2018 23:40, Thomas Monjalon: > The atomic functions smp_load_acquire() and smp_store_release() > were introduced in Linux 3.14. Older kernels miss the functions: > > kni_fifo.h:19:2: error: > implicit declaration of function ‘smp_load_acquire’ > kni_fifo.h:30:2: error: > implicit

[dpdk-dev] [PATCH] kni: fix build on Linux < 3.14

2018-10-26 Thread Thomas Monjalon
The atomic functions smp_load_acquire() and smp_store_release() were introduced in Linux 3.14. Older kernels miss the functions: kni_fifo.h:19:2: error: implicit declaration of function ‘smp_load_acquire’ kni_fifo.h:30:2: error: implicit declaration of function ‘smp_store_release’

[dpdk-dev] [PATCH] kni: fix build for Linux kernel 4.19

2018-10-24 Thread Ferruh Yigit
The build error observed with Linux kernel 4.19 when KNI ethtool support enabled (CONFIG_RTE_KNI_KMOD_ETHTOOL=y) .../build/build/kernel/linux/kni/kni_ethtool.c:193:3: error: ‘struct ethtool_ops’ has no member named ‘get_settings’; .get_settings = kni_get_settings, ^~~~ .../build/

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL 7.5

2018-08-09 Thread Drocula
Sorry for my mistake, This patch fixes compilation errors on Centos 7.5 when CONFIG_RTE_KNI_KMOD_ETHTOOL is set to 'y'. On RHEL75 ndo_change_mtu has changed to ndo_change_mtu_rh74. See commit 37d477b6863e5c06e20be434b559d3a03d89f46a -- Drocula Lambda Thanks On Aug 9, 2018 18:25, "Drocula" wro

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL 7.5

2018-08-09 Thread Drocula
This patch Fix compilation errors on Centos 7.4 when CONFIG_RTE_KNI_KMOD_ETHTOOL is set to 'y'. On Thu, Aug 9, 2018, 17:59 Ferruh Yigit wrote: > On 8/6/2018 1:06 PM, Drocula wrote: > > Signed-off-by: Drocula > > +1 for fixing build error, hence > Acked-by: Ferruh Yigit > > But it would be bett

Re: [dpdk-dev] [PATCH] kni: fix build on RHEL 7.5

2018-08-09 Thread Ferruh Yigit
On 8/6/2018 1:06 PM, Drocula wrote: > Signed-off-by: Drocula +1 for fixing build error, hence Acked-by: Ferruh Yigit But it would be better to have more detail on the build error, why and when this change required in RedHat side, it would be good if this information can make on time, but not I

[dpdk-dev] [PATCH] kni: fix build on RHEL 7.5

2018-08-06 Thread Drocula
Signed-off-by: Drocula --- kernel/linux/kni/ethtool/igb/kcompat.h | 5 + 1 file changed, 5 insertions(+) diff --git a/kernel/linux/kni/ethtool/igb/kcompat.h b/kernel/linux/kni/ethtool/igb/kcompat.h index 40a8d99..ae1b530 100644 --- a/kernel/linux/kni/ethtool/igb/kcompat.h +++ b/kernel/linux

Re: [dpdk-dev] [PATCH] kni: fix build with gcc 8.1

2018-06-20 Thread Ferruh Yigit
On 6/20/2018 5:39 PM, Ferruh Yigit wrote: > On 6/20/2018 12:35 AM, Stephen Hemminger wrote: >> On Tue, 19 Jun 2018 13:08:55 +0100 >> Ferruh Yigit wrote: >> >>> Error observed when CONFIG_RTE_KNI_KMOD_ETHTOOL config option is >>> enabled. >>> >>> build error: >>> In function ‘strncpy’, >>> inli

Re: [dpdk-dev] [PATCH] kni: fix build with gcc 8.1

2018-06-20 Thread Ferruh Yigit
On 6/20/2018 12:35 AM, Stephen Hemminger wrote: > On Tue, 19 Jun 2018 13:08:55 +0100 > Ferruh Yigit wrote: > >> Error observed when CONFIG_RTE_KNI_KMOD_ETHTOOL config option is >> enabled. >> >> build error: >> In function ‘strncpy’, >> inlined from ‘igb_get_drvinfo’ at >> .../dpdk/build/

Re: [dpdk-dev] [PATCH] kni: fix build with gcc 8.1

2018-06-19 Thread Stephen Hemminger
On Tue, 19 Jun 2018 13:08:55 +0100 Ferruh Yigit wrote: > Error observed when CONFIG_RTE_KNI_KMOD_ETHTOOL config option is > enabled. > > build error: > In function ‘strncpy’, > inlined from ‘igb_get_drvinfo’ at > .../dpdk/build/build/kernel/linux/kni/igb_ethtool.c:814:2: > .../includ

[dpdk-dev] [PATCH] kni: fix build with gcc 8.1

2018-06-19 Thread Ferruh Yigit
Error observed when CONFIG_RTE_KNI_KMOD_ETHTOOL config option is enabled. build error: In function ‘strncpy’, inlined from ‘igb_get_drvinfo’ at .../dpdk/build/build/kernel/linux/kni/igb_ethtool.c:814:2: .../include/linux/string.h:246:9: error: ‘__builtin_strncpy’ output may be trun

Re: [dpdk-dev] [PATCH] kni: fix build on Centos 7.4 when Ethtool enabled

2018-05-22 Thread Ferruh Yigit
On 5/22/2018 1:18 AM, Dan Gora wrote: > Fix compilation errors on Centos 7.4 when CONFIG_RTE_KNI_KMOD_ETHTOOL > is set to 'y'. > > Centos 7.4 needs HAVE_NDO_BRIDGE_GETLINK_FILTER_MASK_VLAN_FILL: > > igb_main.c: In function ‘igb_ndo_bridge_getlink’: > igb_main.c:2289:2: error: too few arguments to

[dpdk-dev] [PATCH] kni: fix build on Centos 7.4 when Ethtool enabled

2018-05-21 Thread Dan Gora
Fix compilation errors on Centos 7.4 when CONFIG_RTE_KNI_KMOD_ETHTOOL is set to 'y'. Centos 7.4 needs HAVE_NDO_BRIDGE_GETLINK_FILTER_MASK_VLAN_FILL: igb_main.c: In function ‘igb_ndo_bridge_getlink’: igb_main.c:2289:2: error: too few arguments to function ‘ndo_dflt_bridge_getlin

Re: [dpdk-dev] [PATCH] kni: fix build dependency

2018-01-11 Thread Thomas Monjalon
> > kni library has a dependency to new PCI library, adding that dependency. > > > > build error: > > CC rte_kni.o > > In file included from dpdk/lib/librte_kni/rte_kni.c:48:0: > > dpdk/build/include/rte_kni.h:49:21: > > fatal error: rte_pci.h: No such file or directory > > #include > >

Re: [dpdk-dev] [PATCH] kni: fix build dependency

2017-12-19 Thread Iremonger, Bernard
Hi Ferruh, > -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, December 12, 2017 6:40 PM > To: Yigit, Ferruh > Cc: dev@dpdk.org; Iremonger, Bernard ; > sta...@dpdk.org > Subject: [PATCH] kni: fix build dependency > > kni library has a dependency to new PCI library, adding that de

Re: [dpdk-dev] [PATCH] kni: fix build with kernel 4.15

2017-12-15 Thread Thomas Monjalon
29/11/2017 00:45, Ferruh Yigit: > build error: > .../dpdk/build/build/lib/librte_eal/linuxapp/kni/igb_main.c:2809:2: > error: implicit declaration of function ‘setup_timer’; > did you mean ‘sk_stop_timer’? [-Werror=implicit-function-declaration] > setup_timer(&adapter->watchdog_timer, &igb_watc

[dpdk-dev] [PATCH] kni: fix build dependency

2017-12-12 Thread Ferruh Yigit
kni library has a dependency to new PCI library, adding that dependency. build error: CC rte_kni.o In file included from dpdk/lib/librte_kni/rte_kni.c:48:0: dpdk/build/include/rte_kni.h:49:21: fatal error: rte_pci.h: No such file or directory #include ^ Fixes: c7

[dpdk-dev] [PATCH] kni: fix build with kernel 4.15

2017-11-28 Thread Ferruh Yigit
build error: .../dpdk/build/build/lib/librte_eal/linuxapp/kni/igb_main.c:2809:2: error: implicit declaration of function ‘setup_timer’; did you mean ‘sk_stop_timer’? [-Werror=implicit-function-declaration] setup_timer(&adapter->watchdog_timer, &igb_watchdog, ^~~ sk_stop_timer cc1: a

Re: [dpdk-dev] [PATCH] kni: fix build on SLE12 SP3

2017-08-18 Thread Ferruh Yigit
Hi Nirmoy, On 8/15/2017 10:58 AM, Nirmoy Das wrote: > compilation error: > build/lib/librte_eal/linuxapp/kni/kni_net.c:215:5: error: ‘struct net_device’ > has no member named ‘trans_start’ Commit log is too long, can you please break this line. > dev->trans_start = jiffies; > > Signed-off-by

[dpdk-dev] [PATCH] kni: fix build on SLE12 SP3

2017-08-15 Thread Nirmoy Das
compilation error: build/lib/librte_eal/linuxapp/kni/kni_net.c:215:5: error: ‘struct net_device’ has no member named ‘trans_start’ dev->trans_start = jiffies; Signed-off-by: Nirmoy Das --- lib/librte_eal/linuxapp/kni/compat.h | 32 +++- 1 file changed, 31 insertion

Re: [dpdk-dev] [PATCH] kni: fix build with kernel 4.11

2017-03-21 Thread Pankaj Gupta
Hello, > > compile error: > .../build/build/lib/librte_eal/linuxapp/kni/kni_net.c:124:6: > error: implicit declaration of function ‘signal_pending’ > [-Werror=implicit-function-declaration] > if (signal_pending(current) || ret_val <= 0) { > ^~ > > Linux 4.11 moves signal fun

Re: [dpdk-dev] [PATCH] kni: fix build with kernel 4.11

2017-03-20 Thread Jerin Jacob
On Mon, Mar 20, 2017 at 11:22:08AM +, Ferruh Yigit wrote: > compile error: > .../build/build/lib/librte_eal/linuxapp/kni/kni_net.c:124:6: > error: implicit declaration of function ‘signal_pending’ > [-Werror=implicit-function-declaration] > if (signal_pending(current) || ret_val <= 0) { >

[dpdk-dev] [PATCH] kni: fix build with kernel 4.11

2017-03-20 Thread Ferruh Yigit
compile error: .../build/build/lib/librte_eal/linuxapp/kni/kni_net.c:124:6: error: implicit declaration of function ‘signal_pending’ [-Werror=implicit-function-declaration] if (signal_pending(current) || ret_val <= 0) { ^~ Linux 4.11 moves signal function declarations to its ow

[dpdk-dev] [PATCH] kni: fix build with kernel 4.9

2016-10-25 Thread Thomas Monjalon
> > compile error: > > CC [M] .../lib/librte_eal/linuxapp/kni/igb_main.o > > .../lib/librte_eal/linuxapp/kni/igb_main.c:2317:21: > > error: initialization from incompatible pointer type > > [-Werror=incompatible-pointer-types] > > .ndo_set_vf_vlan = igb_ndo_set_vf_vlan, > >

[dpdk-dev] [PATCH] kni: fix build with kernel 4.9

2016-10-17 Thread De Lara Guarch, Pablo
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ferruh Yigit > Sent: Monday, October 17, 2016 3:23 AM > To: dev at dpdk.org > Cc: Yigit, Ferruh > Subject: [dpdk-dev] [PATCH] kni: fix build with kernel 4.9 > > compile error: >

[dpdk-dev] [PATCH] kni: fix build with kernel 4.9

2016-10-17 Thread Ferruh Yigit
compile error: CC [M] .../lib/librte_eal/linuxapp/kni/igb_main.o .../lib/librte_eal/linuxapp/kni/igb_main.c:2317:21: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types] .ndo_set_vf_vlan = igb_ndo_set_vf_vlan, ^~

[dpdk-dev] [PATCH] kni: fix build with kernel < v3.0

2016-09-21 Thread Thomas Monjalon
2016-09-16 17:09, Ferruh Yigit: > Compile error: > CC [M] .../build/lib/librte_eal/linuxapp/kni/igb_main.o > .../build/lib/librte_eal/linuxapp/kni/igb_main.c: > In function ?igb_check_swap_media?: > .../build/lib/librte_eal/linuxapp/kni/igb_main.c:1556:7: > error: variable ?link? set but not use

[dpdk-dev] [PATCH] kni: fix build with kernel < v3.0

2016-09-16 Thread Ferruh Yigit
Compile error: CC [M] .../build/lib/librte_eal/linuxapp/kni/igb_main.o .../build/lib/librte_eal/linuxapp/kni/igb_main.c: In function ?igb_check_swap_media?: .../build/lib/librte_eal/linuxapp/kni/igb_main.c:1556:7: error: variable ?link? set but not used [-Werror=unused-but-set-variable] bool l

[dpdk-dev] [PATCH] kni: fix build with kernel 4.8

2016-09-08 Thread Ferruh Yigit
Linux kernel v4.8 removes macro DEFINE_PCI_DEVICE_TABLE Linux: 7e9321599011 ("treewide: remove references to the now unnecessary DEFINE_PCI_DEVICE_TABLE") Added the macro to compatibility headers of the kni ethtool drivers. Signed-off-by: Ferruh Yigit --- lib/librte_eal/linuxapp/kni/ethtool/ig

[dpdk-dev] [PATCH] kni: fix build with kernel 4.8

2016-09-08 Thread Stephen Hemminger
On Thu, 8 Sep 2016 18:54:56 +0100 Ferruh Yigit wrote: > Linux kernel v4.8 removes macro DEFINE_PCI_DEVICE_TABLE > > Linux: 7e9321599011 ("treewide: remove references to the now unnecessary > DEFINE_PCI_DEVICE_TABLE") > > Added the macro to compatibility headers of the kni ethtool drivers. > >

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Thomas Monjalon
2016-06-21 13:27, Ferruh Yigit: > On 6/21/2016 12:43 PM, Anupam Kapoor wrote: > > This commit fixes build errors triggered due misleading indentation. > > > > Fixes: 38db3f7f50bde (e1000: update base driver) > Fixes: b9ee370557f1 ("kni: update kernel driver ethtool baseline") > > > Fixes: 3fc5ca2

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Anupam Kapoor
This commit fixes build errors triggered due misleading indentation. Fixes: 38db3f7f50bde (e1000: update base driver) Fixes: 3fc5ca2f63529 (kni: initial import) Signed-off-by: Anupam Kapoor --- lib/librte_eal/linuxapp/kni/ethtool/igb/e1000_phy.c | 6 -- lib/librte_eal/linuxapp/kni/etht

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Anupam Kapoor
On Tue, Jun 21, 2016 at 3:40 PM, Ferruh Yigit wrote: > Hi Anupam, > > Thank you for the patch. > > > On 6/21/2016 9:37 AM, Anupam Kapoor wrote: > > This commit fixes build errors triggered due misleading indentation. > > > > Fixes: 366113dbfb696 (e1000: suppress misleading indentation warning) >

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Anupam Kapoor
This commit fixes build errors triggered due misleading indentation. Fixes: 366113dbfb696 (e1000: suppress misleading indentation warning) Signed-off-by: Anupam Kapoor --- lib/librte_eal/linuxapp/kni/ethtool/igb/e1000_phy.c | 12 lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Ferruh Yigit
On 6/21/2016 12:43 PM, Anupam Kapoor wrote: > This commit fixes build errors triggered due misleading indentation. > > Fixes: 38db3f7f50bde (e1000: update base driver) Fixes: b9ee370557f1 ("kni: update kernel driver ethtool baseline") > Fixes: 3fc5ca2f63529 (kni: initial import) > > > Signed-of

[dpdk-dev] [PATCH] kni : fix build errors for gcc --version >= 6.1

2016-06-21 Thread Ferruh Yigit
Hi Anupam, Thank you for the patch. On 6/21/2016 9:37 AM, Anupam Kapoor wrote: > This commit fixes build errors triggered due misleading indentation. > > Fixes: 366113dbfb696 (e1000: suppress misleading indentation warning) This looks like wrong commit id that fixed, can you please double check

[dpdk-dev] [PATCH] kni: fix build with Linux kernel 4.1 or later

2015-07-10 Thread Stephen Hemminger
On Fri, 10 Jul 2015 02:53:51 +0200 Thomas Monjalon wrote: > 2015-07-09 10:43, Stephen Hemminger: > > From: Stephen Hemminger > > > > The internal API for some functions used by KNI changes in current > > kernel. KNI really needs to be submitted upstream (by Intel) > > to keep it maintainable. >

[dpdk-dev] [PATCH] kni: fix build with Linux kernel 4.1 or later

2015-07-10 Thread Thomas Monjalon
2015-07-10 07:16, De Lara Guarch, Pablo: > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > 2015-07-09 10:43, Stephen Hemminger: > > > From: Stephen Hemminger > > > > > > The internal API for some functions used by KNI changes in current > > > kernel. KNI really needs to

[dpdk-dev] [PATCH] kni: fix build with Linux kernel 4.1 or later

2015-07-10 Thread De Lara Guarch, Pablo
Hi Thomas, > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Friday, July 10, 2015 1:54 AM > To: Stephen Hemminger > Cc: dev at dpdk.org; Stephen Hemminger > Subject: Re: [dpdk-dev] [PATCH] kni: fix build with Linux k

[dpdk-dev] [PATCH] kni: fix build with Linux kernel 4.1 or later

2015-07-10 Thread Thomas Monjalon
2015-07-09 10:43, Stephen Hemminger: > From: Stephen Hemminger > > The internal API for some functions used by KNI changes in current > kernel. KNI really needs to be submitted upstream (by Intel) > to keep it maintainable. > > Signed-off-by: Stephen Hemminger There was already a similar patch

[dpdk-dev] [PATCH] kni: fix build with Linux kernel 4.1 or later

2015-07-09 Thread Stephen Hemminger
From: Stephen Hemminger The internal API for some functions used by KNI changes in current kernel. KNI really needs to be submitted upstream (by Intel) to keep it maintainable. Signed-off-by: Stephen Hemminger --- lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c | 12 +++- lib/librte

[dpdk-dev] [PATCH] kni: fix build for CentOS 6.6

2014-12-18 Thread Thomas Monjalon
> From CentOS 6.6, function skb_set_hash is introduced, this breaks > the previous assumption. So modify RHEL_RELEASE_VERSION from 7.0 > to 6.6 to fix build for rte_kni.ko. > > Related mail from Barak Enat: > http://dpdk.org/ml/archives/dev/2014-December/010124.html > > building error likes: >

[dpdk-dev] [PATCH] kni: fix build for CentOS 6.6

2014-12-18 Thread Jincheng Miao
>From CentOS 6.6, function skb_set_hash is introduced, this breaks the previous assumption. So modify RHEL_RELEASE_VERSION from 7.0 to 6.6 to fix build for rte_kni.ko. Related mail from Barak Enat: http://dpdk.org/ml/archives/dev/2014-December/010124.html building error likes: CC [M] /root/dp

[dpdk-dev] [PATCH] kni: fix build on RHEL6.5

2014-12-18 Thread Thomas Monjalon
2014-12-17 11:26, Jincheng Miao: > On 12/16/2014 11:21 PM, Thomas Monjalon wrote: > > 2014-12-11 13:27, Jincheng Miao: > >> RHEL6.5 kernel is based on 2.6.32. But there are two changing > >> from 2.6.35: > >> 1. socket struct is changed > >> It wrappered previous wait_queue_head_t of socket to > >>

[dpdk-dev] [PATCH] kni: fix build on RHEL6.5

2014-12-17 Thread Jincheng Miao
On 12/16/2014 11:21 PM, Thomas Monjalon wrote: > 2014-12-11 13:27, Jincheng Miao: >> RHEL6.5 kernel is based on 2.6.32. But there are two changing >> from 2.6.35: >> 1. socket struct is changed >> It wrappered previous wait_queue_head_t of socket to >> struct socket_wq. So for the kernel older than

[dpdk-dev] [PATCH] kni: fix build on RHEL6.5

2014-12-16 Thread Thomas Monjalon
2014-12-11 13:27, Jincheng Miao: > RHEL6.5 kernel is based on 2.6.32. But there are two changing > from 2.6.35: > 1. socket struct is changed > It wrappered previous wait_queue_head_t of socket to > struct socket_wq. So for the kernel older than 2.6.35, we should > directly use socket->wait instead

[dpdk-dev] [PATCH] kni: fix build on RHEL6.5

2014-12-11 Thread Jincheng Miao
RHEL6.5 kernel is based on 2.6.32. But there are two changing from 2.6.35: 1. socket struct is changed It wrappered previous wait_queue_head_t of socket to struct socket_wq. So for the kernel older than 2.6.35, we should directly use socket->wait instead. 2. new function sk_sleep() This function i

[dpdk-dev] [PATCH] kni: fix build

2014-11-10 Thread Thomas Monjalon
> > Since commit 08b563ffb19 ("mbuf: replace data pointer by an offset"), > > KNI vhost compilation (CONFIG_RTE_KNI_VHOST=y) was broken. > > > > rte_pktmbuf_mtod() is not used in the kernel context but is replaced > > by a simple addition of the base address and the offset. > > > > Signed-off-by:

[dpdk-dev] [PATCH] kni: fix build

2014-11-07 Thread Olivier MATZ
Hi Thomas, On 11/07/2014 03:51 PM, Thomas Monjalon wrote: > Since commit 08b563ffb19 ("mbuf: replace data pointer by an offset"), > KNI vhost compilation (CONFIG_RTE_KNI_VHOST=y) was broken. > > rte_pktmbuf_mtod() is not used in the kernel context but is replaced > by a simple addition of the bas

[dpdk-dev] [PATCH] kni: fix build

2014-11-07 Thread Thomas Monjalon
Since commit 08b563ffb19 ("mbuf: replace data pointer by an offset"), KNI vhost compilation (CONFIG_RTE_KNI_VHOST=y) was broken. rte_pktmbuf_mtod() is not used in the kernel context but is replaced by a simple addition of the base address and the offset. Signed-off-by: Thomas Monjalon --- lib/l

[dpdk-dev] [PATCH] kni: fix build with Linux 3.17

2014-08-29 Thread Thomas Monjalon
> In the series of ever-lasting ugly #ifdefs trying make out-of-tree > drivers to support latest mainline kernel... > > Signed-off-by: Aaro Koskinen Acked-by: Thomas Monjalon Applied with this commit log: Since Linux commit "set name_assign_type in alloc_netdev" (c835a677331495), the f

[dpdk-dev] [PATCH] kni: fix build with Linux 3.17

2014-08-19 Thread Zhang, Helin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Aaro Koskinen > Sent: Monday, August 18, 2014 8:45 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] kni: fix build with Linux 3.17 > > In the series of ever-lasting ugly #ifdefs tr

[dpdk-dev] [PATCH] kni: fix build with Linux 3.17

2014-08-18 Thread Aaro Koskinen
In the series of ever-lasting ugly #ifdefs trying make out-of-tree drivers to support latest mainline kernel... Signed-off-by: Aaro Koskinen --- lib/librte_eal/linuxapp/kni/kni_misc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/

[dpdk-dev] [PATCH] kni: fix build for debian kernel 3.2

2014-04-30 Thread Thomas Monjalon
2014-04-29 10:23, Neil Horman: > On Tue, Apr 29, 2014 at 04:16:03PM +0200, David Marchand wrote: > > Following debian kernel headers upgrade to 3.2.57-3, pci capability > > accessors have been backported (upstream commit > > 8c0d3a02c1309eb6112d2e7c8172e8ceb26ecfca, ("PCI: Add accessors for PCI > >

[dpdk-dev] [PATCH] kni: fix build for debian kernel 3.2

2014-04-29 Thread David Marchand
Following debian kernel headers upgrade to 3.2.57-3, pci capability accessors have been backported (upstream commit 8c0d3a02c1309eb6112d2e7c8172e8ceb26ecfca, ("PCI: Add accessors for PCI Express Capability", v3.7-rc1)). It results in the same compilation error as redhat 6.x. However, there is no c

[dpdk-dev] [PATCH] kni: fix build for debian kernel 3.2

2014-04-29 Thread Neil Horman
On Tue, Apr 29, 2014 at 04:16:03PM +0200, David Marchand wrote: > Following debian kernel headers upgrade to 3.2.57-3, pci capability accessors > have been backported (upstream commit > 8c0d3a02c1309eb6112d2e7c8172e8ceb26ecfca, > ("PCI: Add accessors for PCI Express Capability", v3.7-rc1)). > > I

[dpdk-dev] [PATCH] kni: fix build with kernel 3.15

2014-04-17 Thread Thomas Monjalon
2014-04-14 16:50, Aaro Koskinen: > rxhash has been renamed to hash. In 3.14 and newer, we can use > skb_set_hash(). > > Signed-off-by: Aaro Koskinen Acked-by: Thomas Monjalon It's applied for version 1.6.0r2. Thanks -- Thomas

[dpdk-dev] [PATCH] kni: fix build with kernel 3.15

2014-04-14 Thread Aaro Koskinen
rxhash has been renamed to hash. In 3.14 and newer, we can use skb_set_hash(). Signed-off-by: Aaro Koskinen --- lib/librte_eal/linuxapp/kni/ethtool/igb/igb_main.c | 3 ++- lib/librte_eal/linuxapp/kni/ethtool/igb/kcompat.h | 11 +++ 2 files changed, 13 insertions(+), 1 deletion(-) diff

[dpdk-dev] [PATCH] kni: fix build with kernel < 3.3 with netdev_features_t backport

2013-07-26 Thread Thomas Monjalon
26/07/2013 13:25, Adrien Mazarguil : > The netdev_features_t typedef appeared in Linux 3.3, but checking the > kernel version isn't enough with some distributions (such as Debian > Wheezy) that backported it into 3.2, causing a compilation failure due to > redefinition. > > Since the presence of a

[dpdk-dev] [PATCH] kni: fix build with 802.1p kernel support

2013-07-26 Thread Thomas Monjalon
26/07/2013 14:12, Adrien Mazarguil : > On Fri, Jul 26, 2013 at 11:51:15AM +0200, Thomas Monjalon wrote: > > C90 compilers forbid mixed declaration and code. > > > > Signed-off-by: Thomas Monjalon > > --- > > > > lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c |7 +-- > > 1 file c

[dpdk-dev] [PATCH] kni: fix build with 802.1p kernel support

2013-07-26 Thread Adrien Mazarguil
On Fri, Jul 26, 2013 at 11:51:15AM +0200, Thomas Monjalon wrote: > C90 compilers forbid mixed declaration and code. > > Signed-off-by: Thomas Monjalon > --- > lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c |7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) Acked-by: Adrie

[dpdk-dev] [PATCH] kni: fix build with kernel < 3.3 with netdev_features_t backport

2013-07-26 Thread Adrien Mazarguil
The netdev_features_t typedef appeared in Linux 3.3, but checking the kernel version isn't enough with some distributions (such as Debian Wheezy) that backported it into 3.2, causing a compilation failure due to redefinition. Since the presence of a typedef can't be tested at compile time, this co

[dpdk-dev] [PATCH] kni: fix build with 802.1p kernel support

2013-07-26 Thread Thomas Monjalon
C90 compilers forbid mixed declaration and code. Signed-off-by: Thomas Monjalon --- lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_main.c b/lib/librte_eal/linuxa