Re: [Qemu-devel] [PATCH 0/3] multiboot: Fix offset of bootloader name

[Paolo Bonzini]

On 15/01/2015 12:26, Kevin Wolf wrote:
> This fixes a recent Multiboot load regression (see patch 2) that we
> noticed while hacking on the advent calendar image for December 24.
> 
> Kevin Wolf (3):
>   tests/multiboot: Update reference output
>   multiboot: Fix offset of bootloader name
>   tests/multiboot: Add test for modules
> 
>  hw/i386/multiboot.c         | 16 +++++++------
>  tests/multiboot/Makefile    |  5 ++++-
>  tests/multiboot/libc.c      | 12 ++++++++++
>  tests/multiboot/libc.h      |  1 +
>  tests/multiboot/mmap.out    | 37 +++++++++++++++---------------
>  tests/multiboot/module.txt  |  1 +
>  tests/multiboot/modules.c   | 55 
> +++++++++++++++++++++++++++++++++++++++++++++
>  tests/multiboot/modules.out | 38 +++++++++++++++++++++++++++++++
>  tests/multiboot/run_test.sh |  9 +++++++-
>  9 files changed, 146 insertions(+), 28 deletions(-)
>  create mode 100644 tests/multiboot/module.txt
>  create mode 100644 tests/multiboot/modules.c
>  create mode 100644 tests/multiboot/modules.out
> 

Thanks, I'll put this in the next pull request.

Paolo