Re: [PATCH 1/7] target/ppc: Fix vmul[eo]* instructions marked 2.07

[Mark Cave-Ayland]

On 05/03/2022 06:36, Cédric Le Goater wrote:

> On 3/4/22 18:51, matheus.ferst@eldorado.org.br wrote:
> > From: "Lucas Mateus Castro (alqotel)" <lucas.araujo@eldorado.org.br>
> >
> > Some ISA v2.03 Vector Multiply instructions marked to be ISA v2.07 only.
> > This patch fixes it.
>
> and MacOSX 10 is also fixed.
>
> There are of lot invalid writes when openbios is loaded :
>
>    ...
>    Invalid write at addr 0xB70A8, size 4, region 'ppc_core99.bios', reason: 
> rejected
>    Invalid write at addr 0xB70AC, size 4, region 'ppc_core99.bios', reason: 
> rejected
>    Invalid write at addr 0xB70B0, size 4, region 'ppc_core99.bios', reason: 
> rejected
>    Invalid write at addr 0xB70B4, size 4, region 'ppc_core99.bios', reason: 
> rejected
>    ...
>
> Mark,
>
> shouldn't we model the FW region with RAM instead ?
>
> @@ -162,7 +162,7 @@ static void ppc_core99_init(MachineState
>       memory_region_add_subregion(get_system_memory(), 0, machine->ram);
>
>       /* allocate and load firmware ROM */
> -    memory_region_init_rom(bios, NULL, "ppc_core99.bios", PROM_SIZE,
> +    memory_region_init_ram(bios, NULL, "ppc_core99.bios", PROM_SIZE,
>                              &error_fatal);
>       memory_region_add_subregion(get_system_memory(), PROM_BASE, bios);
>
>
> Thanks,
>
> C.

I don't believe so. The original aim of OpenBIOS was to run as a free replacement 
firmware on real hardware, so what should happen is that OpenBIOS should copy at 
least the bss section into RAM early in the startup process. There could well be 
errors in those calculations as I have tried to adjust them over time though.


ATB,

Mark.