[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-4.0 v3 3/4] i386: import bootparam.h
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH for-4.0 v3 3/4] i386: import bootparam.h |
Date: |
Mon, 3 Dec 2018 10:44:52 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.1 |
On 12/3/18 8:48 AM, Li Zhijian wrote:
> #include "hw/net/ne2000-isa.h"
> +#include <asm/bootparam.h>
>
> /* debug PC/ISA interrupts */
> //#define DEBUG_IRQ
> @@ -820,20 +821,6 @@ static long get_file_size(FILE *f)
> return size;
> }
>
> -/* setup_data types */
> -#define SETUP_NONE 0
> -#define SETUP_E820_EXT 1
> -#define SETUP_DTB 2
> -#define SETUP_PCI 3
> -#define SETUP_EFI 4
> -
> -struct setup_data {
> - uint64_t next;
> - uint32_t type;
> - uint32_t len;
> - uint8_t data[0];
> -} __attribute__((packed));
> -
No, this will fail to build for a non-x86 host, which will not have
<asm/bootparam.h>. Or, perhaps, have a *different* asm/bootparam.h that will
be specific to a different host cpu.
r~
[Qemu-devel] [PATCH for-4.0 v3 1/4] unify len and addr type for memory/address APIs, Li Zhijian, 2018/12/03