From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Edgar Iglesias" <edgar.iglesias@xilinx.com>,
"QEMU Developers" <qemu-devel@nongnu.org>,
qemu-arm <qemu-arm@nongnu.org>,
"Sergey Fedorov" <serge.fdrv@gmail.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH v2 8/8] target-arm: Use isyn.swstep.ex to hold the is_ldex state
Date: Fri, 26 Feb 2016 09:22:12 +0100 [thread overview]
Message-ID: <20160226082212.GB16305@toto> (raw)
In-Reply-To: <CAFEAcA94S0eWzdUHBQ69PbRXgEJcvkSczU6qBE5FMboSfmxWZg@mail.gmail.com>
On Thu, Feb 25, 2016 at 06:28:54PM +0000, Peter Maydell wrote:
> On 19 February 2016 at 20:04, Edgar E. Iglesias
> <edgar.iglesias@gmail.com> wrote:
> > From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
> >
> > Switch to using isyn.swstep.ex to hold the is_ldex state for
> > SWStep syndrome generation.
> >
> > No functional change.
> >
> > Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
>
> What's the rationale for renaming the field? If this is groundwork
> for something else it might make more sense in a patchset with
> whatever is going to use this. Or is it just trying to make the
> swstep handling parallel the dabt syndrome info approach a bit more?
Yes, the latter. I can drop this one...
Cheers,
Edgar
prev parent reply other threads:[~2016-02-26 8:22 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-19 20:04 [Qemu-devel] [PATCH v2 0/8] arm: Steps towards EL2 support round 6 Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 1/8] tcg: Add tcg_set_insn_param Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 2/8] gen-icount: Use tcg_set_insn_param Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 3/8] target-arm: Add the IL flag to syn_data_abort Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 4/8] target-arm: Add more fields to the data abort syndrome generator Edgar E. Iglesias
2016-02-25 17:41 ` Peter Maydell
2016-02-26 20:08 ` Sergey Fedorov
2016-04-29 11:54 ` Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 5/8] target-arm/translate-a64.c: Use extract32 in disas_ldst_reg_imm9 Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 6/8] target-arm/translate-a64.c: Unify some of the ldst_reg decoding Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 7/8] target-arm: A64: Create Instruction Syndromes for Data Aborts Edgar E. Iglesias
2016-02-25 18:26 ` Peter Maydell
2016-04-29 11:58 ` Edgar E. Iglesias
2016-02-19 20:04 ` [Qemu-devel] [PATCH v2 8/8] target-arm: Use isyn.swstep.ex to hold the is_ldex state Edgar E. Iglesias
2016-02-25 18:28 ` Peter Maydell
2016-02-26 8:22 ` Edgar E. Iglesias [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=20160226082212.GB16305@toto \
--to=edgar.iglesias@gmail.com \
--cc=alex.bennee@linaro.org \
--cc=edgar.iglesias@xilinx.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=serge.fdrv@gmail.com \
/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).