[Qemu-devel] [PULL 0/1] target-arm queue

[Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

Last target-arm pullreq before the hardfreeze; just a
single patch. (Actually it's a bugfix patch so strictly
it could go in post-freeze, but anyway.)

Please pull.

thanks
-- PMM


The following changes since commit 743bddb4b35ceaaf6f95aea581a4130dcae6205a:

  Merge remote-tracking branch 'stefanha/tracing' into staging (2013-05-03 11:20:16 -0500)

are available in the git repository at:


  git://git.linaro.org/people/pmaydell/qemu-arm.git target-arm.next

for you to fetch changes up to 536f25e4c77592b936e50728c83894c23f4f61c8:

  target-arm: Fix incorrect check of kvm_vcpu_ioctl return value (2013-05-03 18:47:22 +0100)

----------------------------------------------------------------
Peter Maydell (1):
      target-arm: Fix incorrect check of kvm_vcpu_ioctl return value

 target-arm/kvm.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Re: [Qemu-devel] [PULL 0/1] target-arm queue

[Blue Swirl]

Thanks, pulled.

On Fri, May 3, 2013 at 5:58 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> Last target-arm pullreq before the hardfreeze; just a
> single patch. (Actually it's a bugfix patch so strictly
> it could go in post-freeze, but anyway.)
>
> Please pull.
>
> thanks
> -- PMM
>
>
> The following changes since commit 743bddb4b35ceaaf6f95aea581a4130dcae6205a:
>
>   Merge remote-tracking branch 'stefanha/tracing' into staging (2013-05-03 11:20:16 -0500)
>
> are available in the git repository at:
>
>
>   git://git.linaro.org/people/pmaydell/qemu-arm.git target-arm.next
>
> for you to fetch changes up to 536f25e4c77592b936e50728c83894c23f4f61c8:
>
>   target-arm: Fix incorrect check of kvm_vcpu_ioctl return value (2013-05-03 18:47:22 +0100)
>
> ----------------------------------------------------------------
> Peter Maydell (1):
>       target-arm: Fix incorrect check of kvm_vcpu_ioctl return value
>
>  target-arm/kvm.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

[Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

Only one change in this pullreq, but I'm going away for a week
and I think the bugfix is worth getting into master before then.
Please pull.

thanks
-- PMM

The following changes since commit f10acc8b38d65a66ffa0588a036489d7fa6a593e:

  tcx: Fix 24-bit display mode (2013-06-02 16:45:40 +0000)

are available in the git repository at:

  git://git.linaro.org/people/pmaydell/qemu-arm.git target-arm.next

for you to fetch changes up to 5866e078770cba631a33e6d3062afa7a4068adf8:

  Fix rfe instruction (2013-06-03 14:59:29 +0100)

----------------------------------------------------------------
Peter Chubb (1):
      Fix rfe instruction

 target-arm/translate.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

Ping!

thanks
-- PMM

On 3 June 2013 15:14, Peter Maydell <peter.maydell@linaro.org> wrote:
> Only one change in this pullreq, but I'm going away for a week
> and I think the bugfix is worth getting into master before then.
> Please pull.
>
> thanks
> -- PMM
>
> The following changes since commit f10acc8b38d65a66ffa0588a036489d7fa6a593e:
>
>   tcx: Fix 24-bit display mode (2013-06-02 16:45:40 +0000)
>
> are available in the git repository at:
>
>   git://git.linaro.org/people/pmaydell/qemu-arm.git target-arm.next
>
> for you to fetch changes up to 5866e078770cba631a33e6d3062afa7a4068adf8:
>
>   Fix rfe instruction (2013-06-03 14:59:29 +0100)
>
> ----------------------------------------------------------------
> Peter Chubb (1):
>       Fix rfe instruction
>
>  target-arm/translate.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Re: [Qemu-devel] [PULL 0/1] target-arm queue

[Anthony Liguori]

Pulled.  Thanks.

Regards,

Anthony Liguori

[Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

Only one patch for target-arm for rc2...

thanks
-- PMM

The following changes since commit 5144fe36056d3a94b0d8da9e10adc555c057baa8:

  Merge remote-tracking branch 'remotes/kraxel/tags/pull-vga-20160411-1' into staging (2016-04-11 13:32:50 +0100)

are available in the git repository at:


  git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20160411

for you to fetch changes up to 3a15cc0e1ee7168db0782133d2607a6bfa422d66:

  net: stellaris_enet: check packet length against receive buffer (2016-04-11 14:22:33 +0100)

----------------------------------------------------------------
target-arm queue:
 * stellaris_enet: don't overrun buffer if fed oversize packet

----------------------------------------------------------------
Prasad J Pandit (1):
      net: stellaris_enet: check packet length against receive buffer

 hw/net/stellaris_enet.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

Re: [Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

On 11 April 2016 at 14:37, Peter Maydell <peter.maydell@linaro.org> wrote:
> Only one patch for target-arm for rc2...
>
> thanks
> -- PMM
>
> The following changes since commit 5144fe36056d3a94b0d8da9e10adc555c057baa8:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-vga-20160411-1' into staging (2016-04-11 13:32:50 +0100)
>
> are available in the git repository at:
>
>
>   git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20160411
>
> for you to fetch changes up to 3a15cc0e1ee7168db0782133d2607a6bfa422d66:
>
>   net: stellaris_enet: check packet length against receive buffer (2016-04-11 14:22:33 +0100)
>
> ----------------------------------------------------------------
> target-arm queue:
>  * stellaris_enet: don't overrun buffer if fed oversize packet

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/1] target-arm queue

[Peter Maydell]

One patch for 2.8-rc3, which is Alex's partial revert
of 1dd089d0 to fix A64 ldaxp.

thanks
-- PMM

The following changes since commit bc66cedb4141fb7588f2462c74310d8fb5dd4cf1:

  Merge remote-tracking branch 'yongbok/tags/mips-20161204' into staging (2016-12-05 10:56:45 +0000)

are available in the git repository at:

  git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20161205

for you to fetch changes up to 5460da501a57cd72eda6fec736d76539122e2f99:

  target-arm/translate-a64: fix gen_load_exclusive (2016-12-05 17:52:01 +0000)

----------------------------------------------------------------
target-arm queue:
 * fix gen_load_exclusive handling of ldaxp

----------------------------------------------------------------
Alex Bennée (1):
      target-arm/translate-a64: fix gen_load_exclusive

 target-arm/translate-a64.c | 42 +++++++++++++++++++-----------------------
 1 file changed, 19 insertions(+), 23 deletions(-)

[Qemu-devel] [PULL 1/1] target-arm/translate-a64: fix gen_load_exclusive

[Peter Maydell]

From: Alex Bennée <alex.bennee@linaro.org>

While testing rth's latest TCG patches with risu I found ldaxp was
broken. Investigating further I found it was broken by 1dd089d0 when
the cmpxchg atomic work was merged. As part of that change the code
attempted to be clever by doing a single 64 bit load and then shuffle
the data around to set the two 32 bit registers.

As I couldn't quite follow the endian magic I've simply partially
reverted the change to the original code gen_load_exclusive code. This
doesn't affect the cmpxchg functionality as that is all done on in
gen_store_exclusive part which is untouched.

I've also restored the comment that was removed (with a slight tweak
to mention cmpxchg).

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Richard Henderson <rth@twiddle.net>
Message-id: 20161202173454.19179-1-alex.bennee@linaro.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
 target-arm/translate-a64.c | 42 +++++++++++++++++++-----------------------
 1 file changed, 19 insertions(+), 23 deletions(-)

diff --git a/target-arm/translate-a64.c b/target-arm/translate-a64.c
index de48747..6dc27a6 100644
--- a/target-arm/translate-a64.c
+++ b/target-arm/translate-a64.c
@@ -1839,41 +1839,37 @@ static void disas_b_exc_sys(DisasContext *s, uint32_t insn)
     }
 }
 
+/*
+ * Load/Store exclusive instructions are implemented by remembering
+ * the value/address loaded, and seeing if these are the same
+ * when the store is performed. This is not actually the architecturally
+ * mandated semantics, but it works for typical guest code sequences
+ * and avoids having to monitor regular stores.
+ *
+ * The store exclusive uses the atomic cmpxchg primitives to avoid
+ * races in multi-threaded linux-user and when MTTCG softmmu is
+ * enabled.
+ */
 static void gen_load_exclusive(DisasContext *s, int rt, int rt2,
                                TCGv_i64 addr, int size, bool is_pair)
 {
     TCGv_i64 tmp = tcg_temp_new_i64();
-    TCGMemOp be = s->be_data;
+    TCGMemOp memop = s->be_data + size;
 
     g_assert(size <= 3);
+    tcg_gen_qemu_ld_i64(tmp, addr, get_mem_index(s), memop);
+
     if (is_pair) {
+        TCGv_i64 addr2 = tcg_temp_new_i64();
         TCGv_i64 hitmp = tcg_temp_new_i64();
 
-        if (size == 3) {
-            TCGv_i64 addr2 = tcg_temp_new_i64();
-
-            tcg_gen_qemu_ld_i64(tmp, addr, get_mem_index(s),
-                                MO_64 | MO_ALIGN_16 | be);
-            tcg_gen_addi_i64(addr2, addr, 8);
-            tcg_gen_qemu_ld_i64(hitmp, addr2, get_mem_index(s),
-                                MO_64 | MO_ALIGN | be);
-            tcg_temp_free_i64(addr2);
-        } else {
-            g_assert(size == 2);
-            tcg_gen_qemu_ld_i64(tmp, addr, get_mem_index(s),
-                                MO_64 | MO_ALIGN | be);
-            if (be == MO_LE) {
-                tcg_gen_extr32_i64(tmp, hitmp, tmp);
-            } else {
-                tcg_gen_extr32_i64(hitmp, tmp, tmp);
-            }
-        }
-
+        g_assert(size >= 2);
+        tcg_gen_addi_i64(addr2, addr, 1 << size);
+        tcg_gen_qemu_ld_i64(hitmp, addr2, get_mem_index(s), memop);
+        tcg_temp_free_i64(addr2);
         tcg_gen_mov_i64(cpu_exclusive_high, hitmp);
         tcg_gen_mov_i64(cpu_reg(s, rt2), hitmp);
         tcg_temp_free_i64(hitmp);
-    } else {
-        tcg_gen_qemu_ld_i64(tmp, addr, get_mem_index(s), size | MO_ALIGN | be);
     }
 
     tcg_gen_mov_i64(cpu_exclusive_val, tmp);
-- 
2.7.4

Re: [Qemu-devel] [PULL 0/1] target-arm queue

[Stefan Hajnoczi]

[-- Attachment #1: Type: text/plain, Size: 1202 bytes --]

On Mon, Dec 05, 2016 at 05:59:34PM +0000, Peter Maydell wrote:
> One patch for 2.8-rc3, which is Alex's partial revert
> of 1dd089d0 to fix A64 ldaxp.
> 
> thanks
> -- PMM
> 
> The following changes since commit bc66cedb4141fb7588f2462c74310d8fb5dd4cf1:
> 
>   Merge remote-tracking branch 'yongbok/tags/mips-20161204' into staging (2016-12-05 10:56:45 +0000)
> 
> are available in the git repository at:
> 
>   git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20161205
> 
> for you to fetch changes up to 5460da501a57cd72eda6fec736d76539122e2f99:
> 
>   target-arm/translate-a64: fix gen_load_exclusive (2016-12-05 17:52:01 +0000)
> 
> ----------------------------------------------------------------
> target-arm queue:
>  * fix gen_load_exclusive handling of ldaxp
> 
> ----------------------------------------------------------------
> Alex Bennée (1):
>       target-arm/translate-a64: fix gen_load_exclusive
> 
>  target-arm/translate-a64.c | 42 +++++++++++++++++++-----------------------
>  1 file changed, 19 insertions(+), 23 deletions(-)
> 

Thanks, applied to my staging tree:
https://github.com/stefanha/qemu/commits/staging

Stefan

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]