From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933594Ab3HHGeT (ORCPT ); Thu, 8 Aug 2013 02:34:19 -0400 Received: from lgeamrelo02.lge.com ([156.147.1.126]:42502 "EHLO LGEAMRELO02.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755822Ab3HHGeR (ORCPT ); Thu, 8 Aug 2013 02:34:17 -0400 X-AuditID: 9c93017e-b7b62ae000000eeb-04-52033be7eb16 From: Namhyung Kim To: Arnaldo Carvalho de Melo Cc: Peter Zijlstra , Paul Mackerras , Ingo Molnar , Namhyung Kim , LKML , Pekka Enberg , Andi Kleen , Jiri Olsa Subject: [PATCH/RFC 0/3] perf ui/gtk: Separate out GTK code to a shared object (v3) Date: Thu, 8 Aug 2013 15:34:07 +0900 Message-Id: <1375943650-2273-1-git-send-email-namhyung@kernel.org> X-Mailer: git-send-email 1.7.11.7 X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, This is v3 of gtk code separation patchset to reduce library dependencies of the perf executable. I only built libperf-gtk.so with -fPIC, and it's not linked to libperf at build time. All unresolved symbols used for perf should be resolved at runtime via perf executable (so libperf.a) - I didn't know that the linker permits unresolved symbols in a shared library at build time. Tested on my x86-64 machine only. It seems work well for me. v3 changes: * fix build failure on O= option * fix build failure on NO_GTK2= option * add Reviewed-by tag from Pekka * patch 1 of previous series already merged You can find it on my 'perf/separate-v3' branch in my tree at: git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git Any comments are welcome, thanks Namhyung Cc: Pekka Enberg Cc: Andi Kleen Namhyung Kim (3): perf tools: Separate out GTK codes to libperf-gtk.so perf tools: Setup GTK browser dynamically perf tools: Run dynamic loaded GTK browser tools/perf/Makefile | 39 +++++++++++++++++++++++---------- tools/perf/builtin-annotate.c | 26 +++++++++++++++++++--- tools/perf/builtin-report.c | 16 ++++++++++++-- tools/perf/config/Makefile | 12 ++++++++--- tools/perf/ui/gtk/annotate.c | 13 ++++++++--- tools/perf/ui/gtk/gtk.h | 16 ++++++++++++++ tools/perf/ui/setup.c | 50 +++++++++++++++++++++++++++++++++++++++++-- tools/perf/ui/ui.h | 12 +---------- tools/perf/util/annotate.h | 24 --------------------- tools/perf/util/hist.h | 15 ------------- 10 files changed, 149 insertions(+), 74 deletions(-) -- 1.7.11.7