qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Sebastian Huber <sebastian.huber@embedded-brains.de>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] SPARC: Add and use CPU_FEATURE_CASA
Date: Thu, 28 Nov 2013 10:55:06 +0100	[thread overview]
Message-ID: <529712FA.2070906@embedded-brains.de> (raw)
In-Reply-To: <1385474661-8717-1-git-send-email-sebastian.huber@embedded-brains.de>

Hello,

On 2013-11-26 15:04, Sebastian Huber wrote:
> The LEON3 processor has support for the CASA instruction which is
> normally only available for SPARC V9 processors.  Binutils 2.24
> and GCC 4.9 will support this instruction for LEON3.  GCC uses it to
> generate C11 atomic operations.
> ---
>   target-sparc/cpu.c         |    3 +-
>   target-sparc/cpu.h         |    4 ++-
>   target-sparc/helper.h      |    4 ++-
>   target-sparc/ldst_helper.c |   26 +++++++++++++-----------
>   target-sparc/translate.c   |   47 ++++++++++++++++++++++++++++---------------
>   5 files changed, 52 insertions(+), 32 deletions(-)
[...]

this patch doesn't work since the ASI 0x80 used for the synthetic CAS 
instruction is not implemented in helper_ld_asi() for !TARGET_SPARC64.

I tried to add a

     case 0x80: /* Primary */
         {
             switch (size) {
             case 1:
                 ret = ldub_raw(addr);
                 break;
             case 2:
                 ret = lduw_raw(addr);
                 break;
             case 4:
                 ret = ldl_raw(addr);
                 break;
             default:
             case 8:
                 ret = ldq_raw(addr);
                 break;
             }
         }
         break;

but this results in a Qemu segmentation fault.

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber@embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.

  reply	other threads:[~2013-11-28  9:55 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-26 14:04 [Qemu-devel] [PATCH] SPARC: Add and use CPU_FEATURE_CASA Sebastian Huber
2013-11-28  9:55 ` Sebastian Huber [this message]
2014-02-05  9:58   ` Fabien Chouteau
2013-11-28 10:27 ` [Qemu-devel] [PATCH v2] " Sebastian Huber
2013-12-10  8:09   ` Sebastian Huber
2013-12-10 17:16     ` Fabien Chouteau
2014-02-05  9:21 ` [Qemu-devel] [PATCH] " Fabien Chouteau
2014-02-05  9:27   ` Fabien Chouteau

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=529712FA.2070906@embedded-brains.de \
    --to=sebastian.huber@embedded-brains.de \
    --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).