Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-31 Thread John Williams
On Mon, Jan 30, 2012 at 12:48 PM, Anthony Liguori anth...@codemonkey.ws wrote: On Jan 29, 2012 8:41 PM, John Williams john.willi...@petalogix.com wrote: There's an opportunity here - QEMU needs the cmdline ability to load random binaries/elfs anyway, such as --load file@address

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread John Williams
On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Hi All, So on the topic of these command line arguments for initrd, dtb and friends, another related issue we have encountered (and have hacked around in our tree) is not being able to relocate the

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread John Williams
On Mon, Jan 30, 2012 at 10:28 AM, John Williams john.willi...@petalogix.com wrote: On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Hi All, So on the topic of these command line arguments for initrd, dtb and friends, another related issue we have

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread Anthony Liguori
On 01/24/2012 12:23 PM, Stefan Weil wrote: Am 24.01.2012 08:22, schrieb Andreas Färber: Am 23.01.2012 08:20, schrieb Peter A. G. Crosthwaite: Added linux specific kernel dtb option. This option can be specified to inject an argument device tree blob (dtb) into linux. Signed-off-by: Peter A.

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread Anthony Liguori
On 01/29/2012 06:28 PM, John Williams wrote: On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite peter.crosthwa...@petalogix.com wrote: Hi All, So on the topic of these command line arguments for initrd, dtb and friends, another related issue we have encountered (and have hacked around in our

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread John Williams
On Mon, Jan 30, 2012 at 12:11 PM, Anthony Liguori anth...@codemonkey.ws wrote: On 01/29/2012 06:28 PM, John Williams wrote: On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite peter.crosthwa...@petalogix.com  wrote: Hi All, So on the topic of these command line arguments for initrd, dtb and

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread Anthony Liguori
On 01/29/2012 08:19 PM, John Williams wrote: On Mon, Jan 30, 2012 at 12:11 PM, Anthony Liguorianth...@codemonkey.ws wrote: On 01/29/2012 06:28 PM, John Williams wrote: On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite peter.crosthwa...@petalogix.comwrote: Hi All, So on the topic of

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread John Williams
There's an opportunity here - QEMU needs the cmdline ability to load random binaries/elfs anyway, such as --load file@address Make an elf loader device if you desire this ability but I'm skeptical that it really is all that useful. It is useful for non-Linux use-cases, of which there

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-29 Thread Anthony Liguori
On Jan 29, 2012 8:41 PM, John Williams john.willi...@petalogix.com wrote: There's an opportunity here - QEMU needs the cmdline ability to load random binaries/elfs anyway, such as --load file@address Make an elf loader device if you desire this ability but I'm skeptical that

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-28 Thread Peter Crosthwaite
Hi All, So on the topic of these command line arguments for initrd, dtb and friends, another related issue we have encountered (and have hacked around in our tree) is not being able to relocate the initrd or kernel. Currently these memory locations are hardcoded in arm_boot.c: #define

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-27 Thread Markus Armbruster
Eric Blake ebl...@redhat.com writes: On 01/26/2012 12:34 PM, Scott Wood wrote: On 01/24/2012 12:23 PM, Stefan Weil wrote: I'd prefer a different solution. As far as I have understood, the dtb is only useful with a kernel, so it could be handled as an optional attribute to the -kernel

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-26 Thread Scott Wood
On 01/24/2012 12:23 PM, Stefan Weil wrote: I'd prefer a different solution. As far as I have understood, the dtb is only useful with a kernel, so it could be handled as an optional attribute to the -kernel parameter: -kernel IMAGE[,dtb=DTB] Of course the same applies to -append, but

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-26 Thread Peter Maydell
On 23 January 2012 07:20, Peter A. G. Crosthwaite peter.crosthwa...@petalogix.com wrote: --- a/vl.c +++ b/vl.c @@ -233,6 +233,7 @@ int boot_menu;  uint8_t *boot_splash_filedata;  int boot_splash_filedata_size;  uint8_t qemu_extra_params_fw[2]; +const char *qemu_kerndtb = NULL;  typedef

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-26 Thread Eric Blake
On 01/26/2012 12:34 PM, Scott Wood wrote: On 01/24/2012 12:23 PM, Stefan Weil wrote: I'd prefer a different solution. As far as I have understood, the dtb is only useful with a kernel, so it could be handled as an optional attribute to the -kernel parameter: -kernel IMAGE[,dtb=DTB] Of

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-24 Thread Stefan Weil
Am 24.01.2012 08:22, schrieb Andreas Färber: Am 23.01.2012 08:20, schrieb Peter A. G. Crosthwaite: Added linux specific kernel dtb option. This option can be specified to inject an argument device tree blob (dtb) into linux. Signed-off-by: Peter A. G. Crosthwaite

[Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-23 Thread Peter A. G. Crosthwaite
Added linux specific kernel dtb option. This option can be specified to inject an argument device tree blob (dtb) into linux. Signed-off-by: Peter A. G. Crosthwaite peter.crosthwa...@petalogix.com --- qemu-options.hx |3 +++ vl.c|4 2 files changed, 7 insertions(+), 0

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-23 Thread Andreas Färber
Am 23.01.2012 08:20, schrieb Peter A. G. Crosthwaite: Added linux specific kernel dtb option. This option can be specified to inject an argument device tree blob (dtb) into linux. Signed-off-by: Peter A. G. Crosthwaite peter.crosthwa...@petalogix.com --- qemu-options.hx |3 +++ vl.c

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-23 Thread Peter Crosthwaite
Hi Andreas, Thanks for that, I will rename the switch to -kernel-dtb. You are correct that 7/7 should have the usage in it, that patch was incorrectly generated, I will regenerate and resend the series tomorrow for another review cycle. Regards. Peter On Tue, Jan 24, 2012 at 5:22 PM, Andreas

Re: [Qemu-devel] [RFC PATCH 5/7] vl.c: added -kerndtb option

2012-01-23 Thread Andreas Färber
Hi Peter, Am 24.01.2012 08:35, schrieb Peter Crosthwaite: Thanks for that, I will rename the switch to -kernel-dtb. You are correct that 7/7 should have the usage in it, that patch was incorrectly generated, I will regenerate and resend the series tomorrow for another review cycle. Just