kitune-san wrote: ↑Mon Jun 13, 2022 1:43 pm
I added keybord reset logic.
With this change, the keyboard controller test would pass.
I have sent you a pull request.
Please try it.
Sorry. 8088 BIOS does not seem to accept keyboard...
Please give me a little more time.
Don't worry, take as much time as you need. As BIOS, you can try the original IBM 8560 or the Juko ST clone, both showed the keyboard error. I have not yet been able to test this pull request.
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 3:09 pm
by spark2k06
The BIOS of the IBM 5160 does accept the keyboard with this fix, it no longer shows error 301, which is the one related to the keyboard... it only shows error 101:
IBM_5160_Code_101.jpg (65.36 KiB) Viewed 5088 times
JUKO ST BIOS does not show keyboard errors now either, I accept pull request.
The next big challenge is to get the SDRAM working, so that the 640Kb of main memory is available for testing larger games
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 5:45 pm
by MicroCoreLabs
The next big challenge is to get the SDRAM working, so that the 640Kb of main memory is available for testing larger games
You will get some nice acceleration if you provide the option to bypass the cycle-accurate BIU and access block-ram located BIOS ROMs and SDRAM at full-speed.
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 6:33 pm
by spark2k06
MicroCoreLabs wrote: ↑Mon Jun 13, 2022 5:45 pm
You will get some nice acceleration if you provide the option to bypass the cycle-accurate BIU and access block-ram located BIOS ROMs and SDRAM at full-speed.
Yes, I'd leave that for the last step of all... optionally enabling turbo mode at the maximum provided by the MCL86.
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 6:47 pm
by spark2k06
I just updated this branch from the main branch, keeping SDRAM enabled:
And this is what happens when you pass the tests of ruud:
ruud test sdram 01.jpg (86.44 KiB) Viewed 4949 times
I have even seen it go up to 14Kb. However, every test is different, I've also seen it go as low as 2Kb before displaying the error.
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 7:07 pm
by MicroCoreLabs
Does your SDRAM controller have exclusive access to the physical SDRAM or does it share it with the SOC-CPU? If it is sharing, then when there are conflicts then you may need to assert wait-states back to the BIU. It would be better if the SDRAM access was exclusive to the MCL86, otherwise it may be difficult to be consistently cycle accurate if there is a CPU which can delay the BIU's accesses and cause wait states.
On the other hand, if the SDRAM is fast enough, it may be transparent to the 4.77Mhz BIU!
Re: MiSTer PCXT
Posted: Mon Jun 13, 2022 7:36 pm
by spark2k06
MicroCoreLabs wrote: ↑Mon Jun 13, 2022 7:07 pm
Does your SDRAM controller have exclusive access to the physical SDRAM or does it share it with the SOC-CPU? If it is sharing, then when there are conflicts then you may need to assert wait-states back to the BIU. It would be better if the SDRAM access was exclusive to the MCL86, otherwise it may be difficult to be consistently cycle accurate if there is a CPU which can delay the BIU's accesses and cause wait states.
On the other hand, if the SDRAM is fast enough, it may be transparent to the 4.77Mhz BIU!
SDRAM is independent of the MiSTer SOC CPU
Re: MiSTer PCXT
Posted: Tue Jun 14, 2022 11:30 am
by wark91
Is someone have the steps to produce the bios of JUKO ST.
On Mame, the file is just 8Kb but the core required 64Kb for the bios file.
ROM_START( juko8 )
ROM_REGION(0x10000, "bios", 0)
// 0: BIOS ver 2.00 VEGAS COMPUTER COMMUNICATIONS.
ROM_SYSTEM_BIOS(0, "nestv200", "JUKO NEST v2.00")
ROMX_LOAD( "jukoa.bin", 0xe000, 0x2000, CRC(7d78707e) SHA1(8b09a32658a850e7f03254d1328fe6e336e91871),ROM_BIOS(0))
// 1: Flytek (Protek) ST-12 (a 15MHz ST-15 was also available)
ROM_SYSTEM_BIOS(1, "st-12", "ST-12")
ROMX_LOAD( "flytek_st-12_bios_ver_2.20_c_nel_electronics_ltd.bin", 0xe000, 0x2000, CRC(448c3089) SHA1(779d4138d841783d0e2e5ad29c83d9a8cb4497b6), ROM_BIOS(1))
// 2: Juko ST BIOS ver 2.30 / Copyright 1988 Juko Electronics Industrial Co., Ltd.
ROM_SYSTEM_BIOS(2, "nest230", "JUKO NEST v2.30")
ROMX_LOAD( "juko_st_v2.30.bin", 0xe000, 0x2000, CRC(7a1c6dfa) SHA1(0b343f3028ca06c9e6dc69427d1b15a47c74b9fc),ROM_BIOS(2))
// 3: BIOS Ver 2.32
ROM_SYSTEM_BIOS(3, "nest232", "JUKO NEST v2.32")
ROMX_LOAD( "xt-juko-st-2.32.bin", 0xe000, 0x2000, CRC(0768524e) SHA1(259520bb7a6796e5b987c2b9bef1acd501df1670),ROM_BIOS(3))
ROM_END
I managed to do it for 5160 because it is just two files of 32Kb to merge.
Thanks !
Re: MiSTer PCXT
Posted: Tue Jun 14, 2022 2:03 pm
by kitune-san
It's speculation because I don't know the details of that BIOS.
Program will not work if there is no data at entry address FFFF0.
Perhaps it works if you fill the first 56k of the file with zeros and merge the ROM file into the remaining 8k.
Exactly, that's the right thing to do, as I suggested in a previous post:
spark2k06 wrote: ↑Thu Jun 09, 2022 4:40 am
Finally, I attach the latest version of the XTIDE Universal BIOS and HD detection via the serial port by default, which can be loaded into memory from the OSD menu, as it will be loaded from address F0000h. The same you can do with JukoST, in this case you would have to prepare with a hex editor a 64Kb ROM with the XTIDE Universal BIOS at the beginning and JukoST at the end (from FE000h) and the same, you can load it in memory from the OSD menu.
kitune-san wrote: ↑Mon Jun 13, 2022 1:43 pm
I added keybord reset logic.
With this change, the keyboard controller test would pass.
I have sent you a pull request.
Please try it.
Sorry. 8088 BIOS does not seem to accept keyboard...
Please give me a little more time.
I found the cause.
The reason is that 8088bios disables the interrupt mask after keyboard reset.
With the previous fix for 8259, interrupt edges during masking are not accepted. (Meaning that no interrupts are accepted after the mask is unmasked.)
The modification of the keyboard controller this time is such that 0xAA is received from the keyboard after the keyboard reset pulse(port_b[6]) is output.
As a result, interrupts from the keyboard are received before the mask is unmasked.
Then, since interrupts are not accepted, the system enters a deadlock state in which no data is received from the keyboard thereafter.
Re: MiSTer PCXT
Posted: Tue Jun 14, 2022 3:32 pm
by kitune-san
As a countermeasure, I am considering the following
- No forced LOW of clock by IRQ
- If the next data is received before the interrupt is released, temporarily set the IRQ signal LOW. (At this time, the scan code is 0xFF.)
EDIT:
I will try to fix 8259 module.
Re: MiSTer PCXT
Posted: Wed Jun 15, 2022 4:01 pm
by spark2k06
kitune-san wrote: ↑Tue Jun 14, 2022 3:32 pm
As a countermeasure, I am considering the following
- No forced LOW of clock by IRQ
- If the next data is received before the interrupt is released, temporarily set the IRQ signal LOW. (At this time, the scan code is 0xFF.)
Overall kernel performance is similar, IBM 5160 BIOS error 101 still occurs... but the downside is that the interrupt level 0 error in the SuperSoft test ROM has returned with this patch.
Re: MiSTer PCXT
Posted: Thu Jun 16, 2022 4:05 am
by spark2k06
spark2k06 wrote: ↑Wed Jun 15, 2022 4:01 pm
I have tested the pull request you created:
Overall kernel performance is similar, IBM 5160 BIOS error 101 still occurs... but the downside is that the interrupt level 0 error in the SuperSoft test ROM has returned with this patch.
On the other hand, the one that causes the problems with Interrupt Level 0 in the SuperSoft test is the one called "Fixed 8259 interrupt request logic", the other commit also passes the tests.
I see that you are focusing on the performance of the ROM that I have left set by default, that of a clone of 8088 PCXT of Sergey Kiselev, I keep it because it is Open Source, but it may have its bugs or incompatibilities with this core. In fact, one thing I have noticed with this BIOS is that the execution of the DIR command has a quite slow scroll, compared for example with the consolidated Juko ST BIOS, which is the one I am starting to use for all the tests.
In fact, this BIOS doesn't have any problems with the keyboard, like Sergey's 8088 ROM does. Besides, with the last changes you made, it doesn't show any keyboard error, and the original IBM 5160 BIOS doesn't show 301 code either, so, in this sense I think we could consider the keyboard problem solved, and I could accept the commit "Fixed interrupt_to_cpu to be set back to LOW when interrupt request disappears" if you consider it convenient.
I assume Sergey's 8088 BIOS has some "bugs" that might require future revision to make it compatible with this core.
Re: MiSTer PCXT
Posted: Thu Jun 16, 2022 5:04 am
by spark2k06
In summary, the ROMs we should use for core development are:
- SuperSoft tests
- Ruud tests
- IBM 5150/5160 BIOS
- Juko ST BIOS
And finally, in the future, adapt and improve the Sergey's 8088 BIOS, since it is OpenSource and is the current candidate to be kept on GitHub.
Re: MiSTer PCXT
Posted: Fri Jun 17, 2022 12:06 pm
by spark2k06
beta 0.9
640Kb main memory with SRAM, until SDRAM is available... but perfect for porting to other FPGA
I have created a SRAM branch with this core version, it is only possible to use it with the dual memory addon 32Mb SDRAM and 2 Mb SRAM:
SDRAM_SRAM_DUAL.png (264.75 KiB) Viewed 4042 times
Re: MiSTer PCXT
Posted: Fri Jun 17, 2022 5:35 pm
by MicroCoreLabs
With 640KB the 8088 MPH Demo should be able to run to completion. The music at the end uses the prefetch queue extensively, so it's a good test of the BIU.
Re: MiSTer PCXT
Posted: Fri Jun 17, 2022 6:16 pm
by held
MicroCoreLabs wrote: ↑Fri Jun 17, 2022 5:35 pm
With 640KB the 8088 MPH Demo should be able to run to completion. The music at the end uses the prefetch queue extensively, so it's a good test of the BIU.
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 2:11 am
by kitune-san
@spark2k06
The problem with SDRAM not working is caused by the lack of timing constraints.
In my environment, it works fine by adding the following to PCXT.sdc.
kitune-san wrote: ↑Sat Jun 18, 2022 2:11 am
@spark2k06
The problem with SDRAM not working is caused by the lack of timing constraints.
In my environment, it works fine by adding the following to PCXT.sdc.
Humm... I disabled signaltap and it no longer passes.
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 2:51 am
by kitune-san
Humm... I rebased the commit and disabled signaltap and it stopped working.
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 3:34 am
by kitune-san
In the latest commit, the PCXT.sdc file appears to have been removed from the project.
I added the PCXT.sdc file to the project again and confirmed that SDRAM works.
2022-06-18 123540.png (15.7 KiB) Viewed 3761 times
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 3:38 am
by dmckean
Another fantastic milestone!
Now we just need either floppy or IDE support so those of us without COM adapters can play along at home.
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 4:06 am
by spark2k06
kitune-san wrote: ↑Sat Jun 18, 2022 3:34 am
In the latest commit, the PCXT.sdc file appears to have been removed from the project.
I added the PCXT.sdc file to the project again and confirmed that SDRAM works.
2022-06-18 123540.png
I have synthesised with the latest changes to the branch you have created:
If you have a working version, can you update this branch with the changes, or... can you provide the synthesised .rbf file here for me to test?
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 4:18 am
by thorr
dmckean wrote: ↑Sat Jun 18, 2022 3:38 am
Now we just need either floppy or IDE support so those of us without COM adapters can play along at home.
I am not sure if IDE would even work with this. Back then, the drives were RLL or MFM from what I recall.
Re: MiSTer PCXT
Posted: Sat Jun 18, 2022 4:19 am
by spark2k06
MicroCoreLabs wrote: ↑Fri Jun 17, 2022 5:35 pm
With 640KB the 8088 MPH Demo should be able to run to completion. The music at the end uses the prefetch queue extensively, so it's a good test of the BIU.
It turns out that at some point it stopped working as "well" as it used to. It used to go further but now, even with 640Kb of RAM, it stays in this part of the demo, in a loop:
This is the current mismatch with a real 8088, according to the demo benchmark itself:
8088mph_01.jpg (123.04 KiB) Viewed 3728 times
However, the demo not working properly at the moment is the least of the problems, in the end everything can be related, but I prefer that little by little we solve other aspects such as IDE and SDRAM support properly, so that it can start to be used by more users.