Am Tue, 28 Feb 2017 11:23:16 +0100 schrieb Jan Kiszka <[email protected]>:
> On 2017-02-27 16:40, Henning Schild wrote: > > Signed-off-by: Henning Schild <[email protected]> > > > > diff --git a/Documentation/gcov.txt b/Documentation/gcov.txt > > new file mode 100644 > > --- /dev/null > > +++ b/Documentation/gcov.txt > > @@ -0,0 +1,18 @@ > > +$ cd $SRC > > + > > +$ make CONFIG_JAILHOUSE_GCOV=y clean > > +$ make CONFIG_JAILHOUSE_GCOV=y > > +# make CONFIG_JAILHOUSE_GCOV=y install > > $ instead of #, also below. > > I suppose, the CONFIG_JAILHOUSE_GCOV could also go to > jailhouse/config.h, no? If not, can we make that work as well? > Depending on when Andreas sends a new version of his CONFIG > description patch, that should be updated as well. There is a comment for that in patch 8 where it was introduced. The magic that turns the .h into the Makefile-include is in hypervisor/Makefile and the include is used on levels below that. In order to use it on the hypervisor/-level the magic probably has to move up to /. > > + > > +$ find -iname *.gcda -exec rm -f {} + > > + > > +# modprobe jailhouse > > +# jailhouse enable configs/this.cell > > +# jailhouse ... > > +# jailhouse disable > > +# cp /sys/devices/jailhouse/jailhouse.bin /tmp/ > > +$ jailhouse-gcov-extract /tmp/jailhouse.bin > > Why copying first? Can "jailhouse gcov extract" work against the local > sysfs path by default? It uses mmap, which is currently not implemented for that particular sysfs-file. But i can look into that. At first glance i was afraid it would be a lot of complex code to avoid that copy. > > + > > +$ lcov -o /tmp/coverage.info --capture -d . > > +$ genhtml /tmp/coverage.info --output-directory /tmp/coverage/ > > +$ firefox /tmp/coverage/index.html > > > > Jan > -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
