Re: [hatari-devel] Adding cache support for the MegaSTE -> DONE

[Nicolas Pomarède <npomarede@xxxxxxxxxxxx>]

Le 04/09/2024 à 01:17, Eero Tamminen a écrit :
> Hi,
>
> On 4.9.2024 0.12, Nicolas Pomarède wrote:
> > Note that the cache needs to be flushed on every transfer when using 
> > Gemdos HD emulation, or normal FDC / HDC. Else this would transfer 
> > data to RAM without invalidating the cache (which has no means to know 
> > RAM has been modified), thus creating crashes in the emulated program 
> > sooner or later.
>
> GEMDOS HD (and NF SCSI driver) are already calling cache invalidation 
> functions...?
>
> Otherwise things would not have worked properly for 030+ caches.
>
> See "git grep Flush.*Cache".


They were already calling them partly ; through STMemory_WriteLong for 
example, but STMemory_SafeCopy was missing.

so it's quite likely that STMemory_SafeCopy could have messed with the 
caches since quite some time, it's just that by luck it never triggered 
a bug.

Nicolas