Please review & test this

[Vlad GALU]

On 6/11/05, Vlad GALU <vladgalu at gmail.com> wrote:
> On 6/10/05, Vlad GALU <vladgalu at gmail.com> wrote:
> > On 6/10/05, Vlad GALU <vladgalu at gmail.com> wrote:
> > >         As you may all know, the packet classifier in ALTQ is very
> > > slow on large numbers of classes, because it stores them linearly, in
> > > an array. I rewrote the way classes are stored, replacing the array
> > > with a hash table. I tested [1] on a system with about 8000 classes
> > > and noticed a remarkable performance difference (the system went from
> > > almost unusable to nice & smooth). It breaks the ABI by adding an
> > > extra TAILQ_ENTRY  member to the HFSC class structure, though.
> >
> >   And also replaces the class array in struct hfsc_if with the hash table.
> 
>  Bummer. Increasing the number of classes led to locking the machine
> up. Something slipped my eye: the filters are also linearly searched
> for. I'll try to do something about them as well and come back with
> something.
> 

  Duhhh, I need more coffee. I didn't notice the ALTQ3_COMPAT ifdef in
the source. Luckily, the patch in my first mail deals with it as well.
The classifying is entirely done by pf, so I guess it only depends on
how (un)optimized your ruleset is.


> >
> > > If anyone reviews and tests it, I would be grateful.
> > >
> > > [1] http://night.rdslink.ro/dudu/altq/altq_hfschash.diff
> > >
> > > P.S. please keep in mind that I'm not exactly a black belt in kernel
> > > programming, so glitches might exist. I would be most happy to hear
> > > some suggestions.
> > >
> > > --
> > > If it's there, and you can see it, it's real.
> > > If it's not there, and you can see it, it's virtual.
> > > If it's there, and you can't see it, it's transparent.
> > > If it's not there, and you can't see it, you erased it.
> > >
> >
> >
> > --
> > If it's there, and you can see it, it's real.
> > If it's not there, and you can see it, it's virtual.
> > If it's there, and you can't see it, it's transparent.
> > If it's not there, and you can't see it, you erased it.
> >
> 
> 
> --
> If it's there, and you can see it, it's real.
> If it's not there, and you can see it, it's virtual.
> If it's there, and you can't see it, it's transparent.
> If it's not there, and you can't see it, you erased it.
> 


-- 
If it's there, and you can see it, it's real.
If it's not there, and you can see it, it's virtual.
If it's there, and you can't see it, it's transparent.
If it's not there, and you can't see it, you erased it.