Re: [hatari-devel] defaulting to SMALL_MEM

[M Grove <1982kx250@xxxxxxxxx>]

Hello all,

Only recently signed up to the mailing list, hope this goes out to all :)

The memory error seems related to post already made. Did cause me to think and ask if TOS 2.07 (Sparrow TOS) has been tested?

According to documents, Sparrow supports 1,4,8,14 MB of ST RAM, where the Falcon 1,4,14.

Doubt anything related to the Timer discussion, just thought I'd put this out here.

On Mon, Feb 7, 2022, 6:40 AM Eero Tamminen <oak@xxxxxxxxxxxxxx> wrote:

Hi,

On 6.2.2022 9.25, Thomas Huth wrote:
> Am Sat, 5 Feb 2022 10:42:24 +0100
> schrieb Christian Zietz <czietz@xxxxxxx>:
>
>> Eero Tamminen schrieb:
>>
>>> I think I bumped into additional SMALL_MEM issue:
>>> --------------------
>>> $ hatari -s 10 --machine falcon --tos tos404.img .
>>> WARN : unsupported Falcon ST-RAM amount 10240, changing to 14336 KB
>>> Segmentation fault
>>> --------------------
>>
>> There seems to be something amiss with the Falcon emulation in
>> general. Even if I start Hatari with "-s 4 --machine falcon --tos
>> TOS404.IMG", i.e, a supported RAM size, I just get a black window for
>> a few seconds which then closes without any error message. In
>> contrast, EmuTOS boots under Falcon emulation.
>>
>> This is with commit 175b878a, either using my own Windows builds or
>> the one from http://antarctica.no/~hatari/.
>
> Sorry, I can't reproduce neither of the reported crashes on my Linux
> system here. Also running Hatari with "valgrind" does not show anything
> suspicious here...
> Thanks for the backtrace, Christian, but I'm having a hard time to
> understand where the exact problem comes from...
> Could you both please send me your hatari.cfg and hatari.nvram files?
> Maybe the problem only triggers with a certain setting that I do not
> have enabled here...

My nvram & hatari.cfg are attached.


For some reason, these Valgrind errors happen only with TOS4, not with
EmuTOS:
----------------------------------------------
$ valgrind ./hatari  -s 14 --machine falcon --tos tos404.img --dsp none .
==6209== Memcheck, a memory error detector
==6209== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==6209== Using Valgrind-3.16.1 and LibVEX; rerun with -h for copyright info
==6209== Command: ./src/hatari -s 14 --machine falcon --tos
/home/emulation/atari/tos/tos404.img --dsp none debugger-scripts/
==6209==
==6209== Invalid read of size 4
==6209==    at 0x4B5BC9: Screen_BitplaneToChunky32 (screenConvert.c:205)
==6209==    by 0x4B6E1B: ScreenConv_BitplaneLineTo32bpp
(screenConvert.c:342)
==6209==    by 0x4B6E1B: ScreenConv_BitplaneTo32bppZoomed
(screenConvert.c:787)
==6209==    by 0x4B6E1B: Screen_ConvertWithZoom (screenConvert.c:1049)
==6209==    by 0x4B6E1B: Screen_GenConvert (screenConvert..c:1079)
==6209==    by 0x4CF9CC: VIDEL_renderScreen (videl.c:1002)
==6209==    by 0x4C5D74: Video_DrawScreen (video.c:4373)
==6209==    by 0x4C5D74: Video_InterruptHandler_VBL (video.c:4546)
==6209==    by 0x47CC2E: CycInt_CallActiveHandler (cycInt..c:790)
==6209==    by 0x4FA7E4: CycInt_Process_stop (cycInt.h:135)
==6209==    by 0x4FA7E4: m68k_run_2ce (newcpu.c:6857)
==6209==    by 0x4F81A0: m68k_go (newcpu.c:7607)
==6209==    by 0x4A2EAC: M68000_Start (m68000.c:301)
==6209==    by 0x4A4409: main (main.c:964)
==6209==  Address 0x35484040 is 0 bytes after a block of size 14,680,064
alloc'd
==6209==    at 0x483877F: malloc (vg_replace_malloc.c:307)
==6209==    by 0x517374: memory_init (memory.c:1662)
==6209==    by 0x4BD174: TOS_InitImage (tos.c:1133)
==6209==    by 0x4AE837: Reset_ST (reset.c:62)
==6209==    by 0x4AE9BC: Reset_Cold (reset.c:141)
==6209==    by 0x4A4120: Main_Init (main.c:757)
==6209==    by 0x4A4120: main (main.c:946)
==6209==
==6209== Invalid read of size 4
==6209==    at 0x4B5BCB: Screen_BitplaneToChunky32 (screenConvert.c:206)
==6209==    by 0x4B6E1B: ScreenConv_BitplaneLineTo32bpp
(screenConvert.c:342)
==6209==    by 0x4B6E1B: ScreenConv_BitplaneTo32bppZoomed
(screenConvert.c:787)
==6209==    by 0x4B6E1B: Screen_ConvertWithZoom (screenConvert.c:1049)
==6209==    by 0x4B6E1B: Screen_GenConvert (screenConvert..c:1079)
==6209==    by 0x4CF9CC: VIDEL_renderScreen (videl.c:1002)
==6209==    by 0x4C5D74: Video_DrawScreen (video.c:4373)
==6209==    by 0x4C5D74: Video_InterruptHandler_VBL (video.c:4546)
==6209==    by 0x47CC2E: CycInt_CallActiveHandler (cycInt..c:790)
==6209==    by 0x4FA7E4: CycInt_Process_stop (cycInt.h:135)
==6209==    by 0x4FA7E4: m68k_run_2ce (newcpu.c:6857)
==6209==    by 0x4F81A0: m68k_go (newcpu.c:7607)
==6209==    by 0x4A2EAC: M68000_Start (m68000.c:301)
==6209==    by 0x4A4409: main (main.c:964)
==6209==  Address 0x35484044 is 4 bytes after a block of size 14,680,064
alloc'd
==6209==    at 0x483877F: malloc (vg_replace_malloc.c:307)
==6209==    by 0x517374: memory_init (memory.c:1662)
==6209==    by 0x4BD174: TOS_InitImage (tos.c:1133)
==6209==    by 0x4AE837: Reset_ST (reset.c:62)
==6209==    by 0x4AE9BC: Reset_Cold (reset.c:141)
==6209==    by 0x4A4120: Main_Init (main.c:757)
==6209==    by 0x4A4120: main (main.c:946)
==6209==
==6209== Invalid read of size 4
==6209==    at 0x4B5BC9: Screen_BitplaneToChunky32 (screenConvert.c:205)
==6209==    by 0x4B6DB7: ScreenConv_BitplaneLineTo32bpp
(screenConvert.c:332)
==6209==    by 0x4B6DB7: ScreenConv_BitplaneTo32bppZoomed
(screenConvert.c:787)
==6209==    by 0x4B6DB7: Screen_ConvertWithZoom (screenConvert.c:1049)
==6209==    by 0x4B6DB7: Screen_GenConvert (screenConvert..c:1079)
==6209==    by 0x4CF9CC: VIDEL_renderScreen (videl.c:1002)
==6209==    by 0x4C5D74: Video_DrawScreen (video.c:4373)
==6209==    by 0x4C5D74: Video_InterruptHandler_VBL (video.c:4546)
==6209==    by 0x47CC2E: CycInt_CallActiveHandler (cycInt..c:790)
==6209==    by 0x4FA7E4: CycInt_Process_stop (cycInt.h:135)
==6209==    by 0x4FA7E4: m68k_run_2ce (newcpu.c:6857)
==6209==    by 0x4F81A0: m68k_go (newcpu.c:7607)
==6209==    by 0x4A2EAC: M68000_Start (m68000.c:301)
==6209==    by 0x4A4409: main (main.c:964)
==6209==  Address 0x35484080 is 14,680,128 bytes inside a block of size
14,684,064 in arena "client"
==6209==
==6209== Invalid read of size 4
==6209==    at 0x4B5BCB: Screen_BitplaneToChunky32 (screenConvert.c:206)
==6209==    by 0x4B6DB7: ScreenConv_BitplaneLineTo32bpp
(screenConvert.c:332)
==6209==    by 0x4B6DB7: ScreenConv_BitplaneTo32bppZoomed
(screenConvert.c:787)
==6209==    by 0x4B6DB7: Screen_ConvertWithZoom (screenConvert.c:1049)
==6209==    by 0x4B6DB7: Screen_GenConvert (screenConvert..c:1079)
==6209==    by 0x4CF9CC: VIDEL_renderScreen (videl.c:1002)
==6209==    by 0x4C5D74: Video_DrawScreen (video.c:4373)
==6209==    by 0x4C5D74: Video_InterruptHandler_VBL (video.c:4546)
==6209==    by 0x47CC2E: CycInt_CallActiveHandler (cycInt..c:790)
==6209==    by 0x4FA7E4: CycInt_Process_stop (cycInt.h:135)
==6209==    by 0x4FA7E4: m68k_run_2ce (newcpu.c:6857)
==6209==    by 0x4F81A0: m68k_go (newcpu.c:7607)
==6209==    by 0x4A2EAC: M68000_Start (m68000.c:301)
==6209==    by 0x4A4409: main (main.c:964)
==6209==  Address 0x35484084 is 14,680,132 bytes inside a block of size
14,684,064 in arena "client"
----------------------------------------------

(I'm not sure, but you may need also to move mouse in the Hatari window
while TOS is booting to trigger above warnings.)


        - Eero