From: "Emilio G. Cota" <cota@braap.org>
To: QEMU Developers <qemu-devel@nongnu.org>,
MTTCG Devel <mttcg@listserver.greensocs.com>
Cc: "Alex Bennée" <alex.bennee@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Peter Crosthwaite" <crosthwaite.peter@gmail.com>,
"Richard Henderson" <rth@twiddle.net>,
"Peter Maydell" <peter.maydell@linaro.org>,
"Sergey Fedorov" <serge.fdrv@gmail.com>
Subject: [Qemu-devel] [PATCH v3 00/11] tb hash improvements
Date: Tue, 19 Apr 2016 19:07:39 -0400 [thread overview]
Message-ID: <1461107270-19234-1-git-send-email-cota@braap.org> (raw)
See v2 here:
https://lists.gnu.org/archive/html/qemu-devel/2016-04/msg01307.html
Changes from v2:
- Dropped "add missing fold of tb_ctx into tcg_ctx", already merged
upstream as commit 7e6bd36d611.
- Added reviewed-by tags from Alex and Richard
- xxhash:
+ use rol32 from qemu/bitops.h
+ remove seed parameter from tb_hash_func5
- qht:
+ add comments suggested by Alex, almost all of them about MRU
+ add BUILD_BUG_ON check for the size of qht_bucket
+ add assert(orig != head) in MRU promotion function, and delete
code path that dealt with that case (it was dead code)
+ fold qht_bucket_reset__locked into qht_bucket_reset
+ do not inline qht_lookup
+ move definitions of qht_bucket and qht_map to qht.c
+ remove 'count' variable for knowing whether lookups/insertions
were on non-head buckets; just check 'b != head' instead.
+ add avg bucket chain length to 'info jit'.
- qht-test:
+ drive tests with g_test
+ add avg_bucket_chain_length checks
Thanks,
Emilio
next reply other threads:[~2016-04-19 23:07 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-19 23:07 Emilio G. Cota [this message]
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 01/11] compiler.h: add QEMU_ALIGNED() to enforce struct alignment Emilio G. Cota
2016-04-22 9:32 ` Alex Bennée
2016-04-22 9:35 ` Peter Maydell
2016-04-22 15:50 ` Emilio G. Cota
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 02/11] seqlock: remove optional mutex Emilio G. Cota
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 03/11] seqlock: rename write_lock/unlock to write_begin/end Emilio G. Cota
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 04/11] include/processor.h: define cpu_relax() Emilio G. Cota
2016-04-20 12:15 ` KONRAD Frederic
2016-04-20 17:16 ` Emilio G. Cota
2016-04-20 17:18 ` Peter Maydell
2016-04-20 17:32 ` [Qemu-devel] [UPDATED " Emilio G. Cota
2016-04-22 9:35 ` [Qemu-devel] [PATCH " Alex Bennée
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock Emilio G. Cota
2016-04-20 15:18 ` Richard Henderson
2016-04-20 17:17 ` Emilio G. Cota
2016-04-20 17:55 ` Richard Henderson
2016-04-20 18:11 ` Emilio G. Cota
2016-04-20 19:39 ` Richard Henderson
2016-04-21 16:24 ` Emilio G. Cota
2016-04-20 17:35 ` [Qemu-devel] [UPDATED " Emilio G. Cota
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 06/11] exec: add tb_hash_func5, derived from xxhash Emilio G. Cota
2016-04-20 15:19 ` Richard Henderson
2016-04-22 12:58 ` Alex Bennée
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 07/11] tb hash: hash phys_pc, pc, and flags with xxhash Emilio G. Cota
2016-04-22 12:58 ` Alex Bennée
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 08/11] qht: QEMU's fast, resizable and scalable Hash Table Emilio G. Cota
2016-04-22 14:04 ` Alex Bennée
2016-04-24 20:01 ` Richard Henderson
2016-04-24 21:58 ` Emilio G. Cota
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 09/11] qht: add test program Emilio G. Cota
2016-04-22 14:35 ` Alex Bennée
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 10/11] tb hash: track translated blocks with qht Emilio G. Cota
2016-04-28 13:27 ` Alex Bennée
2016-04-19 23:07 ` [Qemu-devel] [PATCH v3 11/11] translate-all: add tb hash bucket info to 'info jit' dump Emilio G. Cota
2016-04-20 15:21 ` Richard Henderson
2016-04-22 14:38 ` Alex Bennée
2016-04-22 17:41 ` Richard Henderson
2016-04-22 19:23 ` Emilio G. Cota
2016-04-22 19:59 ` Richard Henderson
2016-04-22 23:57 ` Emilio G. Cota
2016-04-24 19:46 ` Richard Henderson
2016-04-24 22:06 ` Emilio G. Cota
2016-04-27 2:43 ` Emilio G. Cota
2016-04-28 16:37 ` Richard Henderson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1461107270-19234-1-git-send-email-cota@braap.org \
--to=cota@braap.org \
--cc=alex.bennee@linaro.org \
--cc=crosthwaite.peter@gmail.com \
--cc=mttcg@listserver.greensocs.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=serge.fdrv@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.