* [PULL v2 00/76] target/microblaze improvements
@ 2020-09-01 15:20 Richard Henderson
2020-09-01 15:20 ` [PULL v2 58/76] tcg: Add tcg_get_insn_start_param Richard Henderson
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Richard Henderson @ 2020-09-01 15:20 UTC (permalink / raw)
To: qemu-devel; +Cc: peter.maydell
Version 2. Serves me right for not testing 32-bit host
when I knew there was a patch that mattered.
r~
The following changes since commit 2f4c51c0f384d7888a04b4815861e6d5fd244d75:
Merge remote-tracking branch 'remotes/kraxel/tags/usb-20200831-pull-request' into staging (2020-08-31 19:39:13 +0100)
are available in the Git repository at:
https://github.com/rth7680/qemu.git tags/pull-mb-20200901
for you to fetch changes up to 19f27b6c2493472fe2790cf08d7b0140d57bdad5:
target/microblaze: Reduce linux-user address space to 32-bit (2020-09-01 07:43:35 -0700)
----------------------------------------------------------------
Convert microblaze to generic translator loop
Convert microblaze to decodetree
Fix mb_cpu_transaction_failed
Other misc cleanups
----------------------------------------------------------------
Richard Henderson (76):
tests/tcg: Add microblaze to arches filter
tests/tcg: Do not require FE_TOWARDZERO
tests/tcg: Do not require FE_* exception bits
target/microblaze: Tidy gdbstub
target/microblaze: Split out PC from env->sregs
target/microblaze: Split out MSR from env->sregs
target/microblaze: Split out EAR from env->sregs
target/microblaze: Split out ESR from env->sregs
target/microblaze: Split out FSR from env->sregs
target/microblaze: Split out BTR from env->sregs
target/microblaze: Split out EDR from env->sregs
target/microblaze: Split the cpu_SR array
target/microblaze: Fix width of PC and BTARGET
target/microblaze: Fix width of MSR
target/microblaze: Fix width of ESR
target/microblaze: Fix width of FSR
target/microblaze: Fix width of BTR
target/microblaze: Fix width of EDR
target/microblaze: Remove cpu_ear
target/microblaze: Tidy raising of exceptions
target/microblaze: Mark raise_exception as noreturn
target/microblaze: Remove helper_debug and env->debug
target/microblaze: Rename env_* tcg variables to cpu_*
target/microblaze: Tidy mb_tcg_init
target/microblaze: Split out MSR[C] to its own variable
target/microblaze: Use DISAS_NORETURN
target/microblaze: Check singlestep_enabled in gen_goto_tb
target/microblaze: Convert to DisasContextBase
target/microblaze: Convert to translator_loop
target/microblaze: Remove SIM_COMPAT
target/microblaze: Remove DISAS_GNU
target/microblaze: Remove empty D macros
target/microblaze: Remove LOG_DIS
target/microblaze: Ensure imm constant is always available
target/microblaze: Add decodetree infrastructure
target/microblaze: Convert dec_add to decodetree
target/microblaze: Convert dec_sub to decodetree
target/microblaze: Implement cmp and cmpu inline
target/microblaze: Convert dec_pattern to decodetree
target/microblaze: Convert dec_and, dec_or, dec_xor to decodetree
target/microblaze: Convert dec_mul to decodetree
target/microblaze: Convert dec_div to decodetree
target/microblaze: Unwind properly when raising divide-by-zero
target/microblaze: Convert dec_bit to decodetree
target/microblaze: Convert dec_barrel to decodetree
target/microblaze: Convert dec_imm to decodetree
target/microblaze: Convert dec_fpu to decodetree
target/microblaze: Fix cpu unwind for fpu exceptions
target/microblaze: Mark fpu helpers TCG_CALL_NO_WG
target/microblaze: Replace MSR_EE_FLAG with MSR_EE
target/microblaze: Cache mem_index in DisasContext
target/microblaze: Fix cpu unwind for stackprot
target/microblaze: Convert dec_load and dec_store to decodetree
target/microblaze: Assert no overlap in flags making up tb_flags
target/microblaze: Move bimm to BIMM_FLAG
target/microblaze: Fix no-op mb_cpu_transaction_failed
target/microblaze: Store "current" iflags in insn_start
tcg: Add tcg_get_insn_start_param
target/microblaze: Use cc->do_unaligned_access
target/microblaze: Replace clear_imm with tb_flags_to_set
target/microblaze: Replace delayed_branch with tb_flags_to_set
target/microblaze: Tidy mb_cpu_dump_state
target/microblaze: Convert brk and brki to decodetree
target/microblaze: Convert mbar to decodetree
target/microblaze: Reorganize branching
target/microblaze: Convert dec_br to decodetree
target/microblaze: Convert dec_bcc to decodetree
target/microblaze: Convert dec_rts to decodetree
target/microblaze: Tidy do_rti, do_rtb, do_rte
target/microblaze: Convert msrclr, msrset to decodetree
target/microblaze: Convert dec_msr to decodetree
target/microblaze: Convert dec_stream to decodetree
target/microblaze: Remove last of old decoder
target/microblaze: Remove cpu_R[0]
target/microblaze: Add flags markup to some helpers
target/microblaze: Reduce linux-user address space to 32-bit
include/tcg/tcg.h | 15 +
target/microblaze/cpu-param.h | 15 +
target/microblaze/cpu.h | 67 +-
target/microblaze/helper.h | 49 +-
target/microblaze/microblaze-decode.h | 59 -
tests/tcg/multiarch/float_helpers.h | 17 +
target/microblaze/insns.decode | 256 +++
linux-user/elfload.c | 9 +-
linux-user/microblaze/cpu_loop.c | 26 +-
linux-user/microblaze/signal.c | 8 +-
target/microblaze/cpu.c | 9 +-
target/microblaze/gdbstub.c | 193 +-
target/microblaze/helper.c | 164 +-
target/microblaze/mmu.c | 4 +-
target/microblaze/op_helper.c | 194 +-
target/microblaze/translate.c | 3223 +++++++++++++++++----------------
tests/tcg/multiarch/float_convs.c | 2 +
tests/tcg/multiarch/float_madds.c | 2 +
target/microblaze/meson.build | 3 +
tests/tcg/configure.sh | 2 +-
20 files changed, 2292 insertions(+), 2025 deletions(-)
delete mode 100644 target/microblaze/microblaze-decode.h
create mode 100644 target/microblaze/insns.decode
^ permalink raw reply [flat|nested] 6+ messages in thread* [PULL v2 58/76] tcg: Add tcg_get_insn_start_param
2020-09-01 15:20 [PULL v2 00/76] target/microblaze improvements Richard Henderson
@ 2020-09-01 15:20 ` Richard Henderson
2020-09-02 14:26 ` [PULL v2 00/76] target/microblaze improvements Peter Maydell
2020-09-02 19:09 ` Thomas Huth
2 siblings, 0 replies; 6+ messages in thread
From: Richard Henderson @ 2020-09-01 15:20 UTC (permalink / raw)
To: qemu-devel; +Cc: Edgar E . Iglesias, peter.maydell
MicroBlaze will shortly need to update a parameter in place.
Add an interface to read to match that for write.
Tested-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
include/tcg/tcg.h | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/include/tcg/tcg.h b/include/tcg/tcg.h
index d40c925d04..53ce94c2c5 100644
--- a/include/tcg/tcg.h
+++ b/include/tcg/tcg.h
@@ -777,11 +777,26 @@ static inline TCGv_i32 TCGV_HIGH(TCGv_i64 t)
}
#endif
+static inline TCGArg tcg_get_insn_param(TCGOp *op, int arg)
+{
+ return op->args[arg];
+}
+
static inline void tcg_set_insn_param(TCGOp *op, int arg, TCGArg v)
{
op->args[arg] = v;
}
+static inline target_ulong tcg_get_insn_start_param(TCGOp *op, int arg)
+{
+#if TARGET_LONG_BITS <= TCG_TARGET_REG_BITS
+ return tcg_get_insn_param(op, arg);
+#else
+ return tcg_get_insn_param(op, arg * 2) |
+ ((uint64_t)tcg_get_insn_param(op, arg * 2 + 1) << 32);
+#endif
+}
+
static inline void tcg_set_insn_start_param(TCGOp *op, int arg, target_ulong v)
{
#if TARGET_LONG_BITS <= TCG_TARGET_REG_BITS
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PULL v2 00/76] target/microblaze improvements
2020-09-01 15:20 [PULL v2 00/76] target/microblaze improvements Richard Henderson
2020-09-01 15:20 ` [PULL v2 58/76] tcg: Add tcg_get_insn_start_param Richard Henderson
@ 2020-09-02 14:26 ` Peter Maydell
2020-09-02 19:09 ` Thomas Huth
2 siblings, 0 replies; 6+ messages in thread
From: Peter Maydell @ 2020-09-02 14:26 UTC (permalink / raw)
To: Richard Henderson; +Cc: QEMU Developers
On Tue, 1 Sep 2020 at 16:20, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> Version 2. Serves me right for not testing 32-bit host
> when I knew there was a patch that mattered.
>
>
> r~
>
> The following changes since commit 2f4c51c0f384d7888a04b4815861e6d5fd244d75:
>
> Merge remote-tracking branch 'remotes/kraxel/tags/usb-20200831-pull-request' into staging (2020-08-31 19:39:13 +0100)
>
> are available in the Git repository at:
>
> https://github.com/rth7680/qemu.git tags/pull-mb-20200901
>
> for you to fetch changes up to 19f27b6c2493472fe2790cf08d7b0140d57bdad5:
>
> target/microblaze: Reduce linux-user address space to 32-bit (2020-09-01 07:43:35 -0700)
>
> ----------------------------------------------------------------
> Convert microblaze to generic translator loop
> Convert microblaze to decodetree
> Fix mb_cpu_transaction_failed
> Other misc cleanups
>
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PULL v2 00/76] target/microblaze improvements
2020-09-01 15:20 [PULL v2 00/76] target/microblaze improvements Richard Henderson
2020-09-01 15:20 ` [PULL v2 58/76] tcg: Add tcg_get_insn_start_param Richard Henderson
2020-09-02 14:26 ` [PULL v2 00/76] target/microblaze improvements Peter Maydell
@ 2020-09-02 19:09 ` Thomas Huth
2020-09-02 23:11 ` Richard Henderson
2 siblings, 1 reply; 6+ messages in thread
From: Thomas Huth @ 2020-09-02 19:09 UTC (permalink / raw)
To: Richard Henderson, qemu-devel
Cc: Alex Bennée, peter.maydell, Philippe Mathieu-Daudé
On 01/09/2020 17.20, Richard Henderson wrote:
> Version 2. Serves me right for not testing 32-bit host
> when I knew there was a patch that mattered.
Hi Richard,
I'm afraid, but I think this PR broke the
tests/acceptance/replay_kernel.py:ReplayKernel.test_microblaze_s3adsp1800 acceptance
test:
https://gitlab.com/qemu-project/qemu/-/jobs/716158589#L176
Could you please have a look?
Thanks,
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PULL v2 00/76] target/microblaze improvements
2020-09-02 19:09 ` Thomas Huth
@ 2020-09-02 23:11 ` Richard Henderson
2020-09-03 2:12 ` Richard Henderson
0 siblings, 1 reply; 6+ messages in thread
From: Richard Henderson @ 2020-09-02 23:11 UTC (permalink / raw)
To: Thomas Huth, qemu-devel
Cc: Alex Bennée, peter.maydell, Philippe Mathieu-Daudé
On 9/2/20 12:09 PM, Thomas Huth wrote:
> On 01/09/2020 17.20, Richard Henderson wrote:
>> Version 2. Serves me right for not testing 32-bit host
>> when I knew there was a patch that mattered.
>
> Hi Richard,
>
> I'm afraid, but I think this PR broke the
> tests/acceptance/replay_kernel.py:ReplayKernel.test_microblaze_s3adsp1800 acceptance
> test:
>
> https://gitlab.com/qemu-project/qemu/-/jobs/716158589#L176
>
> Could you please have a look?
That's odd.
Doesn't replay rely on migration (or at least VMStateDescription) to do its
job? Microblaze doesn't implement that, so how could this have worked before?
I admit that I don't often run check-acceptance, because I rarely see a clean
bill of health, even for stuff I'm not working on.
Looking into it...
r~
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PULL v2 00/76] target/microblaze improvements
2020-09-02 23:11 ` Richard Henderson
@ 2020-09-03 2:12 ` Richard Henderson
0 siblings, 0 replies; 6+ messages in thread
From: Richard Henderson @ 2020-09-03 2:12 UTC (permalink / raw)
To: Thomas Huth, qemu-devel
Cc: Alex Bennée, peter.maydell, Philippe Mathieu-Daudé
On 9/2/20 4:11 PM, Richard Henderson wrote:
> On 9/2/20 12:09 PM, Thomas Huth wrote:
>> On 01/09/2020 17.20, Richard Henderson wrote:
>>> Version 2. Serves me right for not testing 32-bit host
>>> when I knew there was a patch that mattered.
>>
>> Hi Richard,
>>
>> I'm afraid, but I think this PR broke the
>> tests/acceptance/replay_kernel.py:ReplayKernel.test_microblaze_s3adsp1800 acceptance
>> test:
>>
>> https://gitlab.com/qemu-project/qemu/-/jobs/716158589#L176
>>
>> Could you please have a look?
>
> That's odd.
>
> Doesn't replay rely on migration (or at least VMStateDescription) to do its
> job? Microblaze doesn't implement that, so how could this have worked before?
>
> I admit that I don't often run check-acceptance, because I rarely see a clean
> bill of health, even for stuff I'm not working on.
>
> Looking into it...
Ah, in this case it's all about the icount setting.
So the log recorded by replay=record is indeed useless for input to
replay=replay, but not relevant.
r~
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-09-03 2:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-01 15:20 [PULL v2 00/76] target/microblaze improvements Richard Henderson
2020-09-01 15:20 ` [PULL v2 58/76] tcg: Add tcg_get_insn_start_param Richard Henderson
2020-09-02 14:26 ` [PULL v2 00/76] target/microblaze improvements Peter Maydell
2020-09-02 19:09 ` Thomas Huth
2020-09-02 23:11 ` Richard Henderson
2020-09-03 2:12 ` Richard Henderson
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).