[PATCH] remove usage of g_assert() in blockdev

Kay Sievers kay.sievers at vrfy.org
Tue Nov 14 13:44:55 PST 2006


On 11/14/06, Mark Rosenstand <mark at borkware.net> wrote:
> 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?

No, coldplug makes only sense after the rootfs is mounted.

Kay


More information about the hal mailing list