From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:56340) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQynL-0003ah-Eu for qemu-devel@nongnu.org; Sat, 13 Apr 2013 07:39:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UQynK-0005nc-6K for qemu-devel@nongnu.org; Sat, 13 Apr 2013 07:38:59 -0400 Received: from hall.aurel32.net ([2001:470:1f15:c4f::1]:40154) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UQynJ-0005lg-Vx for qemu-devel@nongnu.org; Sat, 13 Apr 2013 07:38:58 -0400 Date: Sat, 13 Apr 2013 13:38:45 +0200 From: Aurelien Jarno Message-ID: <20130413113845.GA13234@ohm.aurel32.net> References: <1364876610-3933-1-git-send-email-rth@twiddle.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline In-Reply-To: <1364876610-3933-1-git-send-email-rth@twiddle.net> Subject: Re: [Qemu-devel] [PATCH v3 00/27] Modernize tcg/ppc64 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson Cc: av1474@comtv.ru, qemu-devel@nongnu.org, agraf@suse.de On Mon, Apr 01, 2013 at 09:23:03PM -0700, Richard Henderson wrote: > Changes v2-v3: > * Incorporated feedback from Aurelien's review. > > Tree updated at > > git://github.com/rth7680/qemu.git tcg-ppc64 > > r~ > > > > Richard Henderson (27): > disas: Disassemble all ppc insns for the host > tcg-ppc64: Use TCGReg everywhere > tcg-ppc64: Introduce and use tcg_out_rlw > tcg-ppc64: Introduce and use tcg_out_ext32u > tcg-ppc64: Introduce and use tcg_out_shli64 > tcg-ppc64: Introduce and use tcg_out_shri64 > tcg-ppc64: Cleanup tcg_out_movi > tcg-ppc64: Introduce and use TAI and SAI > tcg-ppc64: Rearrange integer constant constraints > tcg-ppc64: Improve constant add and sub ops. > tcg-ppc64: Tidy or and xor patterns. > tcg-ppc64: Improve and_i32 with constant > tcg-ppc64: Improve and_i64 with constant > tcg-ppc64: Streamline qemu_ld/st insn selection > tcg-ppc64: Implement rotates > tcg-ppc64: Implement bswap16 and bswap32 > tcg-ppc64: Implement bswap64 > tcg-ppc64: Implement compound logicals > tcg-ppc64: Handle constant inputs for some compound logicals > tcg-ppc64: Implement deposit > tcg-ppc64: Use I constraint for mul > tcg-ppc64: Use TCGType throughout compares > tcg-ppc64: Rewrite setcond > tcg-ppc64: Implement movcond > tcg-ppc64: Use getauxval for ISA detection > tcg-ppc64: Implement add2/sub2_i64 > tcg-ppc64: Implement mulu2/muls2_i64 > > configure | 18 + > disas.c | 1 + > tcg/ppc64/tcg-target.c | 1395 +++++++++++++++++++++++++++++++++--------------- > tcg/ppc64/tcg-target.h | 60 ++- > 4 files changed, 1000 insertions(+), 474 deletions(-) > Please find the review in the individual patches. Except minor nitpicks in patch 16, the series looks fine for me. I have tested it with arm, i386, mips(64), mips(64)el, powerpc, sh4 and sparc guests without problem. OTOH it didn't work for me for x86_64 guests, which hangs in the boot process (both Linux and FreeBSD). i386 guests using qemu-system-x86_64 works fine though. So it's a huge improvement considering it was not working for half of these guests before. Thanks. Tested-by: Aurelien Jarno -- Aurelien Jarno GPG: 1024D/F1BCDB73 aurelien@aurel32.net http://www.aurel32.net