coco3: optimisations & fixes #1149
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
February 3, 2025 10:32
No behavioural change, but saves some (albeit discarded) space if CCPT support omitted.
added 3 commits
February 3, 2025 16:10
Split kernel build from disk image construction. Split SRCS lists up into optional (added to DRIVERS by config options) and non-optional. Only depend on specified, but clean all. Instead of rebuilding objects built in Kernel/ with different options, add new rules.mk to override flags for the earlier compile for: mm/bank16k.o timer.o usermem.o NOTE: Kernel/mm/bank16k.o is actually compiled to Kernel/bank16k.o. Along with several other dependencies, this happens due to the way targets are built in Kernel/Makefile (which also means they get rebuilt every time).
In copy_mmu, shuffling register use allows combining stack ops. In switchin(), merged a couple of sequential STA/STB paids into STD. In dofork(), when the register isn't actually required, LEAS 2,S is slightly quicker than PULS X. Only care about top 8 bits of process size when figuring how much to copy. Don't need to stash D in Y if we use BITA to test the remainder instead of ANDA. Use BSR for calls to adjacent copybank.
Slightly optimise init_hardware. Somewhat optimise user process mapping: - Merge map_proc_2 into map_proc_always. - Use quicker 5-bit indexes instead of postinc. - Fetch two pages at a time. Note: map_proc is not currently used, but keeping around as I don't know if there's an intention to use it. Merge tail calls JSR, RTS -> JMP in a couple of places.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Couple of build tweaks.
More optimisations, space where it doesn't affect speed. On top of the 6809 generic ones, visualize6809 even shows a difference...