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 wrote: > > On Jan 29, 2012 8:41 PM, "John Williams" > 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 Anthony Liguori
On Jan 29, 2012 8:41 PM, "John Williams" 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

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 f

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 Liguori wrote: On 01/29/2012 06:28 PM, John Williams wrote: On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite wrote: Hi All, So on the topic of these command line arguments for initrd, dtb and frien

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 wrote: > On 01/29/2012 06:28 PM, John Williams wrote: >> >> On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite >>  wrote: >>> >>> Hi All, >>> >>> So on the topic of these command line arguments for initrd, dtb and >>> friends, >>> another related

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 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 reloc

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

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 wrote: > On Sun, Jan 29, 2012 at 4:51 PM, Peter Crosthwaite > 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) i

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 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 initrd or kernel. Currently thes

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 KERNEL_ARGS

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

2012-01-27 Thread Markus Armbruster
Eric Blake 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 parameter: >>

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=

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 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 struct FWBootEntry FWBoot

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,

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-options.hx | 3 +++ v

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

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 F

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 > --- > qemu-options.hx |3 +++ > vl.c|4 >

[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 --- qemu-options.hx |3 +++ vl.c|4 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/qemu-op