diff options
| author | Emilio G. Cota | 2018-09-10 19:06:12 +0200 |
|---|---|---|
| committer | Richard Henderson | 2018-09-26 17:55:54 +0200 |
| commit | 78255ba2cc00f41d96aa2911933764f59f24c958 (patch) | |
| tree | 06b33056d900b7cdc79ac7d8540461c6768a4c15 /accel | |
| parent | test-qht: speed up + test qht_resize (diff) | |
| download | qemu-78255ba2cc00f41d96aa2911933764f59f24c958.tar.gz qemu-78255ba2cc00f41d96aa2911933764f59f24c958.tar.xz qemu-78255ba2cc00f41d96aa2911933764f59f24c958.zip | |
qht: drop ht argument from qht iterators
Accessing the HT from an iterator results almost always
in a deadlock. Given that only one qht-internal function
uses this argument, drop it from the interface.
Suggested-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Emilio G. Cota <cota@braap.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'accel')
| -rw-r--r-- | accel/tcg/translate-all.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c index 898c3bb3d1..9ffbbc2fbd 100644 --- a/accel/tcg/translate-all.c +++ b/accel/tcg/translate-all.c @@ -1282,8 +1282,7 @@ void tb_flush(CPUState *cpu) */ #ifdef CONFIG_USER_ONLY -static void -do_tb_invalidate_check(struct qht *ht, void *p, uint32_t hash, void *userp) +static void do_tb_invalidate_check(void *p, uint32_t hash, void *userp) { TranslationBlock *tb = p; target_ulong addr = *(target_ulong *)userp; @@ -1304,8 +1303,7 @@ static void tb_invalidate_check(target_ulong address) qht_iter(&tb_ctx.htable, do_tb_invalidate_check, &address); } -static void -do_tb_page_check(struct qht *ht, void *p, uint32_t hash, void *userp) +static void do_tb_page_check(void *p, uint32_t hash, void *userp) { TranslationBlock *tb = p; int flags1, flags2; |