[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] linuxboot_dma: move common functions in a n
Re: [Qemu-devel] [PATCH 2/4] linuxboot_dma: move common functions in a new header
Fri, 11 Jan 2019 11:55:24 -0600
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.1
On 1/11/19 11:48 AM, Michael S. Tsirkin wrote:
>>>> diff --git a/pc-bios/optionrom/optrom.h b/pc-bios/optionrom/optrom.h
>>>> new file mode 100644
>>>> index 0000000000..36f43b43fd
>>>> --- /dev/null
>>>> +++ b/pc-bios/optionrom/optrom.h
>>>> +#include "../../include/standard-headers/linux/qemu_fw_cfg.h"
>>> This depends on <stdint.h>, please include it first.
> Better to just pull in qemu/osdep.h
Except that qemu/osdep.h should already have been pulled in by whatever
.c file is including this header. We specifically document that .h files
shouldn't need to include osdep.h (and in turn, anything that osdep.h
already pulls in, like <stdint.h>).
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
Description: OpenPGP digital signature
[Qemu-devel] [PATCH 3/4] optionrom: add new PVH option rom, Stefano Garzarella, 2019/01/11
[Qemu-devel] [PATCH 4/4] hw/i386/pc: use PVH option rom, Stefano Garzarella, 2019/01/11