[PATCH] remove usage of g_assert() in blockdev

Mark Rosenstand mark at borkware.net
Tue Nov 14 12:34:54 PST 2006


On Tue, 2006-11-14 at 18:03 +0100, Kay Sievers wrote:
> On 11/13/06, David Zeuthen <david at fubar.dk> wrote:
> > Going forward we probably want to rethink how we do coldplug and ensure
> > it's synchronized with the rest of the system. Which probably means
> > recommending distributors to start HAL just before udev (without any
> > sysfs coldplug) and then relying on udev's coldplugging code to emit the
> > right uevents. But there are probably subtble issues to doing that;
> > you'd have to ensure that all entries that HAL care about in sysfs
> > actually triggers uevents. That's definitely not true today. Kay?
> 
> We can get all what we need from udev. The coldplug-code in udev and
> HAL is indentical. The reworked HAL coldplug (current git) is almost a
> plain copy of the udevtrigger (version 102) code.
> 
> But we would need to move to /lib/hal and /sbin with all the tools,
> because some (confused) people think that /usr should be on a
> different partition, or worse behind a different storage adapter or
> even on nfs. I'm totally convinced that this is crazy, but I'm almost
> never able to make my point with these guys. :)
> 
> > Starting HAL late and doing coldplug while other processes are loading
> > drivers (cause uevents to be triggered) is just too dangerous and leads
> > to bugs like these I think. Danny's fix is appropriate right now, it's
> > probably better not to crash and just have one device less than to
> > totally crash.
> 
> Right. We should give it a try to move HAL to the rootfs, and
> integrate it into the udev colplug process. It sounds like a nice
> option, even when we break some silly setups, it's probably worth to
> do it that way.

Does this mean that dbus and hal all have to be in initramfs for distros
that use a completely dynamic /dev? And have a system dbus session
started from there?



More information about the hal mailing list