[dpdk-dev] BUG - KNI broken in 4.2 kernel
On Thu, Aug 27, 2015 at 10:45:43AM -0700, Stephen Hemminger wrote: > On Thu, 27 Aug 2015 15:56:16 + > "Zhang, Helin" wrote: > > > Based on my experience, only one or two users asked for ethtool support, > > then we have it. Before that time, we don?t have KNI ethtool support. > > I did not mean who uses KNI does not care about it, I mean for those users > > who don?t use KNI, they shouldn?t be bothered by the KNI compilation > > issues. That?s why I was thinking if we can disable it by default, but not > > remove it. > > ? > > Regards, > > Helin > > Can KNI instead use DPDK hooks to provide generic ethtool semantics. > That way it would work with all hardware. Hi Stephen, by this you mean that it's a generic library/kernel driver that acts as a proxy to make calls into the ethdev library, rather than driver-specific calls? If so, that's an idea that should be well worth pursuing. If it's something else you have in mind, please clarify. Thanks, /Bruce
[dpdk-dev] BUG - KNI broken in 4.2 kernel
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Thursday, August 27, 2015 10:46 AM > To: Zhang, Helin > Cc: Jay Rolette; dev at dpdk.org > Subject: Re: [dpdk-dev] BUG - KNI broken in 4.2 kernel > > On Thu, 27 Aug 2015 15:56:16 + > "Zhang, Helin" wrote: > > > Based on my experience, only one or two users asked for ethtool support, > > then > we have it. Before that time, we don?t have KNI ethtool support. > > I did not mean who uses KNI does not care about it, I mean for those users > > who > don?t use KNI, they shouldn?t be bothered by the KNI compilation issues. > That?s why I was thinking if we can disable it by default, but not remove it. > > > > Regards, > > Helin > > Can KNI instead use DPDK hooks to provide generic ethtool semantics. > That way it would work with all hardware. I am not sure if I understood your idea correctly. My personal point is that the best KNI ethtool support should be processed in user space application. Sure if we want to support standard Linux ethtool commands, the KNI kernel driver should have a mechanism to work together with user space application. Is that what you said of hooks? But this may need huge efforts on it. Sure ideas on ethtool support improvements are always welcome! Regards, Helin
[dpdk-dev] BUG - KNI broken in 4.2 kernel
From: Jay Rolette [mailto:role...@infiniteio.com] Sent: Thursday, August 27, 2015 8:49 AM To: Zhang, Helin Cc: Stephen Hemminger; dev at dpdk.org Subject: Re: [dpdk-dev] BUG - KNI broken in 4.2 kernel On Thu, Aug 27, 2015 at 10:23 AM, Zhang, Helin mailto:helin.zhang at intel.com>> wrote: > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org<mailto:stephen > at networkplumber.org>] > Sent: Wednesday, August 26, 2015 5:15 PM > To: dev at dpdk.org<mailto:dev at dpdk.org>; Zhang, Helin > Subject: BUG - KNI broken in 4.2 kernel > > The network device ops handles changed again. > > Does KNI really need to keep yet another copy of the Intel driver code. Yes, it is a bit ugly. But there is no better way till now, as some of users want to have the functionality of KNI ethtool though limited now. I have an idea that is to disable KNI ethtool support by default, as most of users may not care about it. Then these users will not be bothered by these type of issues. Do you know of anyone that uses KNI that doesn't care about it? Since KNI presents as a normal network interface, it really needs to support the normal Linux commands (ifconfig, ethtool, ...) Based on my experience, only one or two users asked for ethtool support, then we have it. Before that time, we don?t have KNI ethtool support. I did not mean who uses KNI does not care about it, I mean for those users who don?t use KNI, they shouldn?t be bothered by the KNI compilation issues. That?s why I was thinking if we can disable it by default, but not remove it. Regards, Helin Jay For those users who are using KNI ethtool support, they need to fix the issues on any new versions of kernel or similar. Any good ideas or comments? Helin > There already are 4 versions: > 1. Out-of tree base driver > 2. In-kernel mainline Linux driver > 3. DPDK driver > 4. KNI DPDK driver > > No wonder they can't stay in sync.
[dpdk-dev] BUG - KNI broken in 4.2 kernel
On Thu, 27 Aug 2015 15:56:16 + "Zhang, Helin" wrote: > Based on my experience, only one or two users asked for ethtool support, then > we have it. Before that time, we don?t have KNI ethtool support. > I did not mean who uses KNI does not care about it, I mean for those users > who don?t use KNI, they shouldn?t be bothered by the KNI compilation issues. > That?s why I was thinking if we can disable it by default, but not remove it. > ? > Regards, > Helin Can KNI instead use DPDK hooks to provide generic ethtool semantics. That way it would work with all hardware.
[dpdk-dev] BUG - KNI broken in 4.2 kernel
2015-08-26 17:15, Stephen Hemminger: > The network device ops handles changed again. > > Does KNI really need to keep yet another copy of the Intel driver code. > There already are 4 versions: > 1. Out-of tree base driver > 2. In-kernel mainline Linux driver > 3. DPDK driver > 4. KNI DPDK driver You forgot the BSD drivers and probably others. > No wonder they can't stay in sync. A patch to remove igb/ixgbe drivers from KNI would be welcome.
[dpdk-dev] BUG - KNI broken in 4.2 kernel
The network device ops handles changed again. Does KNI really need to keep yet another copy of the Intel driver code. There already are 4 versions: 1. Out-of tree base driver 2. In-kernel mainline Linux driver 3. DPDK driver 4. KNI DPDK driver No wonder they can't stay in sync.