qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

      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).