mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	perf tools: Fix bfd dependency libraries detection
There's false assumption in the library detection code assuming -liberty and -lz are always present once bfd is detected. The fails on Ubuntu (14.04) as reported by Ingo. Forcing the bdf dependency libraries detection any time bfd library is detected. Reported-by: Ingo Molnar <mingo@kernel.org> Tested-by: Ingo Molnar <mingo@kernel.org> Cc: Arnaldo Carvalho de Melo <acme@kernel.org> Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com> Cc: David Ahern <dsahern@gmail.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Ingo Molnar <mingo@kernel.org> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/1398676935-6615-1-git-send-email-jolsa@kernel.org Signed-off-by: Jiri Olsa <jolsa@kernel.org>
This commit is contained in:
		
							parent
							
								
									e27a08f53d
								
							
						
					
					
						commit
						2cf9040714
					
				
					 1 changed files with 23 additions and 11 deletions
				
			
		| 
						 | 
				
			
			@ -194,7 +194,10 @@ VF_FEATURE_TESTS =			\
 | 
			
		|||
	stackprotector-all		\
 | 
			
		||||
	timerfd				\
 | 
			
		||||
	libunwind-debug-frame		\
 | 
			
		||||
	bionic
 | 
			
		||||
	bionic				\
 | 
			
		||||
	liberty				\
 | 
			
		||||
	liberty-z			\
 | 
			
		||||
	cplus-demangle
 | 
			
		||||
 | 
			
		||||
# Set FEATURE_CHECK_(C|LD)FLAGS-all for all CORE_FEATURE_TESTS features.
 | 
			
		||||
# If in the future we need per-feature checks/flags for features not
 | 
			
		||||
| 
						 | 
				
			
			@ -512,7 +515,21 @@ else
 | 
			
		|||
endif
 | 
			
		||||
 | 
			
		||||
ifeq ($(feature-libbfd), 1)
 | 
			
		||||
  EXTLIBS += -lbfd -lz -liberty
 | 
			
		||||
  EXTLIBS += -lbfd
 | 
			
		||||
 | 
			
		||||
  # call all detections now so we get correct
 | 
			
		||||
  # status in VF output
 | 
			
		||||
  $(call feature_check,liberty)
 | 
			
		||||
  $(call feature_check,liberty-z)
 | 
			
		||||
  $(call feature_check,cplus-demangle)
 | 
			
		||||
 | 
			
		||||
  ifeq ($(feature-liberty), 1)
 | 
			
		||||
    EXTLIBS += -liberty
 | 
			
		||||
  else
 | 
			
		||||
    ifeq ($(feature-liberty-z), 1)
 | 
			
		||||
      EXTLIBS += -liberty -lz
 | 
			
		||||
    endif
 | 
			
		||||
  endif
 | 
			
		||||
endif
 | 
			
		||||
 | 
			
		||||
ifdef NO_DEMANGLE
 | 
			
		||||
| 
						 | 
				
			
			@ -523,15 +540,10 @@ else
 | 
			
		|||
    CFLAGS += -DHAVE_CPLUS_DEMANGLE_SUPPORT
 | 
			
		||||
  else
 | 
			
		||||
    ifneq ($(feature-libbfd), 1)
 | 
			
		||||
      $(call feature_check,liberty)
 | 
			
		||||
      ifeq ($(feature-liberty), 1)
 | 
			
		||||
        EXTLIBS += -lbfd -liberty
 | 
			
		||||
      else
 | 
			
		||||
        $(call feature_check,liberty-z)
 | 
			
		||||
        ifeq ($(feature-liberty-z), 1)
 | 
			
		||||
          EXTLIBS += -lbfd -liberty -lz
 | 
			
		||||
        else
 | 
			
		||||
          $(call feature_check,cplus-demangle)
 | 
			
		||||
      ifneq ($(feature-liberty), 1)
 | 
			
		||||
        ifneq ($(feature-liberty-z), 1)
 | 
			
		||||
          # we dont have neither HAVE_CPLUS_DEMANGLE_SUPPORT
 | 
			
		||||
          # or any of 'bfd iberty z' trinity
 | 
			
		||||
          ifeq ($(feature-cplus-demangle), 1)
 | 
			
		||||
            EXTLIBS += -liberty
 | 
			
		||||
            CFLAGS += -DHAVE_CPLUS_DEMANGLE_SUPPORT
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue