Re: [PATCHv3 14/14] Pass boot device list to firmware.
On Wed, Nov 10, 2010 at 07:20:51PM +, Blue Swirl wrote: > 2010/11/10 Gleb Natapov : > > On Wed, Nov 10, 2010 at 07:11:54PM +, Blue Swirl wrote: > >> On Wed, Nov 10, 2010 at 5:14 PM, Gleb Natapov wrote: > >> > > >> > Signed-off-by: Gleb Natapov > >> > --- > >> > hw/fw_cfg.h | 1 + > >> > hw/pc.c | 19 +++ > >> > sysemu.h | 1 + > >> > vl.c | 40 > >> > 4 files changed, 61 insertions(+), 0 deletions(-) > >> > > >> > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h > >> > index 4d13a4f..bfbf1f9 100644 > >> > --- a/hw/fw_cfg.h > >> > +++ b/hw/fw_cfg.h > >> > @@ -31,6 +31,7 @@ > >> > #define FW_CFG_FILE_FIRST 0x20 > >> > #define FW_CFG_FILE_SLOTS 0x10 > >> > #define FW_CFG_MAX_ENTRY (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) > >> > +#define FW_CFG_BOOTINDEX (FW_CFG_MAX_ENTRY + 1) > >> > > >> > #define FW_CFG_WRITE_CHANNEL 0x4000 > >> > #define FW_CFG_ARCH_LOCAL 0x8000 > >> > diff --git a/hw/pc.c b/hw/pc.c > >> > index 3bf3862..e51f8ba 100644 > >> > --- a/hw/pc.c > >> > +++ b/hw/pc.c > >> > @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) > >> > } > >> > } > >> > > >> > +static struct machine_ready_notify { > >> > + Notifier n; > >> > + void *fw_cfg; > >> > +} machine_ready; > >> > + > >> > + > >> > +static void pc_fw_register_bootindex(struct Notifier* n) > >> > +{ > >> > + uint32_t len; > >> > + char *bootindex = get_boot_devices_list(&len); > >> > + > >> > + fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, > >> > + (uint8_t*)bootindex, len); > >> > +} > >> > + > >> > void pc_memory_init(ram_addr_t ram_size, > >> > const char *kernel_filename, > >> > const char *kernel_cmdline, > >> > @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, > >> > for (i = 0; i < nb_option_roms; i++) { > >> > rom_add_option(option_rom[i]); > >> > } > >> > + > >> > + machine_ready.n.notify = pc_fw_register_bootindex; > >> > + machine_ready.fw_cfg = fw_cfg; > >> > + qemu_add_machine_init_done_notifier(&machine_ready.n); > >> > >> Actually this and the above should be moved to generic code, like > >> vl.c. I doubt this would be any different for other platforms. > > Agree, but we do not have reference to fw_cfg in generic code. > > How about inside fw_cfg.c then? Will move it there. -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv3 14/14] Pass boot device list to firmware.
2010/11/10 Gleb Natapov : > On Wed, Nov 10, 2010 at 07:11:54PM +, Blue Swirl wrote: >> On Wed, Nov 10, 2010 at 5:14 PM, Gleb Natapov wrote: >> > >> > Signed-off-by: Gleb Natapov >> > --- >> > hw/fw_cfg.h | 1 + >> > hw/pc.c | 19 +++ >> > sysemu.h | 1 + >> > vl.c | 40 >> > 4 files changed, 61 insertions(+), 0 deletions(-) >> > >> > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h >> > index 4d13a4f..bfbf1f9 100644 >> > --- a/hw/fw_cfg.h >> > +++ b/hw/fw_cfg.h >> > @@ -31,6 +31,7 @@ >> > #define FW_CFG_FILE_FIRST 0x20 >> > #define FW_CFG_FILE_SLOTS 0x10 >> > #define FW_CFG_MAX_ENTRY (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) >> > +#define FW_CFG_BOOTINDEX (FW_CFG_MAX_ENTRY + 1) >> > >> > #define FW_CFG_WRITE_CHANNEL 0x4000 >> > #define FW_CFG_ARCH_LOCAL 0x8000 >> > diff --git a/hw/pc.c b/hw/pc.c >> > index 3bf3862..e51f8ba 100644 >> > --- a/hw/pc.c >> > +++ b/hw/pc.c >> > @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) >> > } >> > } >> > >> > +static struct machine_ready_notify { >> > + Notifier n; >> > + void *fw_cfg; >> > +} machine_ready; >> > + >> > + >> > +static void pc_fw_register_bootindex(struct Notifier* n) >> > +{ >> > + uint32_t len; >> > + char *bootindex = get_boot_devices_list(&len); >> > + >> > + fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, >> > + (uint8_t*)bootindex, len); >> > +} >> > + >> > void pc_memory_init(ram_addr_t ram_size, >> > const char *kernel_filename, >> > const char *kernel_cmdline, >> > @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, >> > for (i = 0; i < nb_option_roms; i++) { >> > rom_add_option(option_rom[i]); >> > } >> > + >> > + machine_ready.n.notify = pc_fw_register_bootindex; >> > + machine_ready.fw_cfg = fw_cfg; >> > + qemu_add_machine_init_done_notifier(&machine_ready.n); >> >> Actually this and the above should be moved to generic code, like >> vl.c. I doubt this would be any different for other platforms. > Agree, but we do not have reference to fw_cfg in generic code. How about inside fw_cfg.c then? -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv3 14/14] Pass boot device list to firmware.
On Wed, Nov 10, 2010 at 07:11:54PM +, Blue Swirl wrote: > On Wed, Nov 10, 2010 at 5:14 PM, Gleb Natapov wrote: > > > > Signed-off-by: Gleb Natapov > > --- > > hw/fw_cfg.h | 1 + > > hw/pc.c | 19 +++ > > sysemu.h | 1 + > > vl.c | 40 > > 4 files changed, 61 insertions(+), 0 deletions(-) > > > > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h > > index 4d13a4f..bfbf1f9 100644 > > --- a/hw/fw_cfg.h > > +++ b/hw/fw_cfg.h > > @@ -31,6 +31,7 @@ > > #define FW_CFG_FILE_FIRST 0x20 > > #define FW_CFG_FILE_SLOTS 0x10 > > #define FW_CFG_MAX_ENTRY (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) > > +#define FW_CFG_BOOTINDEX (FW_CFG_MAX_ENTRY + 1) > > > > #define FW_CFG_WRITE_CHANNEL 0x4000 > > #define FW_CFG_ARCH_LOCAL 0x8000 > > diff --git a/hw/pc.c b/hw/pc.c > > index 3bf3862..e51f8ba 100644 > > --- a/hw/pc.c > > +++ b/hw/pc.c > > @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) > > } > > } > > > > +static struct machine_ready_notify { > > + Notifier n; > > + void *fw_cfg; > > +} machine_ready; > > + > > + > > +static void pc_fw_register_bootindex(struct Notifier* n) > > +{ > > + uint32_t len; > > + char *bootindex = get_boot_devices_list(&len); > > + > > + fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, > > + (uint8_t*)bootindex, len); > > +} > > + > > void pc_memory_init(ram_addr_t ram_size, > > const char *kernel_filename, > > const char *kernel_cmdline, > > @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, > > for (i = 0; i < nb_option_roms; i++) { > > rom_add_option(option_rom[i]); > > } > > + > > + machine_ready.n.notify = pc_fw_register_bootindex; > > + machine_ready.fw_cfg = fw_cfg; > > + qemu_add_machine_init_done_notifier(&machine_ready.n); > > Actually this and the above should be moved to generic code, like > vl.c. I doubt this would be any different for other platforms. Agree, but we do not have reference to fw_cfg in generic code. -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv3 14/14] Pass boot device list to firmware.
On Wed, Nov 10, 2010 at 5:14 PM, Gleb Natapov wrote: > > Signed-off-by: Gleb Natapov > --- > hw/fw_cfg.h | 1 + > hw/pc.c | 19 +++ > sysemu.h | 1 + > vl.c | 40 > 4 files changed, 61 insertions(+), 0 deletions(-) > > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h > index 4d13a4f..bfbf1f9 100644 > --- a/hw/fw_cfg.h > +++ b/hw/fw_cfg.h > @@ -31,6 +31,7 @@ > #define FW_CFG_FILE_FIRST 0x20 > #define FW_CFG_FILE_SLOTS 0x10 > #define FW_CFG_MAX_ENTRY (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) > +#define FW_CFG_BOOTINDEX (FW_CFG_MAX_ENTRY + 1) > > #define FW_CFG_WRITE_CHANNEL 0x4000 > #define FW_CFG_ARCH_LOCAL 0x8000 > diff --git a/hw/pc.c b/hw/pc.c > index 3bf3862..e51f8ba 100644 > --- a/hw/pc.c > +++ b/hw/pc.c > @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) > } > } > > +static struct machine_ready_notify { > + Notifier n; > + void *fw_cfg; > +} machine_ready; > + > + > +static void pc_fw_register_bootindex(struct Notifier* n) > +{ > + uint32_t len; > + char *bootindex = get_boot_devices_list(&len); > + > + fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, > + (uint8_t*)bootindex, len); > +} > + > void pc_memory_init(ram_addr_t ram_size, > const char *kernel_filename, > const char *kernel_cmdline, > @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, > for (i = 0; i < nb_option_roms; i++) { > rom_add_option(option_rom[i]); > } > + > + machine_ready.n.notify = pc_fw_register_bootindex; > + machine_ready.fw_cfg = fw_cfg; > + qemu_add_machine_init_done_notifier(&machine_ready.n); Actually this and the above should be moved to generic code, like vl.c. I doubt this would be any different for other platforms. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCHv3 14/14] Pass boot device list to firmware.
On Wed, Nov 10, 2010 at 5:14 PM, Gleb Natapov wrote: > > Signed-off-by: Gleb Natapov > --- > hw/fw_cfg.h | 1 + > hw/pc.c | 19 +++ > sysemu.h | 1 + > vl.c | 40 > 4 files changed, 61 insertions(+), 0 deletions(-) > > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h > index 4d13a4f..bfbf1f9 100644 > --- a/hw/fw_cfg.h > +++ b/hw/fw_cfg.h > @@ -31,6 +31,7 @@ > #define FW_CFG_FILE_FIRST 0x20 > #define FW_CFG_FILE_SLOTS 0x10 > #define FW_CFG_MAX_ENTRY (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) > +#define FW_CFG_BOOTINDEX (FW_CFG_MAX_ENTRY + 1) You should bump FW_CFG_MAX_ENTRY instead. > #define FW_CFG_WRITE_CHANNEL 0x4000 > #define FW_CFG_ARCH_LOCAL 0x8000 > diff --git a/hw/pc.c b/hw/pc.c > index 3bf3862..e51f8ba 100644 > --- a/hw/pc.c > +++ b/hw/pc.c > @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) > } > } > > +static struct machine_ready_notify { > + Notifier n; > + void *fw_cfg; > +} machine_ready; > + > + > +static void pc_fw_register_bootindex(struct Notifier* n) > +{ > + uint32_t len; > + char *bootindex = get_boot_devices_list(&len); > + > + fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, > + (uint8_t*)bootindex, len); > +} > + > void pc_memory_init(ram_addr_t ram_size, > const char *kernel_filename, > const char *kernel_cmdline, > @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, > for (i = 0; i < nb_option_roms; i++) { > rom_add_option(option_rom[i]); > } > + > + machine_ready.n.notify = pc_fw_register_bootindex; > + machine_ready.fw_cfg = fw_cfg; > + qemu_add_machine_init_done_notifier(&machine_ready.n); > } > > qemu_irq *pc_allocate_cpu_irq(void) > diff --git a/sysemu.h b/sysemu.h > index 46a588c..bac2df1 100644 > --- a/sysemu.h > +++ b/sysemu.h > @@ -197,4 +197,5 @@ void register_devices(void); > > void add_boot_device_path(int32_t bootindex, DeviceState *dev, > const char *suffix); > +char *get_boot_devices_list(uint32_t *size); > #endif > diff --git a/vl.c b/vl.c > index 00ab2b4..4b1a406 100644 > --- a/vl.c > +++ b/vl.c > @@ -734,6 +734,46 @@ void add_boot_device_path(int32_t bootindex, DeviceState > *dev, > QTAILQ_INSERT_TAIL(&fw_boot_order, node, link); > } > > +char *get_boot_devices_list(uint32_t *size) > +{ > + FWBootEntry *i; > + uint32_t total = 1, c = 0; > + char *list = qemu_malloc(1); > + > + QTAILQ_FOREACH(i, &fw_boot_order, link) { > + char *devpath = NULL, *bootpath; > + int len; > + > + if (i->dev) { > + devpath = qdev_get_fw_dev_path(i->dev); > + assert(devpath); > + } > + > + if (i->suffix && devpath) { > + bootpath = qemu_malloc(strlen(devpath) + strlen(i->suffix) + 2); > + sprintf(bootpath, "%s/%s", devpath, i->suffix); > + qemu_free(devpath); > + } else if (devpath) { > + bootpath = devpath; > + } else { > + bootpath = strdup(i->suffix); > + } > + > + len = strlen(bootpath); > + list = qemu_realloc(list, total + len + 1); > + list[total++] = len; > + memcpy(&list[total], bootpath, len); > + total += len; > + c++; > + qemu_free(bootpath); > + } > + > + list[0] = c; > + *size = total; > + > + return list; > +} The format of the list should be documented, it wasn't immediately obvious where '1' comes from in qemu_malloc(1). -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCHv3 14/14] Pass boot device list to firmware.
Signed-off-by: Gleb Natapov --- hw/fw_cfg.h |1 + hw/pc.c | 19 +++ sysemu.h|1 + vl.c| 40 4 files changed, 61 insertions(+), 0 deletions(-) diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h index 4d13a4f..bfbf1f9 100644 --- a/hw/fw_cfg.h +++ b/hw/fw_cfg.h @@ -31,6 +31,7 @@ #define FW_CFG_FILE_FIRST 0x20 #define FW_CFG_FILE_SLOTS 0x10 #define FW_CFG_MAX_ENTRY(FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS) +#define FW_CFG_BOOTINDEX(FW_CFG_MAX_ENTRY + 1) #define FW_CFG_WRITE_CHANNEL0x4000 #define FW_CFG_ARCH_LOCAL 0x8000 diff --git a/hw/pc.c b/hw/pc.c index 3bf3862..e51f8ba 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -883,6 +883,21 @@ void pc_cpus_init(const char *cpu_model) } } +static struct machine_ready_notify { +Notifier n; +void *fw_cfg; +} machine_ready; + + +static void pc_fw_register_bootindex(struct Notifier* n) +{ +uint32_t len; +char *bootindex = get_boot_devices_list(&len); + +fw_cfg_add_bytes(machine_ready.fw_cfg, FW_CFG_BOOTINDEX, + (uint8_t*)bootindex, len); +} + void pc_memory_init(ram_addr_t ram_size, const char *kernel_filename, const char *kernel_cmdline, @@ -982,6 +997,10 @@ void pc_memory_init(ram_addr_t ram_size, for (i = 0; i < nb_option_roms; i++) { rom_add_option(option_rom[i]); } + +machine_ready.n.notify = pc_fw_register_bootindex; +machine_ready.fw_cfg = fw_cfg; +qemu_add_machine_init_done_notifier(&machine_ready.n); } qemu_irq *pc_allocate_cpu_irq(void) diff --git a/sysemu.h b/sysemu.h index 46a588c..bac2df1 100644 --- a/sysemu.h +++ b/sysemu.h @@ -197,4 +197,5 @@ void register_devices(void); void add_boot_device_path(int32_t bootindex, DeviceState *dev, const char *suffix); +char *get_boot_devices_list(uint32_t *size); #endif diff --git a/vl.c b/vl.c index 00ab2b4..4b1a406 100644 --- a/vl.c +++ b/vl.c @@ -734,6 +734,46 @@ void add_boot_device_path(int32_t bootindex, DeviceState *dev, QTAILQ_INSERT_TAIL(&fw_boot_order, node, link); } +char *get_boot_devices_list(uint32_t *size) +{ +FWBootEntry *i; +uint32_t total = 1, c = 0; +char *list = qemu_malloc(1); + +QTAILQ_FOREACH(i, &fw_boot_order, link) { +char *devpath = NULL, *bootpath; +int len; + +if (i->dev) { +devpath = qdev_get_fw_dev_path(i->dev); +assert(devpath); +} + +if (i->suffix && devpath) { +bootpath = qemu_malloc(strlen(devpath) + strlen(i->suffix) + 2); +sprintf(bootpath, "%s/%s", devpath, i->suffix); +qemu_free(devpath); +} else if (devpath) { +bootpath = devpath; +} else { +bootpath = strdup(i->suffix); +} + +len = strlen(bootpath); +list = qemu_realloc(list, total + len + 1); +list[total++] = len; +memcpy(&list[total], bootpath, len); +total += len; +c++; +qemu_free(bootpath); +} + +list[0] = c; +*size = total; + +return list; +} + static void numa_add(const char *optarg) { char option[128]; -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html