OS_SymbolLookup should also support local symbol lookup

[skliper]

Is your feature request related to a problem? Please describe.
Getting symbol addresses are an integral part of MM and MD. For systems that load modules with local scope this could still be easily supported by looping through the modules w/o requiring app changes.

Currently it just looks for global or static symbols:

osal/src/os/shared/src/osapi-module.c

Lines 352 to 388 in 4cc6dbb

	int32 OS_SymbolLookup(cpuaddr *SymbolAddress, const char *SymbolName)
	{
	int32 return_code;
	int32 staticsym_status;
	/*
	** Check parameters
	*/
	OS_CHECK_POINTER(SymbolAddress);
	OS_CHECK_POINTER(SymbolName);
	/*
	* attempt to find the symbol in the global symbol table.
	*/
	return_code = OS_GlobalSymbolLookup_Impl(SymbolAddress, SymbolName);
	/*
	* If the OS call did not find the symbol or the loader is
	* disabled, then check if a static symbol table is present
	*/
	if (return_code != OS_SUCCESS)
	{
	staticsym_status = OS_SymbolLookup_Static(SymbolAddress, SymbolName, NULL);
	/*
	* Only overwrite the return code if static lookup was successful.
	* Otherwise keep the error code from the low level implementation.
	*/
	if (staticsym_status == OS_SUCCESS)
	{
	return_code = staticsym_status;
	}
	}
	return (return_code);
	} /* end OS_SymbolLookup */

Describe the solution you'd like
Possibly just use OS_ForEach and loop through modules within OS_SymbolLookup, but on systems where local symbol table lookup just calls the global version that would waste cycles if a symbol doesn't exist since it would end up searching the global table for every module.

Consider a slight refactor... a generic OS_SymbolLookup_Impl could search just global on systems with only global, global and local on systems that support it. Really doesn't seem to be a use case for OS_GlobalSymbolLookup_Impl which only searches the global table... possibly remove?

Describe alternatives you've considered
None others.

Additional context
See MM and MD use of symbol lookup, helpful to get from global or local scope.

Requester Info
Jacob Hageman - NASA/GSFC

[skliper]

Really restores former functionality while still being able to unload the module (if loaded w/ global symbols this breaks). No API changes necessary.