-
Notifications
You must be signed in to change notification settings - Fork 215
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
Optional support for symbol table dump and module address/size info on POSIX systems (adds GNU extension dependence) #1205
Comments
Updated suggestion - Add OS_NotifyEvent OS_EVENT_READ_MODULE_INFO and OS_EVENT_DUMP_SYMBOL_TABLE or similar and put the platform specific logic in the PSP. |
Added @acudmore to help with RTEMS approach/concept. Although the OSAL impact is probably the same (add the OS_NotifyEvent calls and fix in the PSP). |
I started reviewing the existing code and OS_NotifyEvent framework. For RTEMS, with Eric's pull request (#1221 ) we have a complete os-impl-loader.c I still have to look at POSIX. |
Definitely useful on the host, I use them all of the time to dump the symbols from executable images, but I don't think these will work on the targets. Nice tools to be familiar with though! |
I think the OS_NotifyEvent call to the PSP was specific to the linux implementation since the recommended calls are not POSIX, but GNU extensions that we tend to avoid in OSAL. |
Is your feature request related to a problem? Please describe.
Currently checksum app does not work on systems that use os-impl-posix-dl-symtab.c since module address/size data is not filled in, nor is OS_SymbolTableDump supported.
Describe the solution you'd like
Would add GNU extension dependence, but dl_iterate_phdr() could be used to get the information required for both symbol table dump and module address/size. Example for getting module address here: https://stackoverflow.com/questions/19451791/get-loaded-address-of-a-elf-binary-dlopen-is-not-working-as-expected
Describe alternatives you've considered
None
Additional context
We've avoided GNU extension dependence, but if supported on the desired system it would be helpful to have the option for this functionality to work. Another use case is when simulating a system that does have these functions implemented, it's likely desirable to support the capability.
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered: