qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Wayne Xia <xiawenc@linux.vnet.ibm.com>
Cc: qemu-devel@nongnu.org, paul@codesourcery.com
Subject: Re: [Qemu-devel] [PATCH v6] showing a splash picture when start
Date: Sat, 23 Jul 2011 10:53:25 -0500	[thread overview]
Message-ID: <4E2AEE75.2050807@codemonkey.ws> (raw)
In-Reply-To: <1310292588-1744-1-git-send-email-xiawenc@linux.vnet.ibm.com>

On 07/10/2011 05:09 AM, Wayne Xia wrote:
>      Added options to let qemu transfer two configuration files to bios:
> "bootsplash.bmp" and "etc/boot-menu-wait", which could be specified by command
>      -boot splash=P,splash-time=T
> P is jpg/bmp file name or an absolute path, T have a max value of 0xffff, unit
> is ms. With these two options, if user invoke qemu with menu=on option, then
> a splash picture would be showed in a given time. For example:
>      qemu -boot menu=on,splash=/root/boot.bmp,splash-time=5000
> would make boot.bmp shown as a brand with 5 seconds in the booting up process.
> This feature need the new seabios's support, which could be got from git.

Please include documentation in qemu-doc.texi including information on 
what image formats are supported and what restrictions are present (for 
instance 24-bit depth bitmaps with what resolution?).

Regards,

Anthony Liguori

>
> Signed-off-by: Wayne Xia<xiawenc@linux.vnet.ibm.com>
> ---
>   hw/fw_cfg.c   |  140 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
>   qemu-config.c |   27 +++++++++++
>   sysemu.h      |    3 +
>   vl.c          |   17 +++++++-
>   4 files changed, 185 insertions(+), 2 deletions(-)
>
> diff --git a/hw/fw_cfg.c b/hw/fw_cfg.c
> index 85c8c3c..434fc96 100644
> --- a/hw/fw_cfg.c
> +++ b/hw/fw_cfg.c
> @@ -26,6 +26,7 @@
>   #include "isa.h"
>   #include "fw_cfg.h"
>   #include "sysbus.h"
> +#include "qemu-error.h"
>
>   /* debug firmware config */
>   //#define DEBUG_FW_CFG
> @@ -56,6 +57,143 @@ struct FWCfgState {
>       Notifier machine_ready;
>   };
>
> +#define JPG_FILE 0
> +#define BMP_FILE 1
> +
> +static FILE *probe_splashfile(char *filename, int *file_sizep, int *file_typep)
> +{
> +    FILE *fp = NULL;
> +    int fop_ret;
> +    int file_size;
> +    int file_type = -1;
> +    unsigned char buf[2] = {0, 0};
> +    unsigned int filehead_value = 0;
> +    int bmp_bpp;
> +
> +    fp = fopen(filename, "rb");
> +    if (fp == NULL) {
> +        error_report("failed to open file '%s'.", filename);
> +        return fp;
> +    }
> +    /* check file size */
> +    fseek(fp, 0L, SEEK_END);
> +    file_size = ftell(fp);
> +    if (file_size<  2) {
> +        error_report("file size is less than 2 bytes '%s'.", filename);
> +        fclose(fp);
> +        fp = NULL;
> +        return fp;
> +    }
> +    /* check magic ID */
> +    fseek(fp, 0L, SEEK_SET);
> +    fop_ret = fread(buf, 1, 2, fp);
> +    filehead_value = (buf[0] + (buf[1]<<  8))&  0xffff;
> +    if (filehead_value == 0xd8ff) {
> +        file_type = JPG_FILE;
> +    } else {
> +        if (filehead_value == 0x4d42) {
> +            file_type = BMP_FILE;
> +        }
> +    }
> +    if (file_type<  0) {
> +        error_report("'%s' not jpg/bmp file,head:0x%x.",
> +                         filename, filehead_value);
> +        fclose(fp);
> +        fp = NULL;
> +        return fp;
> +    }
> +    /* check BMP bpp */
> +    if (file_type == BMP_FILE) {
> +        fseek(fp, 28, SEEK_SET);
> +        fop_ret = fread(buf, 1, 2, fp);
> +        bmp_bpp = (buf[0] + (buf[1]<<  8))&  0xffff;
> +        if (bmp_bpp != 24) {
> +            error_report("only 24bpp bmp file is supported.");
> +            fclose(fp);
> +            fp = NULL;
> +            return fp;
> +        }
> +    }
> +    /* return values */
> +    *file_sizep = file_size;
> +    *file_typep = file_type;
> +    return fp;
> +}
> +
> +static void fw_cfg_bootsplash(FWCfgState *s)
> +{
> +    int boot_splash_time = -1;
> +    const char *boot_splash_filename = NULL;
> +    char *p;
> +    char *filename;
> +    FILE *fp;
> +    int fop_ret;
> +    int file_size;
> +    int file_type = -1;
> +    const char *temp;
> +
> +    /* get user configuration */
> +    QemuOptsList *plist = qemu_find_opts("boot-opts");
> +    QemuOpts *opts = QTAILQ_FIRST(&plist->head);
> +    if (opts != NULL) {
> +        temp = qemu_opt_get(opts, "splash");
> +        if (temp != NULL) {
> +            boot_splash_filename = temp;
> +        }
> +        temp = qemu_opt_get(opts, "splash-time");
> +        if (temp != NULL) {
> +            p = (char *)temp;
> +            boot_splash_time = strtol(p, (char **)&p, 10);
> +        }
> +    }
> +
> +    /* insert splash time if user configurated */
> +    if (boot_splash_time>= 0) {
> +        /* validate the input */
> +        if (boot_splash_time>  0xffff) {
> +            error_report("splash time is big than 65535, force it to 65535.");
> +            boot_splash_time = 0xffff;
> +        }
> +        /* use little endian format */
> +        qemu_extra_params_fw[0] = (uint8_t)(boot_splash_time&  0xff);
> +        qemu_extra_params_fw[1] = (uint8_t)((boot_splash_time>>  8)&  0xff);
> +        fw_cfg_add_file(s, "etc/boot-menu-wait", qemu_extra_params_fw, 2);
> +    }
> +
> +    /* insert splash file if user configurated */
> +    if (boot_splash_filename != NULL) {
> +        filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, boot_splash_filename);
> +        if (filename == NULL) {
> +            error_report("failed to find file '%s'.", boot_splash_filename);
> +            return;
> +        }
> +        /* probing the file */
> +        fp = probe_splashfile(filename,&file_size,&file_type);
> +        if (fp == NULL) {
> +            qemu_free(filename);
> +            return;
> +        }
> +        /* loading file data */
> +        if (boot_splash_filedata != NULL) {
> +            qemu_free(boot_splash_filedata);
> +        }
> +        boot_splash_filedata = qemu_malloc(file_size);
> +        boot_splash_filedata_size = file_size;
> +        fseek(fp, 0L, SEEK_SET);
> +        fop_ret = fread(boot_splash_filedata, 1, file_size, fp);
> +        fclose(fp);
> +        /* insert data */
> +        if (file_type == JPG_FILE) {
> +            fw_cfg_add_file(s, "bootsplash.jpg",
> +                    boot_splash_filedata, boot_splash_filedata_size);
> +        } else {
> +            fw_cfg_add_file(s, "bootsplash.bmp",
> +                    boot_splash_filedata, boot_splash_filedata_size);
> +        }
> +        qemu_free(filename);
> +    }
> +}
> +
>   static void fw_cfg_write(FWCfgState *s, uint8_t value)
>   {
>       int arch = !!(s->cur_entry&  FW_CFG_ARCH_LOCAL);
> @@ -352,7 +490,7 @@ FWCfgState *fw_cfg_init(uint32_t ctl_port, uint32_t data_port,
>       fw_cfg_add_i16(s, FW_CFG_NB_CPUS, (uint16_t)smp_cpus);
>       fw_cfg_add_i16(s, FW_CFG_MAX_CPUS, (uint16_t)max_cpus);
>       fw_cfg_add_i16(s, FW_CFG_BOOT_MENU, (uint16_t)boot_menu);
> -
> +    fw_cfg_bootsplash(s);
>
>       s->machine_ready.notify = fw_cfg_machine_ready;
>       qemu_add_machine_init_done_notifier(&s->machine_ready);
> diff --git a/qemu-config.c b/qemu-config.c
> index c63741c..5b0b4a6 100644
> --- a/qemu-config.c
> +++ b/qemu-config.c
> @@ -469,6 +469,32 @@ static QemuOptsList qemu_machine_opts = {
>       },
>   };
>
> +QemuOptsList qemu_boot_opts = {
> +    .name = "boot-opts",
> +    .head = QTAILQ_HEAD_INITIALIZER(qemu_boot_opts.head),
> +    .desc = {
> +        /* the three names below are not used now */
> +        {
> +            .name = "order",
> +            .type = QEMU_OPT_STRING,
> +        }, {
> +            .name = "once",
> +            .type = QEMU_OPT_STRING,
> +        }, {
> +            .name = "menu",
> +            .type = QEMU_OPT_STRING,
> +        /* following are really used */
> +        }, {
> +            .name = "splash",
> +            .type = QEMU_OPT_STRING,
> +        }, {
> +            .name = "splash-time",
> +            .type = QEMU_OPT_STRING,
> +        },
> +        { /*End of list */ }
> +    },
> +};
> +
>   static QemuOptsList *vm_config_groups[32] = {
>       &qemu_drive_opts,
>       &qemu_chardev_opts,
> @@ -484,6 +510,7 @@ static QemuOptsList *vm_config_groups[32] = {
>   #endif
>       &qemu_option_rom_opts,
>       &qemu_machine_opts,
> +&qemu_boot_opts,
>       NULL,
>   };
>
> diff --git a/sysemu.h b/sysemu.h
> index d3013f5..bd830e5 100644
> --- a/sysemu.h
> +++ b/sysemu.h
> @@ -123,6 +123,9 @@ extern int no_shutdown;
>   extern int semihosting_enabled;
>   extern int old_param;
>   extern int boot_menu;
> +extern uint8_t *boot_splash_filedata;
> +extern int boot_splash_filedata_size;
> +extern uint8_t qemu_extra_params_fw[2];
>   extern QEMUClock *rtc_clock;
>
>   #define MAX_NODES 64
> diff --git a/vl.c b/vl.c
> index fcd7395..07293e0 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -228,6 +228,9 @@ int ctrl_grab = 0;
>   unsigned int nb_prom_envs = 0;
>   const char *prom_envs[MAX_PROM_ENVS];
>   int boot_menu;
> +uint8_t *boot_splash_filedata;
> +int boot_splash_filedata_size;
> +uint8_t qemu_extra_params_fw[2];
>
>   typedef struct FWBootEntry FWBootEntry;
>
> @@ -293,6 +296,14 @@ static struct {
>       { .driver = "qxl-vga",              .flag =&default_vga       },
>   };
>
> +static void res_free(void)
> +{
> +    if (boot_splash_filedata != NULL) {
> +        qemu_free(boot_splash_filedata);
> +        boot_splash_filedata = NULL;
> +    }
> +}
> +
>   static int default_driver_check(QemuOpts *opts, void *opaque)
>   {
>       const char *driver = qemu_opt_get(opts, "driver");
> @@ -2323,7 +2334,8 @@ int main(int argc, char **argv, char **envp)
>               case QEMU_OPTION_boot:
>                   {
>                       static const char * const params[] = {
> -                        "order", "once", "menu", NULL
> +                        "order", "once", "menu",
> +                        "splash", "splash-time", NULL
>                       };
>                       char buf[sizeof(boot_devices)];
>                       char *standard_boot_devices;
> @@ -2366,6 +2378,8 @@ int main(int argc, char **argv, char **envp)
>                                   exit(1);
>                               }
>                           }
> +                        qemu_opts_parse(qemu_find_opts("boot-opts"),
> +                                        optarg, 0);
>                       }
>                   }
>                   break;
> @@ -3327,6 +3341,7 @@ int main(int argc, char **argv, char **envp)
>       main_loop();
>       quit_timers();
>       net_cleanup();
> +    res_free();
>
>       return 0;
>   }

  reply	other threads:[~2011-07-23 15:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-10 10:09 [Qemu-devel] [PATCH v6] showing a splash picture when start Wayne Xia
2011-07-23 15:53 ` Anthony Liguori [this message]
2011-07-27 10:07   ` Wayne Xia

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=4E2AEE75.2050807@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=paul@codesourcery.com \
    --cc=qemu-devel@nongnu.org \
    --cc=xiawenc@linux.vnet.ibm.com \
    /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).