DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9
@ 2018-12-21  6:03 David Zeng
  2018-12-21 14:34 ` Ferruh Yigit
  2018-12-21 16:15 ` [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT David Zeng
  0 siblings, 2 replies; 9+ messages in thread
From: David Zeng @ 2018-12-21  6:03 UTC (permalink / raw)
  To: dev; +Cc: David Zeng

Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>
---
 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 @@
 #undef NET_NAME_UNKNOWN
 #endif
 
+/*
+ * RHEL has two different version with different kernel version: 
+ * 3.10 is for AMD, Intel, IBM POWER7 and POWER8;
+ * 4.14 is for ARM and IBM POWER9
+ */
 #if (defined(RHEL_RELEASE_CODE) && \
-	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)))
+	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
+	(LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
 #define ndo_change_mtu ndo_change_mtu_rh74
 #endif
 
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9
  2018-12-21  6:03 [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9 David Zeng
@ 2018-12-21 14:34 ` Ferruh Yigit
  2018-12-21 16:15 ` [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT David Zeng
  1 sibling, 0 replies; 9+ messages in thread
From: Ferruh Yigit @ 2018-12-21 14:34 UTC (permalink / raw)
  To: David Zeng, dev; +Cc: David Zeng

On 12/21/2018 6:03 AM, David Zeng wrote:
> Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>
> ---
>  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 @@
>  #undef NET_NAME_UNKNOWN
>  #endif
>  
> +/*
> + * RHEL has two different version with different kernel version: 
> + * 3.10 is for AMD, Intel, IBM POWER7 and POWER8;
> + * 4.14 is for ARM and IBM POWER9
> + */
>  #if (defined(RHEL_RELEASE_CODE) && \
> -	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)))
> +	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
> +	(LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
>  #define ndo_change_mtu ndo_change_mtu_rh74
>  #endif

Do you need to update 'kernel/linux/kni/ethtool/igb/kcompat.h' for same check?

Also there is another update on same line for RHEL8 fix [1], if you can rebase
on top of it makes maintainers life easier.

[1] https://patches.dpdk.org/patch/49104/

Thanks,
ferruh

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

* [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21  6:03 [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9 David Zeng
  2018-12-21 14:34 ` Ferruh Yigit
@ 2018-12-21 16:15 ` David Zeng
  2018-12-21 16:15   ` David Zeng
  1 sibling, 1 reply; 9+ messages in thread
From: David Zeng @ 2018-12-21 16:15 UTC (permalink / raw)
  To: dev


Thanks, Ferruh!

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

* [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 16:15 ` [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT David Zeng
@ 2018-12-21 16:15   ` David Zeng
  2018-12-21 16:27     ` [dpdk-dev] " David Zeng
  2018-12-21 16:27     ` Ferruh Yigit
  0 siblings, 2 replies; 9+ messages in thread
From: David Zeng @ 2018-12-21 16:15 UTC (permalink / raw)
  To: dev; +Cc: David Zeng

Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>
---
 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/linux/kni/compat.h
+++ b/kernel/linux/kni/compat.h
@@ -102,8 +102,15 @@
 #undef NET_NAME_UNKNOWN
 #endif
 
+/*
+ * RHEL has two different version with different kernel version:
+ * 3.10 is for AMD, Intel, IBM POWER7 and POWER8;
+ * 4.14 is for ARM and IBM POWER9
+ */
 #if (defined(RHEL_RELEASE_CODE) && \
-	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)))
+	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
+	(RHEL_RELEASE_CODE < RHEL_RELEASE_VERSION(8, 0)) && \
+	(LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
 #define ndo_change_mtu ndo_change_mtu_rh74
 #endif
 
diff --git a/kernel/linux/kni/ethtool/igb/kcompat.h b/kernel/linux/kni/ethtool/igb/kcompat.h
index ae1b530..3c0c8e3 100644
--- a/kernel/linux/kni/ethtool/igb/kcompat.h
+++ b/kernel/linux/kni/ethtool/igb/kcompat.h
@@ -3930,7 +3930,9 @@ static inline struct sk_buff *__kc__vlan_hwaccel_put_tag(struct sk_buff *skb,
 #endif
 
 #if (defined(RHEL_RELEASE_CODE) && \
-	(RHEL_RELEASE_VERSION(7, 5) <= RHEL_RELEASE_CODE))
+	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
+        (RHEL_RELEASE_CODE < RHEL_RELEASE_VERSION(8, 0)) && \
+        (LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
 #define ndo_change_mtu ndo_change_mtu_rh74
 #endif
 
-- 
1.8.3.1

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

* [dpdk-dev] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 16:15   ` David Zeng
@ 2018-12-21 16:27     ` David Zeng
  2018-12-21 16:27       ` [dpdk-dev] [PATCH] " David Zeng
  2018-12-21 16:27     ` Ferruh Yigit
  1 sibling, 1 reply; 9+ messages in thread
From: David Zeng @ 2018-12-21 16:27 UTC (permalink / raw)
  To: dev

Fix the coding style.

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

* [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 16:27     ` [dpdk-dev] " David Zeng
@ 2018-12-21 16:27       ` David Zeng
  2018-12-21 17:02         ` Ferruh Yigit
  0 siblings, 1 reply; 9+ messages in thread
From: David Zeng @ 2018-12-21 16:27 UTC (permalink / raw)
  To: dev; +Cc: David Zeng

Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>
---
 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/linux/kni/compat.h
+++ b/kernel/linux/kni/compat.h
@@ -102,8 +102,15 @@
 #undef NET_NAME_UNKNOWN
 #endif
 
+/*
+ * RHEL has two different version with different kernel version:
+ * 3.10 is for AMD, Intel, IBM POWER7 and POWER8;
+ * 4.14 is for ARM and IBM POWER9
+ */
 #if (defined(RHEL_RELEASE_CODE) && \
-	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)))
+	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
+	(RHEL_RELEASE_CODE < RHEL_RELEASE_VERSION(8, 0)) && \
+	(LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
 #define ndo_change_mtu ndo_change_mtu_rh74
 #endif
 
diff --git a/kernel/linux/kni/ethtool/igb/kcompat.h b/kernel/linux/kni/ethtool/igb/kcompat.h
index ae1b530..430aaba 100644
--- a/kernel/linux/kni/ethtool/igb/kcompat.h
+++ b/kernel/linux/kni/ethtool/igb/kcompat.h
@@ -3930,7 +3930,9 @@ static inline struct sk_buff *__kc__vlan_hwaccel_put_tag(struct sk_buff *skb,
 #endif
 
 #if (defined(RHEL_RELEASE_CODE) && \
-	(RHEL_RELEASE_VERSION(7, 5) <= RHEL_RELEASE_CODE))
+	(RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(7, 5)) && \
+	(RHEL_RELEASE_CODE < RHEL_RELEASE_VERSION(8, 0)) && \
+	(LINUX_VERSION_CODE < KERNEL_VERSION(4, 14, 0)))
 #define ndo_change_mtu ndo_change_mtu_rh74
 #endif
 
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 16:15   ` David Zeng
  2018-12-21 16:27     ` [dpdk-dev] " David Zeng
@ 2018-12-21 16:27     ` Ferruh Yigit
  1 sibling, 0 replies; 9+ messages in thread
From: Ferruh Yigit @ 2018-12-21 16:27 UTC (permalink / raw)
  To: David Zeng, dev; +Cc: David Zeng

On 12/21/2018 4:15 PM, David Zeng wrote:
> Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

* Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 16:27       ` [dpdk-dev] [PATCH] " David Zeng
@ 2018-12-21 17:02         ` Ferruh Yigit
  2019-01-14 13:34           ` Thomas Monjalon
  0 siblings, 1 reply; 9+ messages in thread
From: Ferruh Yigit @ 2018-12-21 17:02 UTC (permalink / raw)
  To: David Zeng, dev; +Cc: David Zeng

On 12/21/2018 4:27 PM, David Zeng wrote:
> Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

* Re: [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT
  2018-12-21 17:02         ` Ferruh Yigit
@ 2019-01-14 13:34           ` Thomas Monjalon
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas Monjalon @ 2019-01-14 13:34 UTC (permalink / raw)
  To: David Zeng; +Cc: dev, Ferruh Yigit, David Zeng

21/12/2018 18:02, Ferruh Yigit:
> On 12/21/2018 4:27 PM, David Zeng wrote:
> > Signed-off-by: David Zeng <zengxhsh@cn.ibm.com>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

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

end of thread, other threads:[~2019-01-14 13:34 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-21  6:03 [dpdk-dev] [PATCH] kni: fix build on RHEL 7.6-ALT on IBM POWER9 David Zeng
2018-12-21 14:34 ` Ferruh Yigit
2018-12-21 16:15 ` [dpdk-dev] [PATCH] kni: fix build on RHEL8 and RHEL7.6-ALT David Zeng
2018-12-21 16:15   ` David Zeng
2018-12-21 16:27     ` [dpdk-dev] " David Zeng
2018-12-21 16:27       ` [dpdk-dev] [PATCH] " David Zeng
2018-12-21 17:02         ` Ferruh Yigit
2019-01-14 13:34           ` Thomas Monjalon
2018-12-21 16:27     ` Ferruh Yigit

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).