[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-13 Thread Ferruh Yigit
On 9/11/2016 10:59 AM, zhouyangchao wrote: > Signed-off-by: zhouyangchao > --- There is a typo in the patch subject, I suggest: kni: fix error rollback kernel crash Fixes: 9c61145ff6f9 ("kni: allow multiple threads") Acked-by: Ferruh Yigit

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-11 Thread zhouyangchao
Signed-off-by: zhouyangchao --- lib/librte_eal/linuxapp/kni/kni_misc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index 67e9b7d..5c58a83 100644 --- a/lib/librte_eal/linuxapp/kni/kni_misc.c +++

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-10 Thread Thomas Monjalon
2016-09-09 14:33, Mcnamara, John: > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > 2016-09-08 18:15, Ferruh Yigit: > > > On 9/8/2016 5:44 PM, Stephen Hemminger wrote: > > > > > > ... > > > > > > > But then again the whole KNI driver fails completely when running > > >

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-09 Thread Thomas Monjalon
2016-09-08 18:15, Ferruh Yigit: > On 9/8/2016 5:44 PM, Stephen Hemminger wrote: > > ... > > > But then again the whole KNI driver fails completely when > > running kernel style check. > > > > Yes, it generates lots of warnings. > I can fix them (excluding ethtool/*), that wouldn't take much

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-09 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Friday, September 9, 2016 1:40 PM > To: Yigit, Ferruh > Cc: Stephen Hemminger ; zhouyangchao > ; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] kni: unre

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-09 Thread zhouyangchao
Signed-off-by: zhouyangchao --- lib/librte_eal/linuxapp/kni/kni_misc.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index 67e9b7d..ad4e603 100644 ---

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-09 Thread zhouyangchao
Signed-off-by: zhouyangchao --- lib/librte_eal/linuxapp/kni/kni_misc.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index 67e9b7d..17b6d7a 100644 ---

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-08 Thread Ferruh Yigit
On 9/8/2016 5:44 PM, Stephen Hemminger wrote: ... > But then again the whole KNI driver fails completely when > running kernel style check. > Yes, it generates lots of warnings. I can fix them (excluding ethtool/*), that wouldn't take much time but how syntax only patches welcomed? Another

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-08 Thread Ferruh Yigit
On 9/9/2016 3:46 AM, zhouyangchao wrote: > Signed-off-by: zhouyangchao > --- > lib/librte_eal/linuxapp/kni/kni_misc.c |4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c > b/lib/librte_eal/linuxapp/kni/kni_misc.c > index

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-08 Thread Stephen Hemminger
On Fri, 9 Sep 2016 10:42:16 +0800 zhouyangchao wrote: > Signed-off-by: zhouyangchao > --- > lib/librte_eal/linuxapp/kni/kni_misc.c |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c >

[dpdk-dev] [PATCH] kni: unregister an unregisterd net_device could cause a kernel crash

2016-09-08 Thread Stephen Hemminger
On Fri, 9 Sep 2016 10:46:07 +0800 zhouyangchao wrote: > Signed-off-by: zhouyangchao > --- > lib/librte_eal/linuxapp/kni/kni_misc.c |4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c >