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

vxWorks OSAL implementation should use static initialization where possible #172

Closed
skliper opened this issue Sep 30, 2019 · 3 comments
Closed

Comments

@skliper
Copy link
Contributor

skliper commented Sep 30, 2019

OSAL has a concept of tables of resources, where the user can configure some maximum amount of those resources.

For example, the binary semaphore table could contain the static bytes of memory for the semaphore itself (using VX_BINARY_SEMAPHORE or an array of size VX_SEMAPHORE_SIZE taking care to maintain proper alignment) and then use semBInitialize on that memory when the OSAL wants to create a binary semaphore instead of allocating one from the resource pool. The user configures the maximum number of resources of each type, so it's ok to use the memory.

This enhancement would increase determinism in the system while maintaining the current API.

@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Imported from trac issue 149. Created by sseeger on 2016-01-10T01:08:42, last modified: 2019-08-14T14:11:46

@skliper skliper self-assigned this Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Sep 30, 2019

Trac comment by jphickey on 2019-03-26 08:42:29:

Just saw this ticket. I totally agree and in fact vxworks-ng (not yet merged, ticket #231) is implemented using static memory for the semaphores just as described.

@skliper skliper assigned jphickey and unassigned skliper Sep 30, 2019
@skliper
Copy link
Contributor Author

skliper commented Oct 22, 2019

Confirmed current implementation uses tables. Closing this issue as no longer applicable (invalid).

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

No branches or pull requests

2 participants