Re: new CBD Patch and Bugs

[Shaun Savage]

OKUJI Yoshinori wrote:
Here is an attached patch file for CBDCBD


> 
> > Also in the documentation disk_read_hook needs to be set.  In the docs
> > it states debug_fs.
> > Also the read_func needs to return a zero 0 if (len == 0)
> 
> What does this mean?

When I was reading the menu out of the CBD filesystem the program would
hang.  The reason was that even with zero length read request I returned
a '1'.  Then the upper layers would ask for more data length zero.  If I
returned a 0 then the upper layers knew it was finished.

> 1>  Now I am going to work on adding dynamic values to kernel
> command-line.  Multi boot OS have modules that can be loaded but linux
> does not.  But I would like to pass information fron the GRUB to the
> kernel.
>
>What does this mean?

The CBD has different partitions on the on the whole device. I only look
at the first partition for booting information.  BUt I have to search
the whole device to find the correct partition to use.  (Thats needed to
create reliable partition updates) The the CBD driver in the linux
kernel searches again.  I would like to pass the partitions start
sections to the kernel, so the search is only done once. This will speed
up the boot time.   

> > 2>  I also am going to work on making the menu/boot screen fancy
> > (consumer eye candy).
> 
> Note that I have a strong objection to that. It is your freedom to
> implement that in your own version, but I wouldn't agree to
> integrating that with the official version.
> 
LILO has the ability to display fancy screens, BUT grub is better
technically!  If you want I can create my own version, but if there are
some hooks that could be added in the offical version that would make
(eye candy) easier would you agree with that?

Shaun

cbd-2.4.0-1.patch.bz2
Description: Binary data