123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103 |
- From dcb45256970b15b672d0004533826c94083356e5 Mon Sep 17 00:00:00 2001
- From: Yuanjie Huang <yuanjie.huang@windriver.com>
- Date: Fri, 17 Apr 2015 14:48:20 +0800
- Subject: [PATCH 4/6] avoid failure on symbol provided by application
- Upstream-Status: Pending
- Undefined symbols in a library can be provided by the application
- that links to the library, such as `logsink' in libmultipath.so.0.
- This fix checks the type of object in which the symbol is needed
- and the existence of the symbol in application, when a symbol
- cannot be provided by libraries. It prevents false alarm on absence
- of symbols.
- Signed-off-by: Yuanjie Huang <yuanjie.huang@windriver.com>
- ---
- src/mklibs | 28 ++++++++++++++++++++++++----
- 1 file changed, 24 insertions(+), 4 deletions(-)
- diff --git a/src/mklibs b/src/mklibs
- index a3533c0..66b7a09 100755
- --- a/src/mklibs
- +++ b/src/mklibs
- @@ -133,9 +133,9 @@ class Symbol(object):
- return '@'.join(ret)
-
- class UndefinedSymbol(Symbol):
- - def __init__(self, name, weak, version, library):
- + def __init__(self, name, weak, version, library, object):
- super(UndefinedSymbol, self).__init__(name, version, library)
- - self.weak, self.library = weak, library
- + self.weak, self.library, self.object = weak, library, object
-
- def symbol_is_blacklisted(name):
- # The ARM Embedded ABI spec states symbols under this namespace as
- @@ -152,6 +152,11 @@ def undefined_symbols(obj):
-
- output = command("mklibs-readelf", "--print-symbols-undefined", obj)
-
- + if len(obj) > len(dest_path) and obj[:len(dest_path)] == dest_path:
- + object = obj[len(dest_path) + 1:-len('-so-stripped')]
- + else:
- + object = obj
- +
- result = []
- for line in output:
- name, weak_string, version_string, library_string = line.split()[:4]
- @@ -171,7 +176,7 @@ def undefined_symbols(obj):
- if library_string.lower() != 'none':
- library = library_string
-
- - result.append(UndefinedSymbol(name, weak, version, library))
- + result.append(UndefinedSymbol(name, weak, version, library, object))
-
- return result
-
- @@ -498,12 +503,13 @@ while 1:
- and re.search("^ps_", str(symbol)))
- and not (re.search("ld-linux.so.3$", str(symbol)))
- and not (re.search("^__gnu_local_gp", str(symbol)))):
- - debug(DEBUG_SPAM, "needed_symbols adding %s, weak: %s" % (symbol, symbol.weak))
- + debug(DEBUG_SPAM, "needed_symbols adding %s, weak: %s, for %s" % (symbol, symbol.weak, obj))
- needed_symbols[str(symbol)] = symbol
- libraries.update(library_depends(obj))
-
- # calculate what symbols are present in small_libs and available_libs
- present_symbols = {}
- + present_symbol_progs = {}
- checked_libs = small_libs
- checked_libs.extend(available_libs)
- checked_libs.append(sysroot + ldlib)
- @@ -513,6 +519,12 @@ while 1:
- names = symbol.base_names()
- for name in names:
- present_symbols[name] = symbol
- + if not so_pattern.match(lib):
- + debug(DEBUG_SPAM, "present_symbol_progs adding %s, from executable %s" % (' '.join(names), lib))
- + for name in names:
- + progs = present_symbol_progs.get(name, set())
- + progs.add(lib)
- + present_symbol_progs[name] = progs
-
- # are we finished?
- num_unresolved = 0
- @@ -568,6 +580,14 @@ while 1:
- for name in needed_symbols:
- if not name in symbol_provider:
- if not needed_symbols[name].weak:
- + # WORKAROUND: Undefined symbols in a library can be provided by the application
- + # that links to the library. So if the object which requires the symbol is a library
- + # and some application can provide the symbol, the undefined symbol is skipped.
- + symbol = needed_symbols[name]
- + if so_pattern.match(symbol.object) and present_symbol_progs.get(name, None):
- + debug(DEBUG_SPAM, "symbol %s in library %s is provided by executable %s" \
- + % (name, symbol.object, ' '.join(present_symbol_progs[name])))
- + continue
- raise Exception("No library provides non-weak %s" % name)
- else:
- lib = symbol_provider[name]
- --
- 2.16.1
|