Skip to content
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

Scas 1.0 tracker #405

Open
6 tasks
pixelherodev opened this issue Sep 11, 2020 · 4 comments
Open
6 tasks

Scas 1.0 tracker #405

pixelherodev opened this issue Sep 11, 2020 · 4 comments
Milestone

Comments

@pixelherodev
Copy link

  • Check hashes against sass-built kernel for each platform

This means using a static KERNEL_VERSION with sass and noting down the hashes of every produced binary. Then, we need to use that same static KERNEL_VERSION with scas and ensure the hashes match. If they don't, looking at the symbol file and debug output should be enough to narrow down the cause.

  • Ensure symbol files are accurate

Symbol files will differ from sass', if only because they don't have the 'produced by sass' line. Furthermore, sass produces symbol files with all-lowercase symbol names, while scas preserves the casing.

For 1.1:

  • Actually implement .echo
  • Fix memory leaks in paths not taken by the kernel
  • Generating listing files
  • Fuzz?
@pixelherodev pixelherodev added this to the Kernel 0.7.0 milestone Sep 11, 2020
@pixelherodev
Copy link
Author

This should be a week away, tops. It's entirely possible scas is already good enough (hence the release candidate), but it's also possible more work will be needed.

@pixelherodev
Copy link
Author

Okay, I've been doing work on libscas, hence the delay. I'm narrowing the scope of this to just have the binary good enough for the kernel for all supported platforms. Anything else is pushed off to 1.1

@pixelherodev
Copy link
Author

Update: after testing on the TI84+, I fixed a few more bugs (hence scas is now on 1.0-rc4). Pages 0 and 1 are 100% correct, but there is a minor difference in the output of the floating point math routines in page 2.

@pixelherodev
Copy link
Author

The kernel is clearly not as representative as I'd thought; castle is disturbingly broken under 1.0-rc4. A full test of KnightOS/KnightOS is needed before marking this as final, and a solution to catch regressions needs to be in place for 1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant