PSP compiler(Minimalist PSPSDK) crashing compiling UniPCemu? \ VOGONS

PSP compiler(Minimalist PSPSDK) crashing compiling UniPCemu?

Topic actions

First post, by superfury

Posted on 2017-09-01, 16:59

superfury Offline

Rank l33t++

Rank: l33t++
Posts: 5909
Joined: 2014-03-08, 11:25
Location: Netherlands

I've put UniPCemu in it's own folder, with the tools repository (https://bitbucket.org/superfury/tools.git) in parallel with it, so e.g.:
Z:\psp-projects\unipcemu containing the UniPCemu repository
Z:\psp-projects\tools containing the Tools repository(my global project workspace for e.g. PSP compiler support, customized SDL libraries and PSP emulators for testing(although it's probably a pretty old version by now)).

Trying to run the rebuild.bat(which worked fine until some point during the Windows 10 updates a year or so ago), gives me the following(I've cleared out the MinGW files to make it not conflict with the PSP compiler(due to incompatible .o file format):

1File Not Found
2psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/dskimage.o basicio/dskimage.c
3psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/dynamicimage.o basicio/dynamicimage.c
4psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/fopen64.o basicio/fopen64.c
5psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/io.o basicio/io.c
6psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/port_io.o basicio/port_io.c
7psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/port_mapper.o basicio/port_mapper.c
8psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/basicio/staticimage.o basicio/staticimage.c
9psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/bios/bios.o bios/bios.c
10psp-gcc -I. -IC:/pspsdk/psp/sdk/include -O3 -G0 -Wall -IC:/pspsdk/psp/include/SDL -Dmain=SDL_main  -D_PSP_FW_VERSION=150 -c -o ../../projects_build/UniPCemu/bios/biosmenu.o bios/biosmenu.c
11bios/biosmenu.c: In function 'BIOS_breakpoint':
12bios/biosmenu.c:6662: internal compiler error: Segmentation fault
13Please submit a full bug report,
14with preprocessed source if appropriate.
15See <http://gcc.gnu.org/bugs.html> for instructions.
16make: *** [../../projects_build/UniPCemu/bios/biosmenu.o] Error 1

So the compiler is crashing on biosmenu.c for some unknown reason, at a line in the middle between two functions (the "}"-closing line of the BIOS_breakpoint() function).

Why would the compiler crash on that line? Can anyone see what's going wrong? It's probably giving that line for some reason, but I don't know why the compiler itself is crashing? I'm using the latest PSPSDK (0.11.2r3) from https://sourceforge.net/projects/minpspw/ for Windows. Can anyone tell me why this is crashing?

Edit: It seems to be crashing when compiling this function(commenting it out will make it compile without errors):

1void BIOS_breakpoint()
2{
3	char breakpointstr[8+1+4+1+1+1]; //32-bits offset, colon, 16-bits segment, mode if required, Ignore EIP/Ignore address and final character(always zero)!
4	cleardata(&breakpointstr[0],sizeof(breakpointstr));
5	//First, convert the current breakpoint to a string format!
6	switch ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_MODE_SHIFT)) //What mode?
7	{
8		case 0: //No breakpoint?
9			sprintf(breakpointstr,"%04X:%04X",0,0); //seg16:offs16 default!
10			if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREADDRESS_SHIFT)&1) //Ignore address?
11			{
12				strcat(breakpointstr,"M"); //Ignore mode!
13			}
14			else if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREEIP_SHIFT)&1) //Ignore EIP?
15			{
16				strcat(breakpointstr,"I"); //Ignore EIP!
17			}
18			break;
19		case 1: //Real mode?
20			sprintf(breakpointstr,"%04X:%04X",(word)((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_SEGMENT_SHIFT)&SETTINGS_BREAKPOINT_SEGMENT_MASK),(word)((BIOS_Settings.breakpoint&SETTINGS_BREAKPOINT_OFFSET_MASK)&0xFFFF)); //seg16:offs16!
21			if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREADDRESS_SHIFT)&1) //Ignore address?
22			{
23				strcat(breakpointstr,"M"); //Ignore mode!
24			}
25			else if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREEIP_SHIFT)&1) //Ignore EIP?
26			{
27				strcat(breakpointstr,"I"); //Ignore EIP!
28			}
29			break;
30		case 2: //Protected mode?
31			sprintf(breakpointstr,"%04X:%08XP",(word)((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_SEGMENT_SHIFT)&SETTINGS_BREAKPOINT_SEGMENT_MASK),(uint_32)(BIOS_Settings.breakpoint&SETTINGS_BREAKPOINT_OFFSET_MASK)); //seg16:offs16!
32			if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREADDRESS_SHIFT)&1) //Ignore address?
33			{
34				strcat(breakpointstr,"M"); //Ignore mode!
35			}
36			else if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREEIP_SHIFT)&1) //Ignore EIP?
37			{
38				strcat(breakpointstr,"I"); //Ignore EIP!
39			}
40			break;
41		case 3: //Virtual 8086 mode?
42			sprintf(breakpointstr,"%04X:%04XV",(word)((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_SEGMENT_SHIFT)&SETTINGS_BREAKPOINT_SEGMENT_MASK),(word)((BIOS_Settings.breakpoint&SETTINGS_BREAKPOINT_OFFSET_MASK)&0xFFFF)); //seg16:offs16!
43			if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREADDRESS_SHIFT)&1) //Ignore address?
44			{
45				strcat(breakpointstr,"M"); //Ignore mode!
46			}
47			else if ((BIOS_Settings.breakpoint>>SETTINGS_BREAKPOINT_IGNOREEIP_SHIFT)&1) //Ignore EIP?
48			{
49				strcat(breakpointstr,"I"); //Ignore EIP!
50			}
51			break;
52		default: //Just in case!
53			break;
54	}
55	//I is added with bit set when it's done(Ignore EIP).
56
57	BIOSClearScreen(); //Clear the screen!
58	BIOS_Title("Breakpoint"); //Full clear!
59	EMU_locktext();
60	EMU_gotoxy(0, 4); //Goto position for info!

…Show last 73 lines

61	GPU_EMU_printscreen(0, 4, "Address: "); //Show the filename!
62	EMU_unlocktext();
63	byte mode; //The mode to use!
64	word semicolonpos;
65	char *temp;
66	word maxsegmentsize = 4;
67	word maxoffsetsize = 4;
68	word segment; //The segment buffer to load!
69	uint_32 offset; //The offset buffer to load!
70	if (BIOS_InputAddressWithMode(9, 4, &breakpointstr[0], sizeof(breakpointstr)-1)) //Input text confirmed?
71	{
72		if (strcmp(breakpointstr, "") != 0) //Got valid input?
73		{
74			//Convert the string back into our valid numbers for storage!
75			mode = 1; //Default to real mode!
76			byte ignoreEIP = 0;
77			byte ignoreAddress = 0;
78			ignoreEIP = (breakpointstr[strlen(breakpointstr)-1]=='I'); //Ignore EIP?
79			if (ignoreEIP) breakpointstr[strlen(breakpointstr)-1] = '\0'; //Take off the mode identifier!
80			ignoreAddress = (breakpointstr[strlen(breakpointstr)-1]=='M'); //Ignore address?
81			if (ignoreAddress) breakpointstr[strlen(breakpointstr)-1] = '\0'; //Take off the mode identifier!
82			switch (breakpointstr[strlen(breakpointstr)-1]) //Identifier for the mode?
83			{
84				case 'P': //Protected mode?
85					mode = 2; //Protected mode!
86					breakpointstr[strlen(breakpointstr)-1] = '\0'; //Take off the mode identifier!
87					maxoffsetsize = 8; //We're up to 8 hexadecimal values in this mode!
88					goto handlemode;
89				case 'V': //Virtual 8086 mode?
90					mode = 3; //Virtual 8086 mode!
91					breakpointstr[strlen(breakpointstr)-1] = '\0'; //Take off the mode identifier!
92				default: //Real mode?
93					handlemode: //Handle the other modes!
94						temp = &breakpointstr[0]; //First character!
95						for (;(*temp && *temp!=':');++temp); //No seperator yet?
96						if (*temp!=':') //No seperator found?
97						{
98							goto abortcoloninput; //Invalid: can't handle!
99						}
100						if (*(temp+1)=='\0') //Invalid ending?
101						{
102							goto abortcoloninput; //Invalid: can't handle colon at the end!							
103						}
104						//Temp points to the colon!
105						semicolonpos = temp-&breakpointstr[0]; //length up to the semicolon, which should be valid!
106						if ((semicolonpos==0) || (semicolonpos>maxsegmentsize)) //Too long segment?
107						{
108							goto abortcoloninput; //Invalid: can't handle segment length!							
109						}
110						if (((strlen(breakpointstr)-semicolonpos)-1)>maxoffsetsize) //Too long segment?
111						{
112							goto abortcoloninput; //Invalid: can't handle segment length!							
113						}
114
115						breakpointstr[semicolonpos] = '\0'; //Convert the semicolon into an EOS character to apply the string length!
116						segment = hex2int(&breakpointstr[0]); //Convert the number to our usable format!
117						offset = hex2int(&breakpointstr[semicolonpos+1]); //Convert the number to our usable format!
118
119						//Apply the new breakpoint!
120						BIOS_Settings.breakpoint = (((uint_64)mode&3)<<SETTINGS_BREAKPOINT_MODE_SHIFT)|(((ignoreEIP?1LLU:0LLU)<<SETTINGS_BREAKPOINT_IGNOREEIP_SHIFT))|(((ignoreAddress?1LLU:0LLU)<<SETTINGS_BREAKPOINT_IGNOREADDRESS_SHIFT))|(((uint_64)segment&SETTINGS_BREAKPOINT_SEGMENT_MASK)<<SETTINGS_BREAKPOINT_SEGMENT_SHIFT)|((uint_64)offset&SETTINGS_BREAKPOINT_OFFSET_MASK); //Set the new breakpoint!
121						BIOS_Changed = 1; //We've changed!			
122					break;
123			}
124		}
125		else //Unset?
126		{
127			BIOS_Changed = BIOS_Changed||((BIOS_Settings.breakpoint!=0)?1:0); //We've changed!			
128			BIOS_Settings.breakpoint = 0; //No breakpoint!
129		}
130	}
131	abortcoloninput:
132	BIOS_Menu = 35; //Goto CPU menu!
133}

Author of the UniPCemu emulator.
UniPCemu Git repository
UniPCemu for Android, Windows, PSP, Vita and Switch on itch.io

Reply 1 of 3, by superfury

Posted on 2017-09-06, 21:53

superfury Offline

Rank l33t++

Rank: l33t++
Posts: 5909
Joined: 2014-03-08, 11:25
Location: Netherlands

It seems to be faulting when the hex2int lines up to the break instruction directly after it isn't commented out. I've moved that piece of code to a seperate function. Commenting out the function call makes it compile without problems&errors, but enabling the function call ( https://bitbucket.org/superfury/unipcemu/src/ … enu.c?at=master line 6641) makes it crash the compiler still. Commenting it out with // will make the compile successfull(although destroying the whole setting being applied)? Why is the compiler struggling with that simple code(block or function call)?

Author of the UniPCemu emulator.
UniPCemu Git repository
UniPCemu for Android, Windows, PSP, Vita and Switch on itch.io

Reply 2 of 3, by superfury

Posted on 2017-09-23, 16:25

superfury Offline

Rank l33t++

Rank: l33t++
Posts: 5909
Joined: 2014-03-08, 11:25
Location: Netherlands

I've managed to get UniPCemu compiled again, although it's giving lots of little errors:
- I had to disable the Settings menu's breakpoint option, because the compiler will crash if it's used in it's entirety(only on PSP builds).
- I've modified lots of other code in order to make it not complain about lots of stuff. Somehow it complains about the %u sprintf constants being 16-bit, even though it should be 32-bit or 64-bit values(e.g. long or long long variables) that are used with it? Any other gcc-based compiler(Android/Windows) doesn't complain one bit?

It seems to compile somewhat on MInPSPw 0.9.6+, but for some reason it complains about a missing isfinitef function when using version 0.11.2? Afaik isfinitef is supposed to be a #define? It's the libm.a file that's somehow causing problems with the most recent MinPSPw builds? The 0.10.0 version seems to use gcc 4.3.5 according to "psp-gcc -v"?

Author of the UniPCemu emulator.
UniPCemu Git repository
UniPCemu for Android, Windows, PSP, Vita and Switch on itch.io

Reply 3 of 3, by superfury

Posted on 2017-10-01, 22:20

superfury Offline

Rank l33t++

Rank: l33t++
Posts: 5909
Joined: 2014-03-08, 11:25
Location: Netherlands

Just tried some other versions of the minimalist PSPSDK. It seems 0.11.1 and up compiles without errors, but screws up linking because of the missing isfinitef 'function' linkage(should be a macro afaik)?

And the first older version I can find is 0.10.0, which compiles, but complains about nearly every %u (s)printf on 32-bit or 64-bit parameters needing to be an 'unsigned int', which is only 16-bit(or 32-bit at the most), truncating any 64-bit data(and/or 32-bit data)?

Author of the UniPCemu emulator.
UniPCemu Git repository
UniPCemu for Android, Windows, PSP, Vita and Switch on itch.io

Go to top of page Go to top of page

Back to PC Emulation