-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
freedos-specific problems #26
Comments
Sorry, I missed this as I haven't been watching for a while. I'll try out my patch later. |
stsp
added a commit
that referenced
this issue
Dec 7, 2018
init_tos includes init_text, init_data, init_bss and init_stack. Don't add init_text size the second time. This fixes the crash with UMB at A0 as the stack then clashes with this UMB.
stsp
added a commit
that referenced
this issue
Dec 24, 2018
stsp
added a commit
that referenced
this issue
Jan 13, 2019
Add every new MCB as MCB_LAST, instead of MCB_NORMAL. With MCB_NORMAL as last MCB, there is a problem: prev_mcb() may "complete" the chain by finding the MCBs from previous boot, unless memset()ed by dosemu: https://github.com/stsp/dosemu2/blob/devel/src/dosext/misc/xms.c#L128 In this case umb_prev appears to be a link even when inserting above umb_max, and umb_next==umb_seg, after which umb_seg is overwritten with a link. Assertion is added to make sure umb_prev is not link when inserting above umb_max.
I think everything here is fixed. |
And I'd say we are quite low on a bug count here, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Tracking bug.
dosemu2/dosemu2#50
https://github.com/stsp/dosemu2/blob/devel/src/dosext/misc/xms.c#L128
dosemu2/dosemu2#478
@andrewbird I've seen you were reporting some
bugs to the original freedos kernel, and even with
patches. How about moving them here and testing
your patches? Redirector works long ago, everything
is set and ready for a try.
The text was updated successfully, but these errors were encountered: