qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: malc <av1474@comtv.ru>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/10] tcg: improve logical op support for ppc hosts
Date: Sun, 29 Mar 2009 01:43:24 +0300 (MSK)	[thread overview]
Message-ID: <Pine.LNX.4.64.0903290136390.2728@linmac.oyster.ru> (raw)
In-Reply-To: <1238274167-7890-1-git-send-email-froydnj@codesourcery.com>

On Sat, 28 Mar 2009, Nathan Froyd wrote:

> This patch series implements the necessary support for native TCG
> implementations the logical operations andc, orc, eqv, nor, and nand (as
> opposed to the synthesized implementations currently used by TCG).  It
> also implements backend support for said operations in the ppc and ppc64
> backends.
> 
> I have tested it sparsely on a ppc64 machine I have access to; Linux
> user-mode emulation seems to be broken on ppc hosts.  I could not run a
> given binary (something as simple as ls or complicated as gcc) with
> ppc{,64}-linux-user targeted QEMU.  Prior to the guest application
> crashing, however, I could see in qemu.log that several
> andc/orc/eqv/nand/nor instructions were being used and being translated
> correctly.

Hmm.. ls works, gcc dies with:

gcc: /usr/lib/gcc/powerpc-slackware-linux/3.4.6/specs: Value too large for 
defined data type

I know that some frame setup logic is missing but the core (translatio/tcg)
should be fine.

> The patch series also includes one bugfix at the very end, which removes
> r0 from the allocation pool.  The logic of the patch seems
> straightforward enough to me, but I confess to not being a TCG expert,
> so if TCG somehow avoids generating instructions with r0 in the "wrong"
> places, I'd be happy to hear of an explanation why.
> 
> -Nathan
> 
> 

-- 
mailto:av1474@comtv.ru

      parent reply	other threads:[~2009-03-28 22:43 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-28 21:02 [Qemu-devel] [PATCH 0/10] tcg: improve logical op support for ppc hosts Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 01/10] Add TCG ops for various logical operations Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 02/10] Specialize tcg_gen_not_i64 for 32-bit targets Nathan Froyd
2009-03-31 22:45   ` Paul Brook
2009-03-28 21:02 ` [Qemu-devel] [PATCH 03/10] Implement specialized andc_i{32,64} Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 04/10] Implement specialized eqv_{i32,i64} Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 05/10] Implement specialized nand_i{32,64} Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 06/10] Implement specialized nor_i{32,64} Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 07/10] Implement specialized orc_i{32,64} Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 08/10] Implement new logical instructions for ppc Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 09/10] Implement new logical instructions for ppc64 Nathan Froyd
2009-03-28 21:02 ` [Qemu-devel] [PATCH 10/10] Remove r0 from the allocation pool on ppc/ppc64 Nathan Froyd
2009-03-28 22:26   ` malc
2009-03-28 22:43 ` malc [this message]

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=Pine.LNX.4.64.0903290136390.2728@linmac.oyster.ru \
    --to=av1474@comtv.ru \
    --cc=qemu-devel@nongnu.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).