[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via c
From: |
Marc Marí |
Subject: |
Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline. |
Date: |
Mon, 19 Oct 2015 14:22:36 +0200 |
On Mon, 19 Oct 2015 08:12:22 -0400
"Kevin O'Connor" <address@hidden> wrote:
> On Mon, Oct 19, 2015 at 01:02:41PM +0100, Peter Maydell wrote:
> > On 19 October 2015 at 12:52, Kevin O'Connor <address@hidden>
> > wrote:
> > > On Mon, Oct 19, 2015 at 12:12:34PM +0100, Peter Maydell wrote:
> > >> Windows fails to compile:
> > >> /home/petmay01/linaro/qemu-for-merges/hw/nvram/fw_cfg.c: In
> > >> function ‘fw_cfg_dma_mem_read’:
> > >> /home/petmay01/linaro/qemu-for-merges/hw/nvram/fw_cfg.c:406:
> > >> warning: integer constant is too large for ‘long’ type
> > >
> > > I don't have a Windows test environment, but I suspect the
> > > following:
> > >
> > > #define FW_CFG_DMA_SIGNATURE 0x51454d5520434647 /* "QEMU CFG" */
> > >
> > > should be changed to:
> > >
> > > #define FW_CFG_DMA_SIGNATURE 0x51454d5520434647ULL /* "QEMU CFG"
> > > */
> >
> > Yes. All 64 bit constants should have an LL or ULL suffix.
>
> Okay - I'll respin patch 7.
Thanks
> > (The test failure is presumably something different.)
>
> I think the fix for the test failure is to make the test less picky
> (it should accept a feature bitmap of either 1 or 3). But I'll leave
> that to Marc or Gerd.
Yes, that's the solution. We updated the version number, and the test
expects exclusively version 1. And it would also be nice to update the
test to test also fw_cfg DMA
I can send a patch for the test (at least the version) tomorrow.
Thanks
Marc
- [Qemu-devel] [PULL 5/7] Enable fw_cfg DMA interface for ARM, (continued)
- [Qemu-devel] [PULL 5/7] Enable fw_cfg DMA interface for ARM, Gerd Hoffmann, 2015/10/19
- [Qemu-devel] [PULL 3/7] fw_cfg DMA interface documentation, Gerd Hoffmann, 2015/10/19
- [Qemu-devel] [PULL 1/7] fw_cfg: insert string blobs via qemu cmdline, Gerd Hoffmann, 2015/10/19
- [Qemu-devel] [PULL 4/7] Implement fw_cfg DMA interface, Gerd Hoffmann, 2015/10/19
- [Qemu-devel] [PULL 7/7] fw_cfg: Define a static signature to be returned on DMA port reads, Gerd Hoffmann, 2015/10/19
- [Qemu-devel] [PULL 2/7] fw_cfg: document fw_cfg_modify_iXX() update functions, Gerd Hoffmann, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Peter Maydell, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Kevin O'Connor, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Peter Maydell, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Kevin O'Connor, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline.,
Marc Marí <=
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Gerd Hoffmann, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Gerd Hoffmann, 2015/10/19
- Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Peter Maydell, 2015/10/19
Re: [Qemu-devel] [PULL 0/7] fw_cfg: add dma interface, add strings via cmdline., Laszlo Ersek, 2015/10/19