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

MM IntLock removed, no longer implements "write with interrupts disabled" requirement MM2003 #2

Open
skliper opened this issue Apr 22, 2022 · 0 comments
Labels

Comments

@skliper
Copy link
Contributor

skliper commented Apr 22, 2022

Historical - (note the lock/unlock have been removed, see comments):
Memory Manager uses OS_IntLock() and OS_IntUnlock(). That's the only place any of the OS_Int* functions are used in the GSFC apps. These functions will likely be deprecated in the next OSAL release.

I'm guessing it does this with the intent of loading the mem block "atomically" with the hope of preventing another task from writing to it while this happens?

IntLock/Unlock has always been a no-op on POSIX, and furthermore even on platforms where it does something, it will not achieve that exclusivity effect on Multi-Core CPUs, as it only locks the interrupts on the core which calls it, and other cores continue to run anyway, interrupt or not.

Probably worth re-evaulating what MM is trying to achieve with the intlock... might be able to simply take it out with no loss of function.

20220324 - OS_IntLoc and OS_IntUnlock were removed as part of GSFCCFS-1156. Although now requirement GSFCCFS-611 isn't being met.

Imported from GSFCCFS-1132

@skliper skliper added the bug label Apr 22, 2022
@skliper skliper changed the title MM IntLock removed, no longer satisfies GSFCCFS-611 MM IntLock removed, no longer implements "write with interrupts disabled" requirement MM2003 Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant