[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 1/8] mac_oldworld: Allow loading binary ROM image
From: |
Mark Cave-Ayland |
Subject: |
Re: [PATCH v7 1/8] mac_oldworld: Allow loading binary ROM image |
Date: |
Tue, 30 Jun 2020 20:13:27 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 |
On 29/06/2020 19:55, BALATON Zoltan wrote:
> The beige G3 Power Macintosh has a 4MB firmware ROM. Fix the size of
> the rom region and fall back to loading a binary image with -bios if
> loading ELF image failed. This allows testing emulation with a ROM
> image from real hardware as well as using an ELF OpenBIOS image.
>
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
> ---
> v4: use load address from ELF to check if ROM is too big
>
> hw/ppc/mac_oldworld.c | 29 ++++++++++++++++++++---------
> 1 file changed, 20 insertions(+), 9 deletions(-)
>
> diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c
> index f8c204ead7..baf3da6f90 100644
> --- a/hw/ppc/mac_oldworld.c
> +++ b/hw/ppc/mac_oldworld.c
> @@ -59,6 +59,8 @@
> #define NDRV_VGA_FILENAME "qemu_vga.ndrv"
>
> #define GRACKLE_BASE 0xfec00000
> +#define PROM_BASE 0xffc00000
> +#define PROM_SIZE (4 * MiB)
>
> static void fw_cfg_boot_set(void *opaque, const char *boot_device,
> Error **errp)
> @@ -99,6 +101,7 @@ static void ppc_heathrow_init(MachineState *machine)
> SysBusDevice *s;
> DeviceState *dev, *pic_dev;
> BusState *adb_bus;
> + uint64_t bios_addr;
> int bios_size;
> unsigned int smp_cpus = machine->smp.cpus;
> uint16_t ppc_boot_device;
> @@ -127,24 +130,32 @@ static void ppc_heathrow_init(MachineState *machine)
>
> memory_region_add_subregion(sysmem, 0, machine->ram);
>
> - /* allocate and load BIOS */
> - memory_region_init_rom(bios, NULL, "ppc_heathrow.bios", BIOS_SIZE,
> + /* allocate and load firmware ROM */
> + memory_region_init_rom(bios, NULL, "ppc_heathrow.bios", PROM_SIZE,
> &error_fatal);
> + memory_region_add_subregion(sysmem, PROM_BASE, bios);
>
> - if (bios_name == NULL)
> + if (!bios_name) {
> bios_name = PROM_FILENAME;
> + }
> filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
> - memory_region_add_subregion(sysmem, PROM_ADDR, bios);
> -
> - /* Load OpenBIOS (ELF) */
> if (filename) {
> - bios_size = load_elf(filename, NULL, 0, NULL, NULL, NULL, NULL, NULL,
> - 1, PPC_ELF_MACHINE, 0, 0);
> + /* Load OpenBIOS (ELF) */
> + bios_size = load_elf(filename, NULL, NULL, NULL, NULL, &bios_addr,
> + NULL, NULL, 1, PPC_ELF_MACHINE, 0, 0);
> + if (bios_size <= 0) {
> + /* or load binary ROM image */
> + bios_size = load_image_targphys(filename, PROM_BASE, PROM_SIZE);
> + bios_addr = PROM_BASE;
> + } else {
> + /* load_elf sets high 32 bits for some reason, strip those */
> + bios_addr &= 0xffffffffULL;
Repeating my earlier comment from v5: something is wrong here if you need to
manually
strip the high bits. If you compare with SPARC32 which uses the same approach,
there
is no such strip required - have a look there to try and figure out what's
going on here.
ATB,
Mark.
- [PATCH v7 0/8] Mac Old World ROM experiment, BALATON Zoltan, 2020/06/29
- [PATCH v7 3/8] mac_oldworld: Drop a variable, use get_system_memory() directly, BALATON Zoltan, 2020/06/29
- [PATCH v7 4/8] mac_oldworld: Drop some variables, BALATON Zoltan, 2020/06/29
- [PATCH v7 5/8] mac_oldworld: Change PCI address of macio to match real hardware, BALATON Zoltan, 2020/06/29
- [PATCH v7 2/8] mac_newworld: Allow loading binary ROM image, BALATON Zoltan, 2020/06/29
- [PATCH v7 1/8] mac_oldworld: Allow loading binary ROM image, BALATON Zoltan, 2020/06/29
- Re: [PATCH v7 1/8] mac_oldworld: Allow loading binary ROM image,
Mark Cave-Ayland <=
- [PATCH v7 8/8] mac_oldworld: Add SPD data to cover RAM, BALATON Zoltan, 2020/06/29
- [PATCH v7 6/8] i2c: Match parameters of i2c_start_transfer and i2c_send_recv, BALATON Zoltan, 2020/06/29
- [RFC PATCH v7 7/8] WIP macio/cuda: Attempt to add i2c support, BALATON Zoltan, 2020/06/29
- Re: [PATCH v7 0/8] Mac Old World ROM experiment, Mark Cave-Ayland, 2020/06/30