Re: [Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-07 Thread Aurelien Jarno
On Wed, Sep 05, 2012 at 03:11:13PM +0200, Kevin Wolf wrote: Am 05.09.2012 14:56, schrieb Henning Schild: Find a hopefully proper patch attached. Take it or leave it. Signed-off-by: Henning Schild henn...@hennsch.de Reviewed-by: Kevin Wolf kw...@redhat.com Aurelien, I think in the

[Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-05 Thread Henning Schild
This patch fixes a bug in qemu which prevents Multiboot ELF kernels from being loaded with the -kernel option. Find a full description of the problem here https://bugs.launchpad.net/qemu/+bug/1044727 . --- hw/elf_ops.h | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff

Re: [Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-05 Thread Dunrong Huang
2012/9/5 Henning Schild henn...@hennsch.de: This patch fixes a bug in qemu which prevents Multiboot ELF kernels from being loaded with the -kernel option. Find a full description of the problem here https://bugs.launchpad.net/qemu/+bug/1044727 . --- hw/elf_ops.h | 10 ++ 1 files

Re: [Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-05 Thread Kevin Wolf
Am 05.09.2012 11:05, schrieb Henning Schild: This patch fixes a bug in qemu which prevents Multiboot ELF kernels from being loaded with the -kernel option. Find a full description of the problem here https://bugs.launchpad.net/qemu/+bug/1044727 . The logic looks good to me, but there are

Re: [Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-05 Thread Henning Schild
Find a hopefully proper patch attached. Take it or leave it. Signed-off-by: Henning Schild henn...@hennsch.de --- hw/elf_ops.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/hw/elf_ops.h b/hw/elf_ops.h index fa65ce2..731a983 100644 --- a/hw/elf_ops.h +++

Re: [Qemu-devel] [PATCH] fix entry pointer for ELF kernels loaded with -kernel option

2012-09-05 Thread Kevin Wolf
Am 05.09.2012 14:56, schrieb Henning Schild: Find a hopefully proper patch attached. Take it or leave it. Signed-off-by: Henning Schild henn...@hennsch.de Reviewed-by: Kevin Wolf kw...@redhat.com Aurelien, I think in the past you committed some changes in this area. Does this look good to