From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47954) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asekP-00025M-R6 for qemu-devel@nongnu.org; Tue, 19 Apr 2016 19:07:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1asekN-0000hE-4v for qemu-devel@nongnu.org; Tue, 19 Apr 2016 19:07:57 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:60605) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asekN-0000go-1a for qemu-devel@nongnu.org; Tue, 19 Apr 2016 19:07:55 -0400 Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.nyi.internal (Postfix) with ESMTP id 3D78A2066B for ; Tue, 19 Apr 2016 19:07:54 -0400 (EDT) From: "Emilio G. Cota" Date: Tue, 19 Apr 2016 19:07:39 -0400 Message-Id: <1461107270-19234-1-git-send-email-cota@braap.org> Subject: [Qemu-devel] [PATCH v3 00/11] tb hash improvements List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: QEMU Developers , MTTCG Devel Cc: =?UTF-8?q?Alex=20Benn=C3=A9e?= , Paolo Bonzini , Peter Crosthwaite , Richard Henderson , Peter Maydell , Sergey Fedorov 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