From: Kevin Wolf <kwolf@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Qemu-devel@nongnu.org, Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] Re: [Qemu-commits] [COMMIT 45a50b1] Reorganize option rom (+linux kernel) loading.
Date: Mon, 12 Oct 2009 11:49:13 +0200 [thread overview]
Message-ID: <4AD2FB99.8000004@redhat.com> (raw)
In-Reply-To: <200910062005.n96K5O2q013344@d01av03.pok.ibm.com>
Am 06.10.2009 22:05, schrieb Anthony Liguori:
> From: Gerd Hoffmann <kraxel@redhat.com>
>
> This patch adds infrastructure to maintain memory regions which must be
> restored on reset. That includes roms (vga bios and option roms on pc),
> but is also used when loading linux kernels directly. Features:
>
> - loading files is supported.
> - passing blobs is supported.
> - target address range is supported (for optionrom area).
> - fixed target memory address is supported (linux kernel).
>
> New in v2:
> - writes to ROM are done only at initial boot.
> - also handle aout and uimage loaders.
> - drop unused fread_targphys() function.
>
> The final memory layout is created once all memory regions are
> registered. The option roms get addresses assigned and the
> registered regions are checked against overlaps. Finally all data
> is copyed to the guest memory.
>
> Advantages:
>
> (1) Filling memory on initial boot and on reset takes the same
> code path, making reset more robust.
> (2) The need to keep track of the option rom load address is gone.
> (3) Due to (2) option roms can be loaded outside pc_init(). This
> allows to move the pxe rom loading into the nic drivers for
> example.
>
> Additional bonus: There is a 'info roms' monitor command now.
>
> The patch also switches over pc.c and removes the
> option_rom_setup_reset() and load_option_rom() functions.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Is it intended that -initrd files are no longer searched in the current
directory? Before this change you could use -initrd file, now you need
-initrd ./file
Kevin
next parent reply other threads:[~2009-10-12 9:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200910062005.n96K5O2q013344@d01av03.pok.ibm.com>
2009-10-12 9:49 ` Kevin Wolf [this message]
2009-10-12 13:23 ` [Qemu-devel] Re: [Qemu-commits] [COMMIT 45a50b1] Reorganize option rom (+linux kernel) loading Gerd Hoffmann
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=4AD2FB99.8000004@redhat.com \
--to=kwolf@redhat.com \
--cc=Qemu-devel@nongnu.org \
--cc=aliguori@us.ibm.com \
--cc=kraxel@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.