[PATCH 0/8] Various large optimisations

[Danny Kukawka]

On Montag, 19. November 2007, Richard Hughes wrote:
> On Mon, 2007-11-19 at 11:55 -0500, Doug Klima wrote:
> > Rob Taylor wrote:
> > >  4) Allow hotplug events to process in parallel. This intruduces into
> > > HAL a similar parallel event processing method as udev. Along with 2),
> > > this removes the coldplug memory peak almost entirely (Cutting down to
> > > 350K on my intel hardware and about 150K on an N800)
>
> Be careful with this. I had big problems with this a few months ago with
> coldplug ordering issues - I'm sure the list has all the juicy details.

I would also assume we get (many) regressions with this change.

> > >  5) Add the ability in HalDeviceStore to index given properties by
> > > value. Profiling showed that about 20% of coldplug time was spend in
> > >  hal_device_store_match_key_value_string to look up devices by
> > > sysfs_path. This generic patch cuts each lookup to a couple of hash
> > > table lookups, versus n strcmps.
>
> Can't we just hardcode sysfs path?

How this? IMO we don't speak about /sys, but about the indivdual path of the 
devices.

Danny