From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41241) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1awBYr-0004LF-Ck for qemu-devel@nongnu.org; Fri, 29 Apr 2016 12:46:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1awBYf-0001Jz-Kj for qemu-devel@nongnu.org; Fri, 29 Apr 2016 12:46:31 -0400 Received: from mail-qg0-x244.google.com ([2607:f8b0:400d:c04::244]:36218) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1awBYe-0001Gn-Db for qemu-devel@nongnu.org; Fri, 29 Apr 2016 12:46:25 -0400 Received: by mail-qg0-x244.google.com with SMTP id f74so7749002qge.3 for ; Fri, 29 Apr 2016 09:46:10 -0700 (PDT) Sender: Richard Henderson References: <1461879235-7694-1-git-send-email-sergey.fedorov@linaro.org> From: Richard Henderson Message-ID: <689542a1-6e05-83b3-b9b6-b7c1ade3bbfe@twiddle.net> Date: Fri, 29 Apr 2016 09:46:03 -0700 MIME-Version: 1.0 In-Reply-To: <1461879235-7694-1-git-send-email-sergey.fedorov@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v5 00/10] tcg: Direct block chaining clean-up List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sergey Fedorov , qemu-devel@nongnu.org Cc: =?UTF-8?Q?Alex_Benn=c3=a9e?= , Sergey Fedorov , Paolo Bonzini , Peter Crosthwaite On 04/28/2016 02:33 PM, Sergey Fedorov wrote: > From: Sergey Fedorov > > This series combines a set of patches which is meant to improve overall code > structure and readability of the direct block chaining mechanism. The other > point is to make a step towards thread safety of TB chainig. > > This series is based on commit: 1d02fa9e045b ("translate-all: Adjust 256mb > testing for mips64") from git://github.com/rth7680/qemu.git tcg-next and is > available at git://github.com/sergefdrv/qemu.git tb-chaining-cleanup-v5 > > Summary of changes: > Changes in v5: > * Fixed rebase conflicts > * Don't check for in_superpage() in target-alpha/translate.c for > user-mode [PATCH v5 10/10] > Changes in v4: > * Removed assert from tb_add_jump() [PATCH v4 02/10] > * Added comment on TB stuff synchronization [PATCH v4 04/10] > * Documented tcg_gen_goto_tb() and moved its usage notes there > [PATCH v4 09/10] and [PATCH v4 10/10] > * Cc'ed usermode maintainers in commit message [PATCH v4 10/10] > Changes in v3: > * New patch to clean up safety checks [PATCH v3 09/10] > * New patch to eliminate unneeded checks in user-mode [PATCH v3 10/10] > Changes in v2: > * Eliminated duplicate dereference of 'ptb' in tb_jmp_remove() [PATCH v2 2/8] > * Tweaked a comment [PATCH v2 4/8] > * Complete rewrite [PATCH v2 5/8] > * Tweaked a comment; eliminated duplicate dereference of 'ptb' in > tb_jmp_unlink() [PATCH v2 8/8] > > Sergey Fedorov (10): > tcg: Clean up direct block chaining data fields > tcg: Use uintptr_t type for jmp_list_{next|first} fields of TB > tcg: Rearrange tb_link_page() to avoid forward declaration > tcg: Init TB's direct jumps before making it visible > tcg: Clarify thread safety check in tb_add_jump() > tcg: Rename tb_jmp_remove() to tb_remove_from_jmp_list() > tcg: Extract removing of jumps to TB from tb_phys_invalidate() > tcg: Clean up tb_jmp_unlink() > tcg: Clean up direct block chaining safety checks > tcg: Allow goto_tb to any target PC in user mode Applied to tcg-next. r~