Add a '--with-instrument' configuration option pointing to user-provided instrumentation callbacks.
Make is invoked on the user-provided directory, which must build a static library that might contain extra code needed by the user-provided instrumentation. Signed-off-by: Lluís Vilanova <vilan...@ac.upc.edu> --- Makefile.target | 29 ++++++++++++++++++++++++++--- configure | 19 +++++++++++++++++++ 2 files changed, 45 insertions(+), 3 deletions(-) diff --git a/Makefile.target b/Makefile.target index 9152723..90867e7 100644 --- a/Makefile.target +++ b/Makefile.target @@ -351,12 +351,35 @@ libbackdoor-clean: VPATH=$(VPATH) SRC_PATH=$(SRC_PATH) V="$(V)" clean || true endif +######################################################### +# static instrumentation +ifdef CONFIG_INSTRUMENT +VPATH := $(VPATH):$(INSTRUMENT_PATH) + +LIBINSTRUMENT_LIB = libinstrument/libinstrument.a +LIBINSTRUMENT_CLEAN = libinstrument-clean + +libinstrument/Makefile: + $(call quiet-command, mkdir -p libinstrument, " CREAT $(TARGET_DIR)$@") + $(call quiet-command, rm -f libinstrument/Makefile) + $(call quiet-command, ln -s $(INSTRUMENT_PATH)/Makefile libinstrument/Makefile) + +libinstrument/libinstrument.a: libinstrument/Makefile force + $(call quiet-command,$(MAKE) $(SUBDIR_MAKEFLAGS) -C libinstrument \ + QEMU_CFLAGS="$(QEMU_CFLAGS) -I../target-$(TARGET_BASE_ARCH)" \ + TARGET_DIR=$(TARGET_DIR)libinstrument/ VPATH=$(VPATH) \ + SRC_PATH=$(SRC_PATH) V="$(V)" libinstrument.a) + +libinstrument-clean: + $(MAKE) $(SUBDIR_MAKEFLAGS) -C $(LIBINSTRUMENT_DIR) \ + VPATH=$(VPATH) SRC_PATH=$(SRC_PATH) V="$(V)" clean || true +endif -$(QEMU_PROG)-prepare: $(GENERATED_HEADERS) $(LIBBACKDOOR_LIB) $(QEMU_PROG) +$(QEMU_PROG)-prepare: $(GENERATED_HEADERS) $(LIBBACKDOOR_LIB) $(LIBINSTRUMENT_LIB) $(QEMU_PROG) -$(QEMU_PROG): $(obj-y) $(obj-$(TARGET_BASE_ARCH)-y) $(LIBBACKDOOR_LIB) - $(call LINK,$(obj-y) $(obj-$(TARGET_BASE_ARCH)-y)) $(LIBBACKDOOR_LIB) +$(QEMU_PROG): $(obj-y) $(obj-$(TARGET_BASE_ARCH)-y) $(LIBBACKDOOR_LIB) $(LIBINSTRUMENT_LIB) + $(call LINK,$(obj-y) $(obj-$(TARGET_BASE_ARCH)-y)) $(LIBBACKDOOR_LIB) $(LIBINSTRUMENT_LIB) gdbstub-xml.c: $(TARGET_XML_FILES) $(SRC_PATH)/feature_to_c.sh diff --git a/configure b/configure index 0de937e..454124c 100755 --- a/configure +++ b/configure @@ -333,6 +333,7 @@ trace_file="trace" spice="" backdoor="" +instrument="" # OS specific if check_define __linux__ ; then @@ -750,6 +751,14 @@ for opt do backdoor=`readlink -f $backdoor` fi ;; + --with-instrument=*) instrument="$optarg" + if test ! -f "$instrument/Makefile"; then + echo "ERROR: cannot make into $instrument" + show_help="yes" + else + instrument=`readlink -f $instrument` + fi + ;; *) echo "ERROR: unknown option $opt"; show_help="yes" ;; esac @@ -942,6 +951,7 @@ echo " Default:trace-<pid>" echo " --disable-spice disable spice" echo " --enable-spice enable spice" echo " --with-backdoor=PATH enable backdoor communication and compile implementation in PATH" +echo " --with-instrument=PATH enable static instrumentation and compile user code in PATH" echo "" echo "NOTE: The object files are built at the place where configure is launched" exit 1 @@ -2335,6 +2345,9 @@ echo "spice support $spice" if test -n "$backdoor"; then echo "Backdoor comm. $backdoor" fi +if test -n "$instrument"; then + echo "Instrumentation $instrument" +fi if test $sdl_too_old = "yes"; then echo "-> Your SDL version is too old - please upgrade to have SDL support" @@ -2599,6 +2612,12 @@ if test -n "$backdoor"; then echo "BACKDOOR_PATH=$backdoor" >> $config_host_mak rm -rf *-{linux-user,softmmu}/libbackdoor/ fi +if test -n "$instrument"; then + echo "CONFIG_INSTRUMENT=y" >> $config_host_mak + echo "INSTRUMENT_PATH=$instrument" >> $config_host_mak + QEMU_CFLAGS="-I$instrument $QEMU_CFLAGS" + rm -rf *-{linux-user,softmmu}/libinstrument/ +fi # XXX: suppress that if [ "$bsd" = "yes" ] ; then -- 1.7.1 -- "And it's much the same thing with knowledge, for whenever you learn something new, the whole world becomes that much richer." -- The Princess of Pure Reason, as told by Norton Juster in The Phantom Tollbooth