-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix A/UX tablet panic take 2 #5
base: main
Are you sure you want to change the base?
Conversation
Successfully removes driver when A/UX boots. Now to reconnect it afterwards. Successfully reattaches, but does not yet work. incremental removed unnecessary debug info, reverted changed debug outputs Formatted for readability Removed cruft. Added another case where a different control signal value is sent by A/UX to the device Added heap init flag to slotexec, s.t. the device now receives the signal when, for example, the machine goes down for reboot, and it can then handle the finalization correctly. Removed extraneous cases in the switch since it appears they are not invoked. It now comes back up properly after a reboot, but still no dice on after starting A/UX. Reverted having moved InitLog for debugging purposes. Added it to finalize incase it hasn't already been so. Removed extraneous flag for kDriverSupportDMSuspendAndResume, which is not used. Cleaned up doubly-applied gcc unused indicator. Looks like the reboot issue in os8 was a sync problem. So, added an extra IO sync. Removed unused local in VFinal. Added more verbose debug message. Fixed incorrect return value in finalize. Changed control determination to use more accurate consts from the API. Removed unnecessary debug output(s). More reverse-engineering/rtfm of the dev docs trying to get A/UX to recognize the tablet. Revert mask I added on the code switch Revert pointless changes. Justify changes with commentary. Sqaush commits
Thanks for revising the patchset. I really appreciate you addressing this serious deficiency in the drivers. Few thoughts:
|
|
Patrick, thanks for your help with this. I have pushed a change that makes the drivers clean up neatly on close. (I wrote a longer reply to this message but it seems to have disappeared.) Let me know if there are still problems starting A/UX. Moving the cursor while restarting with still cause a dsBadSlotInt though, because the shutdown process doesn't close drivers. I'm not sure what to do about this. |
Warnings also fixed and -Wall enabled. Are you interested in taking a crack at true A/UX driver support? |
Successfully removes driver when A/UX boots. Now to reconnect it afterwards.
Successfully reattaches, but does not yet work.
incremental
removed unnecessary debug info, reverted changed debug outputs
Formatted for readability
Removed cruft. Added another case where a different control signal value is sent by A/UX to the device
Added heap init flag to slotexec, s.t. the device now receives the signal when, for example, the machine goes down for reboot, and it can then handle the finalization correctly. Removed extraneous cases in the switch since it appears they are not invoked. It now comes back up properly after a reboot, but still no dice on after starting A/UX.
Reverted having moved InitLog for debugging purposes. Added it to finalize incase it hasn't already been so.
Removed extraneous flag for kDriverSupportDMSuspendAndResume, which is not used. Cleaned up doubly-applied gcc unused indicator.
Looks like the reboot issue in os8 was a sync problem. So, added an extra IO sync. Removed unused local in VFinal.
Added more verbose debug message. Fixed incorrect return value in finalize.
Changed control determination to use more accurate consts from the API. Removed unnecessary debug output(s). More reverse-engineering/rtfm of the dev docs trying to get A/UX to recognize the tablet.
Revert mask I added on the code switch
Revert pointless changes. Justify changes with commentary. Sqaush commits