[Qemu-commits] [qemu/qemu] 422f32: s390x/helper: Fixed real-to-absolute address trans...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 422f32c5b12e392676017e637a0ebeaae194faab
      
https://github.com/qemu/qemu/commit/422f32c5b12e392676017e637a0ebeaae194faab
  Author: Thomas Huth <address@hidden>
  Date:   2014-05-07 (Wed, 07 May 2014)

  Changed paths:
    M target-s390x/helper.c

  Log Message:
  -----------
  s390x/helper: Fixed real-to-absolute address translation

The real-to-absolute address translation in mmu_translate() was
missing the second part for translating the page at the prefix
address back to the 0 page. And while we're at it, also moved the
code into a separate helper function since this might come in
handy for other parts of the code, too.

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: David Hildenbrand <address@hidden>
Reviewed-by: Alexander Graf <address@hidden>
Signed-off-by: Jens Freimann <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: c4400206d43b6a235299c7047cca0af93269fc03
      
https://github.com/qemu/qemu/commit/c4400206d43b6a235299c7047cca0af93269fc03
  Author: Thomas Huth <address@hidden>
  Date:   2014-05-07 (Wed, 07 May 2014)

  Changed paths:
    M target-s390x/cpu.h
    M target-s390x/helper.c

  Log Message:
  -----------
  s390x/helper: Added format control bit to MMU translation

With the EDAT-1 facility, the MMU translation can stop at the
segment table already, pointing to a 1 MB block. And while we're
at it, move the page table entry handling to a separate function,
too, as suggested by Alexander Graf.

Acked-by: Alexander Graf <address@hidden>
Signed-off-by: Thomas Huth <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 56bf1a8e90e76701428885656743515af53f19ef
      
https://github.com/qemu/qemu/commit/56bf1a8e90e76701428885656743515af53f19ef
  Author: Cornelia Huck <address@hidden>
  Date:   2014-05-07 (Wed, 07 May 2014)

  Changed paths:
    M hw/s390x/css.c
    M hw/s390x/css.h
    M hw/s390x/virtio-ccw.c

  Log Message:
  -----------
  s390x/css: Don't save orb in subchannel.

Current css code saves the operation request block (orb) in the
subchannel structure for later consumption by the start function
handler. This might make sense for asynchronous execution of the
start function (which qemu doesn't support), but not in our case;
it would even be wrong since orb contains a reference to a local
variable in the base ssch handler.

Let's just pass the orb through the start function call chain for
ssch; for rsch, we can pass NULL as the backend function does not
use any information passed via the orb there.

Acked-by: Christian Borntraeger <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>


  Commit: 8d1dc5d188b84a2b2aeb9bd39fddb3d2c67be60c
      
https://github.com/qemu/qemu/commit/8d1dc5d188b84a2b2aeb9bd39fddb3d2c67be60c
  Author: Peter Maydell <address@hidden>
  Date:   2014-05-07 (Wed, 07 May 2014)

  Changed paths:
    M hw/s390x/css.c
    M hw/s390x/css.h
    M hw/s390x/virtio-ccw.c
    M target-s390x/cpu.h
    M target-s390x/helper.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20140507' into staging

Some improvements for s390.

Two patches deal with address translation, one fixes a problem in the
channel subsystem code.

# gpg: Signature made Wed 07 May 2014 09:29:30 BST using RSA key ID C6F02FAF
# gpg: Can't check signature: public key not found

* remotes/cohuck/tags/s390x-20140507:
  s390x/css: Don't save orb in subchannel.
  s390x/helper: Added format control bit to MMU translation
  s390x/helper: Fixed real-to-absolute address translation

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/c9541f67df48...8d1dc5d188b8