Re: [Qemu-devel] [PULL 0/1] updated fw_cfg option ROM 2016-07-13

[Paolo Bonzini]

On 14/07/2016 14:43, Peter Maydell wrote:
> On 13 July 2016 at 13:55, Paolo Bonzini <address@hidden> wrote:
>> The following changes since commit a91a4e7d8cfe6ece610aacf7c52738188f5b5cb5:
>>
>>   Merge remote-tracking branch 'remotes/ehabkost/tags/x86-pull-request' into 
>> staging (2016-07-11 15:08:47 +0100)
>>
>> are available in the git repository at:
>>
>>
>>   git://github.com/bonzini/qemu.git tags/for-upstream-fwcfg
>>
>> for you to fetch changes up to 7282d40ba0937512ec94c30895e2ef0299069569:
>>
>>   Add optionrom compatible with fw_cfg DMA version (2016-07-13 14:49:03 
>> +0200)
>>
>> ----------------------------------------------------------------
>> * Updated fw_cfg option ROM to include DMA support
>>
>> ----------------------------------------------------------------
>> Marc Marí (1):
>>       Add optionrom compatible with fw_cfg DMA version
> 
> Hi. I'm afraid this fails to build with the win32 build:
> 
>   CC    optionrom/linuxboot_dma.o
>   Building optionrom/linuxboot_dma.img
> linuxboot_dma.o:linuxboot_dma.c:(.text+0x57): undefined reference to
> `load_kernel'
> 
> This thread has the discussion for the same problem from May:
> https://lists.gnu.org/archive/html/qemu-devel/2016-05/msg04978.html

Aha, I fixed another issue (that was not discussed on the mailing list
and only found by the docker tests) but forgot about this one.  Well,
that's why I separated this patch from the other pull request.

Paolo