mfld-fr / emu86

Intel IA16 emulator for embedded development
35 stars 6 forks source link

[rom] Implement some BIOS services #51

Closed cocus closed 3 years ago

cocus commented 3 years ago

Implements int10h 0,1,7; int13h 4, 0x15; int14h 0; int16h 2; int17h 1; int1ah 2, 4. MS-DOS 6.22 boots to an A:> prompt!

cocus commented 3 years ago

Well, the build error comes when trying to fetch an outdated (I think) package? Maybe there's something missing in the action to do a fetch? Or the repos are outdated somehow.

ghaerr commented 3 years ago

Hello @cocus,

Wow, you've added quite a few new enhancements to the ROM BIOS, as well as some non- or partially-implemented ones. Nice work getting MSDOS to boot!

I am thinking that perhaps this entire PR should be contributed as a seperate rom-pc.c, instead of changes to rom-elks.c, at least for now. The reasons for this are:

If these changes were to temporarily be committed as entirely as a new 'rom-pc.c', then you could work separately on it until it works well, and IMO the EMU86 project could gain another target: generic PC (rather than just ELKS). On a longer term basis, the files could be diff'd, or perhaps even the BIOS INT functions could be kept in seperate files and gathered together in some way. Just some thoughts.

@mfld-fr will have to be the decider for what to do. Let us see what he wants.

Thank you very much for this contribution!

cocus commented 3 years ago

Yeah, I can migrate this to a "generic pc" architecture rom file, even tho it's going to be really minimalistic, since I don't want to have all the whistles as a fully fledged PC emulator (particularly because I want to use whatever I'm running on my 80C188 board, and it's going to be difficult, almost impossible, to emulate a PC properly).

ghaerr commented 3 years ago

Well, perhaps "pc" or "generic pc" isn't the right name for it. Lets see what @mfld-fr wants to do before you change it. Perhaps a discussion about targets is in order.

Thanks!

mfld-fr commented 3 years ago

Build error gone after retrying the pipe, looks like a GitHub transient failure.

mfld-fr commented 3 years ago

@cocus : please copy the existing rom-elks.c to a new rom-or556.c (same name as your TARGET), and add your improvements in that latest, so @ghaerr could still manage the ELKS dependencies on the BIOS in rom-elks.c.

We will consider a more generic PC BIOS support when the dust would have settle a bit...

mfld-fr commented 3 years ago

Related to issue #30

mfld-fr commented 3 years ago

Please hold on this PR, I am picking the changes from it to #56

mfld-fr commented 3 years ago

Changes picked and superseded by #56