Support non-bootable disks #1
Comments
simonowen
added a commit
that referenced
this issue
Mar 9, 2024
Changed the patching to support data disks, which still require a boot sector DPB to work with a previously booted patched version of CP/M 3. We now check the boot sector looks normal enough to patch, with either a byte filler or a a common decrementing byte pattern. Only then do we add the DPB to the end of the sector. This can still be overridden using the --force option if the user decides it's safe. We previously required both CPM3.SYS and CPMLDR to be present and patchable before patching the boot sector. This has been relaxed to patch them only if they are present. The patched disk image is now only written if anything was patched.
|
Fixed in v1.1. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The current script expects to find cpm3.sys and cpmldr to patch before it considers writing the DPB to the boot sector. Non-bootable data only disks might not have them but would still benefit from the DPB, which would be picked up if the patched CP/M 3 was booted.
After this is relaxed more care should be taken before writing to the end of the boot sector. Currently 0xE5 filler or a decrementing byte pattern is likely safe, and other cases should be refused unless forced.
The text was updated successfully, but these errors were encountered: