From b19f1607c6cf3d125af387046cd26797354bd252 Mon Sep 17 00:00:00 2001 From: Nguyen Anh Quynh Date: Mon, 19 Dec 2016 20:30:46 +0800 Subject: [PATCH 1/3] Revert "remove qemu/util/qemu-timer-common.c" This reverts commit 934fa2c90f0c49ae209f697d44d681ddbcdea2d1. --- qemu/util/Makefile.objs | 2 +- qemu/util/qemu-timer-common.c | 61 +++++++++++++++++++++++++++++++++++ 2 files changed, 62 insertions(+), 1 deletion(-) create mode 100644 qemu/util/qemu-timer-common.c diff --git a/qemu/util/Makefile.objs b/qemu/util/Makefile.objs index 0c9866f5..9f4021b9 100644 --- a/qemu/util/Makefile.objs +++ b/qemu/util/Makefile.objs @@ -1,4 +1,4 @@ -util-obj-y = cutils.o unicode.o +util-obj-y = cutils.o unicode.o qemu-timer-common.o util-obj-$(CONFIG_WIN32) += oslib-win32.o qemu-thread-win32.o util-obj-$(CONFIG_POSIX) += oslib-posix.o qemu-thread-posix.o util-obj-y += module.o diff --git a/qemu/util/qemu-timer-common.c b/qemu/util/qemu-timer-common.c new file mode 100644 index 00000000..95e0847c --- /dev/null +++ b/qemu/util/qemu-timer-common.c @@ -0,0 +1,61 @@ +/* + * QEMU System Emulator + * + * Copyright (c) 2003-2008 Fabrice Bellard + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + */ +#include "qemu/timer.h" + +/***********************************************************/ +/* real time host monotonic timer */ + +#ifdef _WIN32 + +int64_t clock_freq; + +static void __attribute__((constructor)) init_get_clock(void) +{ + LARGE_INTEGER freq; + int ret; + ret = QueryPerformanceFrequency(&freq); + if (ret == 0) { + fprintf(stderr, "Could not calibrate ticks\n"); + exit(1); + } + clock_freq = freq.QuadPart; +} + +#else + +int use_rt_clock; + +static void __attribute__((constructor)) init_get_clock(void) +{ + use_rt_clock = 0; +#ifdef CLOCK_MONOTONIC + { + struct timespec ts; + if (clock_gettime(CLOCK_MONOTONIC, &ts) == 0) { + use_rt_clock = 1; + } + } +#endif +} +#endif From 19b92a4a7eecdadb7d45fe65ac2d182f22e9034b Mon Sep 17 00:00:00 2001 From: Ryan Hileman Date: Wed, 21 Dec 2016 11:50:40 -0500 Subject: [PATCH 2/3] fix possible segfault in hook del (#691) (#697) --- uc.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/uc.c b/uc.c index 10ebdb6a..3738ab6f 100644 --- a/uc.c +++ b/uc.c @@ -1069,19 +1069,19 @@ uc_err uc_hook_add(uc_engine *uc, uc_hook *hh, int type, void *callback, UNICORN_EXPORT uc_err uc_hook_del(uc_engine *uc, uc_hook hh) { - int i = 0; + int i; struct hook *hook = (struct hook *)hh; - int type = hook->type; - - while ((type >> i) > 0 && i < UC_HOOK_MAX) { - if ((type >> i) & 1) { - if (list_remove(&uc->hook[i], (void *)hh)) { - if (--hook->refs == 0) { - free(hook); - } + // we can't dereference hook->type if hook is invalid + // so for now we need to iterate over all possible types to remove the hook + // which is less efficient + // an optimization would be to align the hook pointer + // and store the type mask in the hook pointer. + for (i = 0; i < UC_HOOK_MAX; i++) { + if (list_remove(&uc->hook[i], (void *)hook)) { + if (--hook->refs == 0) { + free(hook); } } - i++; } return UC_ERR_OK; } From 238629302030e307909916695f82241d8b61dbe6 Mon Sep 17 00:00:00 2001 From: Nguyen Anh Quynh Date: Thu, 22 Dec 2016 01:12:10 +0800 Subject: [PATCH 3/3] Makefile: add a note on 'header' target --- Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile b/Makefile index 9ce4ad25..8c9d60e6 100644 --- a/Makefile +++ b/Makefile @@ -282,6 +282,7 @@ dist: git archive --format=zip --prefix=unicorn-$(DIST_VERSION)/ $(TAG) > unicorn-$(DIST_VERSION).zip +# run "make header" whenever qemu/header_gen.py is modified header: $(eval TARGETS := m68k arm aarch64 mips mipsel mips64 mips64el\ powerpc sparc sparc64 x86_64)