Patchwork Dynamic Binary Instrumentation

login
register
mail settings
Submitter 陳韋任
Date Sept. 25, 2012, 4:44 p.m.
Message ID <20120925164435.GA72354@cs.nctu.edu.tw>
Download mbox | patch
Permalink /patch/186853/
State New
Headers show

Comments

陳韋任 - Sept. 25, 2012, 4:44 p.m.
On Fri, Sep 21, 2012 at 03:39:32PM +0200, Lluís Vilanova wrote:
> Sorry, it's up again. The server is low on RAM and Linux' OOM killer kicks in
> sometimes.

  Thanks. I am playing around your qemu-dbi, and has error below.

$ make
  LINK  i386-softmmu/qemu-system-i386
translate.o: In function `trace_tcg_instr__before':
/tmp/chenwj/qemu-dbi/trace/tcg-instr-internal.h:134: undefined reference to `trace_tcg_vbbl_before'

Seems some files or functions are missing? Also, do you consider
move your repo to github or other more reliable server?

Regards,
chenwj 

P.S. Attach is a minor patch which fix a typo (I guess).
Lluís Vilanova - Sept. 25, 2012, 7:19 p.m.
陳韋任 (Wei-Ren Chen) writes:

> On Fri, Sep 21, 2012 at 03:39:32PM +0200, Lluís Vilanova wrote:
>> Sorry, it's up again. The server is low on RAM and Linux' OOM killer kicks in
>> sometimes.

>   Thanks. I am playing around your qemu-dbi, and has error below.

> $ make
>   LINK  i386-softmmu/qemu-system-i386
> translate.o: In function `trace_tcg_instr__before':
> /tmp/chenwj/qemu-dbi/trace/tcg-instr-internal.h:134: undefined reference to `trace_tcg_vbbl_before'

How did you configure it?

After applying a small build patch, this works for me:

  cp ~/qemu-dbi/trace-events ~/
  sed -i -e "s/disable tcg vmem(/instrument tcg vmem(/g" trace-events
  mkdir -p ~/qemu-dbi-build
  cd ~/qemu-dbi-build
  ~/qemu-src/configure --with-trace-events=../trace-events --with-trace-instrument=dynamic --prefix=/tmp/qemu-install --enable-trace-backend=stderr --target-list="i386-softmmu"
  make
  make install

I've update the repo with the aforementioned patch.


> Seems some files or functions are missing? Also, do you consider
> move your repo to github or other more reliable server?

I might do it in the future, but for now my todo is full enough with other
tasks :)



> P.S. Attach is a minor patch which fix a typo (I guess).

Right, I should proofread the docs.



Thanks,
  Lluis

Patch

diff --git a/docs/instrumentation.txt b/docs/instrumentation.txt
index 58ce4cb..c2e79b0 100644
--- a/docs/instrumentation.txt
+++ b/docs/instrumentation.txt
@@ -39,7 +39,7 @@  property, if any) in the events listed in the "trace-events" file.
 In order to avoid modifying the QEMU sources, you can simply create a new
 trace-events file with your modifications:
 
-    cp /path/to/qemu-source/ /tmp/trace-events
+    cp /path/to/qemu-source/trace-events /tmp/trace-events
     sed -i -e "s/qemu_vmalloc(/instrument qemu_vmalloc(/g" /tmp/trace-events
     sed -i -e "s/disable tcg vbbl_before(/instrument tcg vbbl(/g" /tmp/trace-events
     sed -i -e "s/disable tcg vinstr_before_before(/instrument tcg vinstr_before(/g" /tmp/trace-events