[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3.1 10/31] pc: pass QEMUMachineInitArgs to pc_m
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3.1 10/31] pc: pass QEMUMachineInitArgs to pc_memory_init |
Date: |
Wed, 7 May 2014 17:50:36 +0300 |
On Tue, May 06, 2014 at 05:27:31PM +0800, Hu Tao wrote:
> From: Paolo Bonzini <address@hidden>
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Hu Tao <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/i386/pc.c | 11 +++++------
> hw/i386/pc_piix.c | 8 +++-----
> hw/i386/pc_q35.c | 4 +---
> include/hw/i386/pc.h | 7 +++----
> 4 files changed, 12 insertions(+), 18 deletions(-)
>
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index 249e504..d5fb884 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -1161,10 +1161,8 @@ void pc_acpi_init(const char *default_dsdt)
> }
> }
>
> -FWCfgState *pc_memory_init(MemoryRegion *system_memory,
> - const char *kernel_filename,
> - const char *kernel_cmdline,
> - const char *initrd_filename,
> +FWCfgState *pc_memory_init(QEMUMachineInitArgs *args,
> + MemoryRegion *system_memory,
> ram_addr_t below_4g_mem_size,
> ram_addr_t above_4g_mem_size,
> MemoryRegion *rom_memory,
> @@ -1176,7 +1174,7 @@ FWCfgState *pc_memory_init(MemoryRegion *system_memory,
> MemoryRegion *ram_below_4g, *ram_above_4g;
> FWCfgState *fw_cfg;
>
> - linux_boot = (kernel_filename != NULL);
> + linux_boot = (args->kernel_filename != NULL);
>
> /* Allocate RAM. We allocate it as a single memory region and use
> * aliases to address portions of it, mostly for backwards compatibility
> @@ -1217,7 +1215,8 @@ FWCfgState *pc_memory_init(MemoryRegion *system_memory,
> rom_set_fw(fw_cfg);
>
> if (linux_boot) {
> - load_linux(fw_cfg, kernel_filename, initrd_filename, kernel_cmdline,
> below_4g_mem_size);
> + load_linux(fw_cfg, args->kernel_filename, args->initrd_filename,
> + args->kernel_cmdline, below_4g_mem_size);
> }
>
> for (i = 0; i < nb_option_roms; i++) {
> diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> index 7930a26..3bd8821 100644
> --- a/hw/i386/pc_piix.c
> +++ b/hw/i386/pc_piix.c
> @@ -151,11 +151,9 @@ static void pc_init1(QEMUMachineInitArgs *args,
>
> /* allocate ram and load rom/bios */
> if (!xen_enabled()) {
> - fw_cfg = pc_memory_init(system_memory,
> - args->kernel_filename, args->kernel_cmdline,
> - args->initrd_filename,
> - below_4g_mem_size, above_4g_mem_size,
> - rom_memory, &ram_memory, guest_info);
> + fw_cfg = pc_memory_init(args, system_memory,
> + below_4g_mem_size, above_4g_mem_size,
> + rom_memory, &ram_memory, guest_info);
> }
>
> gsi_state = g_malloc0(sizeof(*gsi_state));
> diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
> index c844dc2..01794fc 100644
> --- a/hw/i386/pc_q35.c
> +++ b/hw/i386/pc_q35.c
> @@ -138,9 +138,7 @@ static void pc_q35_init(QEMUMachineInitArgs *args)
>
> /* allocate ram and load rom/bios */
> if (!xen_enabled()) {
> - pc_memory_init(get_system_memory(),
> - args->kernel_filename, args->kernel_cmdline,
> - args->initrd_filename,
> + pc_memory_init(args, get_system_memory(),
> below_4g_mem_size, above_4g_mem_size,
> rom_memory, &ram_memory, guest_info);
> }
> diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
> index 9010246..8fc0527 100644
> --- a/include/hw/i386/pc.h
> +++ b/include/hw/i386/pc.h
> @@ -3,6 +3,7 @@
>
> #include "qemu-common.h"
> #include "exec/memory.h"
> +#include "hw/boards.h"
> #include "hw/isa/isa.h"
> #include "hw/block/fdc.h"
> #include "net/net.h"
> @@ -134,10 +135,8 @@ PcGuestInfo *pc_guest_info_init(ram_addr_t
> below_4g_mem_size,
> void pc_pci_as_mapping_init(Object *owner, MemoryRegion *system_memory,
> MemoryRegion *pci_address_space);
>
> -FWCfgState *pc_memory_init(MemoryRegion *system_memory,
> - const char *kernel_filename,
> - const char *kernel_cmdline,
> - const char *initrd_filename,
> +FWCfgState *pc_memory_init(QEMUMachineInitArgs *args,
> + MemoryRegion *system_memory,
> ram_addr_t below_4g_mem_size,
> ram_addr_t above_4g_mem_size,
> MemoryRegion *rom_memory,
> --
> 1.8.5.2.229.g4448466
>
- [Qemu-devel] [PATCH v3.1 09/31] qmp: improve error reporting for -object and object-add, (continued)
- [Qemu-devel] [PATCH v3.1 09/31] qmp: improve error reporting for -object and object-add, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 12/31] add memdev backend infrastructure, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 11/31] numa: introduce memory_region_allocate_system_memory, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 15/31] memory: move mem_path handling to memory_region_allocate_system_memory, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 08/31] qmp: allow object-add completion handler to get canonical path, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 01/31] NUMA: move numa related code to new file numa.c, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 05/31] NUMA: expand MAX_NODES from 64 to 128, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 13/31] numa: add -numa node, memdev= option, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 07/31] vl: redo -object parsing, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 10/31] pc: pass QEMUMachineInitArgs to pc_memory_init, Hu Tao, 2014/05/06
- Re: [Qemu-devel] [PATCH v3.1 10/31] pc: pass QEMUMachineInitArgs to pc_memory_init,
Michael S. Tsirkin <=
- [Qemu-devel] [PATCH v3.1 04/31] NUMA: convert -numa option to use OptsVisitor, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 06/31] man: improve -numa doc, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 16/31] memory: add error propagation to file-based RAM allocation, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 18/31] memory: move RAM_PREALLOC_MASK to exec.c, rename, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 19/31] hostmem: add file-based HostMemoryBackend, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 20/31] hostmem: separate allocation from UserCreatable complete method, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 17/31] memory: move preallocation code out of exec.c, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 14/31] memory: reorganize file-based allocation, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 21/31] hostmem: add merge and dump properties, Hu Tao, 2014/05/06
- [Qemu-devel] [PATCH v3.1 24/31] configure: add Linux libnuma detection, Hu Tao, 2014/05/06