Dave Miner wrote: > Sanjay Nadkarni wrote: >> The changes enable booting Indiana under a Xvm domu. >> Location: >> http://cr.opensolaris.org/~nadkarni/listcd/ >> > > listcd.c > > 29: while we're in here, can we add a comment on what the output > format looks like so somebody doesn't mess it up in the next fix? > > 55 optimize > > 58 it's > > 66: a comment here would be nice Ok with all the above > 106: this ought to be inside the block at 108 > I am inclined to keep it here since the next two statement describe the action i.e, stripping out the :raw and printing. > 124: probably a question for moinak, but why do we need the sync()? > Can be removed.
-Sanjay > Dave
