From: Kevin O'Connor <kevin@koconnor.net>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCHv3] Load "bootsplash.jpg" if present
Date: Tue, 3 Aug 2010 21:07:12 -0400 [thread overview]
Message-ID: <20100804010712.GB8802@morn.localdomain> (raw)
In-Reply-To: <4C5829F8.30406@redhat.com>
On Tue, Aug 03, 2010 at 04:38:48PM +0200, Paolo Bonzini wrote:
> On 08/02/2010 06:47 PM, Kevin O'Connor wrote:
> >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.
>
> KVM or TCG?
TCG.
> Also, what's the time for a completely black bootsplash?
Here are the timings (times are in seconds measured on the host) for a
completely black 640x480 jpeg on my machine:
00.135: Checking for bootsplash
00.139: VESA 2.0
00.140: VENDOR: VGABIOS Cirrus extension
00.143: PRODUCT: VGABIOS Cirrus extension
00.145: Copying bootsplash.jpg
00.146: Decoding bootsplash.jpg
00.154: Finding vesa mode with dimensions 640/480
00.160: mode: 0111
00.162: framebuffer: 0xf0000000
00.164: bytes per scanline: 1280
00.165: bits per pixel: 16
00.166: Decompressing bootsplash.jpg
00.283: Switching to graphics mode
00.322: Showing bootsplash.jpg
00.327: Bootsplash copy complete
One can compare this output with the dprintf() calls in the code at:
http://git.linuxtogo.org/?p=kevin/seabios.git;a=blob;f=src/bootsplash.c;h=14bdd4cfaa6b931032e30236039b0fd4934f75ac;hb=eb6dc785475e4676de728f99a0fcd638d81c5b68#l144
The times weren't notably different with a real picture instead of
just a black picture.
As before, SeaBIOS should probably be changed so that it only shows
the image if the boot menu has been enabled.
-Kevin
prev parent reply other threads:[~2010-08-04 1:07 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-02 16:11 [Qemu-devel] [PATCHv3] Load "bootsplash.jpg" if present Kevin O'Connor
2010-08-02 16:33 ` Anthony Liguori
2010-08-02 16:47 ` Kevin O'Connor
2010-08-02 16:52 ` Anthony Liguori
2010-08-02 18:10 ` Kevin O'Connor
2010-08-02 18:54 ` Anthony Liguori
2010-08-03 9:52 ` Richard W.M. Jones
2010-08-03 12:51 ` Anthony Liguori
2010-08-03 12:54 ` Richard W.M. Jones
2010-08-03 13:04 ` Richard W.M. Jones
2010-08-03 13:33 ` Anthony Liguori
2010-08-03 14:38 ` [Qemu-devel] " Paolo Bonzini
2010-08-04 1:07 ` Kevin O'Connor [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100804010712.GB8802@morn.localdomain \
--to=kevin@koconnor.net \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).