On Tue, Jun 29, 2010 at 01:58:27PM -0700, Keith Mitchell wrote:
> The webrev has been updated in place with my proposed fix:
> http://cr.opensolaris.org/~kemitche/16400/
The fix looks OK, but now I think about it, it's relying on some unusual
code in lofiadm's wait_until_dev_complete():
311 /* First use di_devlink_init() */
312 if (hdl = di_devlink_init("lofi", DI_MAKE_LINK)) {
313 (void) di_devlink_fini(&hdl);
314 goto out;
315 }
I've only just noticed that lofiadm tries to create links itself, which
is pretty weird, and now you'll be relying on it.
I guess it's not worth it since it's a different gate, but this piece of
lofiadm code could really do with a comment as well stating why the code
is there (now).
Longer term it might be better for lofiadm to call out to devfsadm -i
lofi on some trigger (an env var??)
regards
john
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
