Re: [PATCH 0/2][v2] Add BIOS splash image support

[Anthony Liguori <anthony@codemonkey.ws>]

Laurent Vivier wrote:
> This series of patches adds a nice BIOS startup splash screen.
>
> It adds a "-splash" option allowing to specify the picture file name (a 640x480 (or less) and true color PNG) to display. You can enable/disable a fade in,
> fade out and the bootmenu. The time to display the image can be also given (in
> seconds).
>
> Idea stollen from VirtualBox.
>
> [PATCH 1/2] [BIOS] Add splash image support
> [PATCH 2/2] [QEMU] Add BIOS splash image
>   

So my general feeling here is that this code is well isolated and 
provides a feature that some people will find interesting.  I know I 
personally am interested in this feature.  I also think that a good 
number of vendors will want to use this mechanism to brand their 
virtualization offerings.

So I'm inclined to take this patch set unless there is objection.  I 
don't necessarily disagree with some of the feedback about making the 
BIOS not delay at all but I think that's an orthogonal discussion to 
this one.  The Press F12 thing also annoys me and I'd be happy to look 
at patches to make it optional.

Regards,

Anthony Liguori

> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>