Re: [Qemu-devel] [PATCHv3] Load "bootsplash.jpg" if present

[Anthony Liguori]

On 08/02/2010 11:47 AM, Kevin O'Connor wrote:
> On Mon, Aug 02, 2010 at 11:33:33AM -0500, Anthony Liguori wrote:
>    
> > On 08/02/2010 11:11 AM, Kevin O'Connor wrote:
> >      
> > > Load the "bootsplash.jpg" file into fw_cfg if it is found in the roms
> > > directory.
> > >        
> > Sorry, I should have provided this in the first response.
> >
> > Does the bootsplash cause a delay in startup time?  If so, we'll
> > want to be able to disable this at boot time via the -boot option.
> > If it doesn't, this patch is probably fine as is.
> >      
> That's okay - the patch is intended to start discussion anyway.
>
> There is no added delay for the bootsplash.  However, it does take
> time to decompress it.  On my machine it can take between 200-300ms
> for the jpeg code.
>
> It may make more sense for SeaBIOS to only bother with the bootsplash
> if the boot menu is enabled (or some new option) - as otherwise the
> picture is basically cleared immiediately after it's shown.
>    

Yeah, if there's no boot menu and no boot menu delay, I imagine the boot 
splash ends up just being a blip on the screen, right?

We definitely want a "boot as fast as you possibly can" mode for the 
BIOS although for non-developer environments, I think a boot splash is a 
nice touch.

BTW, we need to document somewhere any assumptions SeaBIOS has about the 
JPEG.  I see that it expects a 1024x768 image.  Any additional 
restrictions on the jpeg image?

Regards,

Anthony Liguori

> -Kevin
>