Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-15 Thread Jiri Kosina
On Sat, 13 Aug 2016, Cong Wang wrote: > > How about we actually extend a little bit the TCQ_F_BUILTIN special case > > test in qdisc_match_from_root()? > > > > After the change, the only way how qdisc_dev() could be NULL should be a > > TCQ_F_BUILTIN case, right? > > > > I was thinking about somet

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-14 Thread Cong Wang
On Fri, Aug 12, 2016 at 6:53 AM, Jiri Kosina wrote: > On Fri, 12 Aug 2016, Daniel Borkmann wrote: > >> This results in below panic. Tested reverting this patch and it fixes >> the panic. >> >> Did you test this also with ingress or clsact qdisc (just try adding >> it to lo dev for example) ? > > H

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-12 Thread Daniel Borkmann
On 08/12/2016 04:36 PM, Jiri Kosina wrote: On Fri, 12 Aug 2016, Daniel Borkmann wrote: I was thinking about something like the patch below (the reasong being that ->dev would be NULL only in cases of singletonish qdiscs) ... wouldn't that also fix the issue you're seeing? Have to think it throu

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-12 Thread Jiri Kosina
On Fri, 12 Aug 2016, Daniel Borkmann wrote: > > I was thinking about something like the patch below (the reasong being > > that ->dev would be NULL only in cases of singletonish qdiscs) ... > > wouldn't that also fix the issue you're seeing? Have to think it > > through a little bit more .. >

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-12 Thread Daniel Borkmann
On 08/12/2016 03:53 PM, Jiri Kosina wrote: On Fri, 12 Aug 2016, Daniel Borkmann wrote: This results in below panic. Tested reverting this patch and it fixes the panic. Did you test this also with ingress or clsact qdisc (just try adding it to lo dev for example) ? Hi Daniel, thanks for the

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-12 Thread Jiri Kosina
On Fri, 12 Aug 2016, Daniel Borkmann wrote: > This results in below panic. Tested reverting this patch and it fixes > the panic. > > Did you test this also with ingress or clsact qdisc (just try adding > it to lo dev for example) ? Hi Daniel, thanks for the report. Hmm, I am pretty sure clsact

Re: [PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-12 Thread Daniel Borkmann
Hi Jiri, On 08/10/2016 11:05 AM, Jiri Kosina wrote: From: Jiri Kosina Convert the per-device linked list into a hashtable. The primary motivation for this change is that currently, we're not tracking all the qdiscs in hierarchy (e.g. excluding default qdiscs), as the lookup performed over the

[PATCH 2/2] net: sched: convert qdisc linked list to hashtable

2016-08-10 Thread Jiri Kosina
From: Jiri Kosina Convert the per-device linked list into a hashtable. The primary motivation for this change is that currently, we're not tracking all the qdiscs in hierarchy (e.g. excluding default qdiscs), as the lookup performed over the linked list by qdisc_match_from_root() is rather ex