Index out of bounds defining multiple flash/sram peripherals about renode

[SandyWalsh]

I reduced the number of sram entries to 3 and it was fine

sram0: Memory.MappedMemory @ sysbus 0x24000000
    size: 0x80000
sram1: Memory.MappedMemory @ sysbus 0x30000000
    size: 0x40000
sram7: Memory.MappedMemory @ sysbus 0x38800000
    size: 0x1000

[SandyWalsh]

Looks like a fencepost error here
https://github.com/renode/renode-infrastructure/blob/c2e1669c54ac8740018b7ce4e8fcfacc867d533d/src/Emulator/Main/Peripherals/Memory/MappedMemory.cs#L533-L561

[mateusz-holenko]

Hi @SandyWalsh, thanks for reporting the issue.

We believe the main problem here is not the amount of flash/sram peripherals (there should be no hard-limits on the amount of them in Renode), but rather the fact that some of them are defined with size set to 0.
Of course this should not crash Renode and be handled in a more user-friendly way instead - the fix for this will be released soon.

For the record: we also see you have many peripherals defined at the same offset on the system bus, so the presented platform file needs further updates before it can be correctly loaded into Renode.

[mateusz-holenko]

With 44e9126 you should not observe crashes of Renode anymore.