qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Max Filippov <jcmvbkbc@gmail.com>, qemu-devel@nongnu.org
Cc: Peter Maydell <peter.maydell@linaro.org>
Subject: Re: [Qemu-devel] [PULL v3] target/xtensa updates
Date: Fri, 12 Jan 2018 10:20:31 +0100	[thread overview]
Message-ID: <27e44c5d-b04c-df08-bd7f-fc4089d91dcd@redhat.com> (raw)
In-Reply-To: <1515521823-4292-1-git-send-email-jcmvbkbc@gmail.com>

On 09/01/2018 19:17, Max Filippov wrote:
> Hi Peter,
> 
> please pull the following batch of updates for the target/xtensa.
> Changes v2->v3:
> - Don't use g_malloc_n, use g_new instead.
> Changes v1->v2:
> - Drop no longer used function option_bits_enabled.
> 
> The following changes since commit 0a0dc59d27527b78a195c2d838d28b7b49e5a639:
> 
>   Update version for v2.11.0 release (2017-12-13 14:31:09 +0000)
> 
> are available in the git repository at:
> 
>   git://github.com/OSLL/qemu-xtensa.git tags/20180109-xtensa
> 
> for you to fetch changes up to 5a6539e627faf9251e1db78238b9f9b870610518:
> 
>   target/xtensa: implement disassembler (2018-01-09 09:55:39 -0800)

This fails compilation on CentOS 6:

/tmp/qemu-test/src/target/xtensa/translate.c:81: error: redefinition of typedef 'DisasContext'
/tmp/qemu-test/src/target/xtensa/cpu.h:339: note: previous declaration of 'DisasContext' was here
make[1]: *** [target/xtensa/translate.o] Error 1
make[1]: *** Waiting for unfinished jobs....

Paolo

> ----------------------------------------------------------------
> target/xtensa updates:
> 
> - add libisa to the xtensa target;
> - change xtensa instruction translator to use it;
> - switch existing xtensa cores to use it;
> - add support for a number of instructions: salt/saltu, const16,
>   GPIO32 group, debug mode and MMU-related;
> - add disassembler for Xtensa.
> 
> ----------------------------------------------------------------
> Max Filippov (16):
>       target/xtensa: pass actual frame size to the entry helper
>       target/xtensa: import libisa source
>       target/xtensa: extract core opcode translators
>       target/xtensa: extract FPU2000 opcode translators
>       target/xtensa: update import_core.sh script for libisa
>       target/xtensa: switch dc232b to libisa
>       target/xtensa: switch dc233c to libisa
>       target/xtensa: switch fsf to libisa
>       target/xtensa: use libisa for instruction decoding
>       target/xtensa: tests: fix memctl SR test
>       target/xtensa: drop DisasContext::litbase
>       target/xtensa: add internal/noop SRs and opcodes
>       target/xtensa: implement salt/saltu
>       target/xtensa: implement GPIO32
>       target/xtensa: implement const16
>       target/xtensa: implement disassembler
> 
>  MAINTAINERS                                |     1 +
>  disas/Makefile.objs                        |     1 +
>  disas/xtensa.c                             |   133 +
>  include/disas/bfd.h                        |     1 +
>  include/hw/xtensa/xtensa-isa.h             |   838 ++
>  target/xtensa/Makefile.objs                |     1 +
>  target/xtensa/core-dc232b.c                |     4 +
>  target/xtensa/core-dc232b/xtensa-modules.c | 14105 +++++++++++++++++++++++++
>  target/xtensa/core-dc233c.c                |     4 +
>  target/xtensa/core-dc233c/xtensa-modules.c | 15232 +++++++++++++++++++++++++++
>  target/xtensa/core-fsf.c                   |     5 +
>  target/xtensa/core-fsf/xtensa-modules.c    |  9841 +++++++++++++++++
>  target/xtensa/cpu.c                        |     9 +
>  target/xtensa/cpu.h                        |    31 +
>  target/xtensa/helper.c                     |    37 +
>  target/xtensa/import_core.sh               |    15 +
>  target/xtensa/op_helper.c                  |     2 +-
>  target/xtensa/translate.c                  |  5900 +++++++----
>  target/xtensa/xtensa-isa-internal.h        |   231 +
>  target/xtensa/xtensa-isa.c                 |  1745 +++
>  target/xtensa/xtensa-isa.h                 |     1 +
>  tests/tcg/xtensa/test_sr.S                 |     2 +-
>  22 files changed, 45955 insertions(+), 2184 deletions(-)
>  create mode 100644 disas/xtensa.c
>  create mode 100644 include/hw/xtensa/xtensa-isa.h
>  create mode 100644 target/xtensa/core-dc232b/xtensa-modules.c
>  create mode 100644 target/xtensa/core-dc233c/xtensa-modules.c
>  create mode 100644 target/xtensa/core-fsf/xtensa-modules.c
>  create mode 100644 target/xtensa/xtensa-isa-internal.h
>  create mode 100644 target/xtensa/xtensa-isa.c
>  create mode 100644 target/xtensa/xtensa-isa.h
> 

  parent reply	other threads:[~2018-01-12  9:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-09 18:17 [Qemu-devel] [PULL v3] target/xtensa updates Max Filippov
2018-01-11  9:53 ` Peter Maydell
2018-01-12  9:20 ` Paolo Bonzini [this message]
2018-01-12 17:36   ` [Qemu-devel] [PATCH] target/xtensa: don't typedef DisasContext twice Max Filippov
2018-01-12 17:43     ` Max Filippov

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=27e44c5d-b04c-df08-bd7f-fc4089d91dcd@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=jcmvbkbc@gmail.com \
    --cc=peter.maydell@linaro.org \
    --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).