Re: [Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-04 Thread Peter Crosthwaite
On Mon, Aug 4, 2014 at 6:48 PM, Richard W.M. Jones wrote: > On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote: >> On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones wrote: >> > +max_bytes = UBOOT_MAX_GUNZIP_BYTES; >> >> Why does u-boot's maximum size limit apply here? > > We n

Re: [Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-04 Thread Peter Maydell
On 4 August 2014 09:48, Richard W.M. Jones wrote: > On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote: >> On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones wrote: >> > +max_bytes = UBOOT_MAX_GUNZIP_BYTES; >> >> Why does u-boot's maximum size limit apply here? > > We need some

Re: [Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-04 Thread Richard W.M. Jones
On Mon, Aug 04, 2014 at 09:05:39AM +1000, Peter Crosthwaite wrote: > On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones wrote: > > +max_bytes = UBOOT_MAX_GUNZIP_BYTES; > > Why does u-boot's maximum size limit apply here? We need some maximum to prevent people uploading a kernel (perhaps from

Re: [Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-03 Thread Peter Crosthwaite
On Sun, Aug 3, 2014 at 1:45 AM, Richard W.M. Jones wrote: > On aarch64 it is the bootloader's job to uncompress the kernel. UEFI > and u-boot bootloaders do this automatically when the kernel is > gzip-compressed. > > However the qemu -kernel option does not do this. The following > command does

[Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-02 Thread Richard W.M. Jones
On aarch64 it is the bootloader's job to uncompress the kernel. UEFI and u-boot bootloaders do this automatically when the kernel is gzip-compressed. However the qemu -kernel option does not do this. The following command does not work: qemu-system-aarch64 [...] -kernel /boot/vmlinuz because

[Qemu-devel] [PATCH v4] aarch64: Allow -kernel option to take a gzip-compressed

2014-08-02 Thread Richard W.M. Jones
No change compared to v3. I just rebased the patch on top of current HEAD and ensured that it still works. Any comments at all on this approach? It's the last patch I need to make libguestfs work on aarch64 ... Rich.