qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 0ac241: unicore32: abort when entering "x 0"

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 0ac241: unicore32: abort when entering "x 0" on the monito...
Date: Mon, 14 Aug 2017 05:35:06 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0ac241bcf9f9d99a252a352a162f4b13b24732ab
      
https://github.com/qemu/qemu/commit/0ac241bcf9f9d99a252a352a162f4b13b24732ab
  Author: Eduardo Otubo <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M target/unicore32/softmmu.c

  Log Message:
  -----------
  unicore32: abort when entering "x 0" on the monitor

Starting Qemu with "qemu-system-unicore32 -M puv3,accel=qtest -S -nographic"
and entering "x 0 " at the monitor prompt leads to abort():

  $ ./unicore32-softmmu/qemu-system-unicore32 -M puv3,accel=qtest -S -nographic
  QEMU 2.9.90 monitor - type 'help' for more information
  (qemu) x 0
  qemu: fatal: uc32_cpu_get_phys_page_debug not supported yet

  R00=00000000 R01=00000000 R02=00000000 R03=00000000
  R04=00000000 R05=00000000 R06=00000000 R07=00000000
  R08=00000000 R09=00000000 R10=00000000 R11=00000000
  R12=00000000 R13=00000000 R14=00000000 R15=00000000
  R16=00000000 R17=00000000 R18=00000000 R19=00000000
  R20=00000000 R21=00000000 R22=00000000 R23=00000000
  R24=00000000 R25=00000000 R26=00000000 R27=00000000
  R28=00000000 R29=00000000 R30=00000000 R31=03000000
  PSR=40000013 -Z-- PRIV
  Aborted (core dumped)

This happens because uc32_cpu_get_phys_page_debug() is not implemented
yet, this is a temporary workaround to avoid the crash.

Signed-off-by: Eduardo Otubo <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e8ec0117b21150d2ea888bbcf6d8fe9683332a9e
      
https://github.com/qemu/qemu/commit/e8ec0117b21150d2ea888bbcf6d8fe9683332a9e
  Author: Eric Blake <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M tests/libqtest.h

  Log Message:
  -----------
  libqtest: Fix typo in comments

s/continuosly/continuously/

Signed-off-by: Eric Blake <address@hidden>
Reviewed-by: Jeff Cody <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 88552b553b0f2d75f0f91a247f19c2e3b81ed5aa
      
https://github.com/qemu/qemu/commit/88552b553b0f2d75f0f91a247f19c2e3b81ed5aa
  Author: Thomas Huth <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M qemu-doc.texi

  Log Message:
  -----------
  qemu-doc: Fix "-net van" typo

While Andrew S. Tanenbaum has a point by saying "Never underestimate the
bandwidth of a station wagon full of tapes hurtling down the highway",
we don't support that way of transportation in QEMU yet, so replace the
typo with the correct word "vlan".

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 657c572afe8dae393f2321db8866c02b4bf7bc77
      
https://github.com/qemu/qemu/commit/657c572afe8dae393f2321db8866c02b4bf7bc77
  Author: Cleber Rosa <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M tests/qemu-iotests/check
    M tests/qemu-iotests/common.rc

  Log Message:
  -----------
  qemu-iotests: get rid of _full_imgproto_details()

Although this function is used, its implementation does nothing
besides echoing a variable name.  There's no need to wrap this
functionality in a function, and based on the one usage it has, it's
not even required to adhere to a convention or code style.

Signed-off-by: Cleber Rosa <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 80758ec57d26bd2751f204ade356db094ce4b08c
      
https://github.com/qemu/qemu/commit/80758ec57d26bd2751f204ade356db094ce4b08c
  Author: Cleber Rosa <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M tests/qemu-iotests/check

  Log Message:
  -----------
  qemu-iotests: remove commented out variables

The variables FULL_MKFS_OPTIONS and FULL_MOUNT_OPTIONS are commented
out, never used, and even refer to functions that do exist.  The last
time these were touched was around 8 years ago, so I guess it's safe
to assume outputting such information on test execution is still on the
radar.

Signed-off-by: Cleber Rosa <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: a3e08c2dbd7db9b86e174f5e50c6aa7a59a2b21d
      
https://github.com/qemu/qemu/commit/a3e08c2dbd7db9b86e174f5e50c6aa7a59a2b21d
  Author: Cleber Rosa <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M tests/qemu-iotests/check

  Log Message:
  -----------
  qemu-iotests: remove comment about root privileges requirement

The check script contains a commented out root user requirement,
probably because of its xfstests heritage.  This requirement doesn't
apply to qemu-iotests, so it better be gone.

Signed-off-by: Cleber Rosa <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: a808c0865b720e22ca2929ec3d362d4610fbad51
      
https://github.com/qemu/qemu/commit/a808c0865b720e22ca2929ec3d362d4610fbad51
  Author: Thomas Huth <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M hw/misc/mmio_interface.c

  Log Message:
  -----------
  hw/misc/mmio_interface: Return after error_setg() to avoid crash

QEMU currently abort()s if the user tries to specify the mmio_interface
device without parameters:

x86_64-softmmu/qemu-system-x86_64 -nographic -device mmio_interface
qemu-system-x86_64: /home/thuth/devel/qemu/util/error.c:57: error_setv:
 Assertion `*errp == ((void *)0)' failed.
Aborted (core dumped)

This happens because the realize function is trying to set the errp
twice in this case. After setting an error, the realize function
should immediately return instead.

Signed-off-by: Thomas Huth <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 87ecd4f3408265fc6403dfbe34fb885057f804ec
      
https://github.com/qemu/qemu/commit/87ecd4f3408265fc6403dfbe34fb885057f804ec
  Author: Peter Maydell <address@hidden>
  Date:   2017-08-14 (Mon, 14 Aug 2017)

  Changed paths:
    M hw/misc/mmio_interface.c
    M target/unicore32/softmmu.c
    M tests/libqtest.h
    M tests/qemu-iotests/check
    M tests/qemu-iotests/common.rc

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/mjt/tags/trivial-patches-fetch' into 
staging

trivial patches for 2017-08-14

# gpg: Signature made Mon 14 Aug 2017 12:07:03 BST
# gpg:                using RSA key 0x701B4F6B1A693E59
# gpg: Good signature from "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# Primary key fingerprint: 6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 8044 65C5
#      Subkey fingerprint: 7B73 BAD6 8BE7 A2C2 8931  4B22 701B 4F6B 1A69 3E59

* remotes/mjt/tags/trivial-patches-fetch:
  hw/misc/mmio_interface: Return after error_setg() to avoid crash
  qemu-iotests: remove comment about root privileges requirement
  qemu-iotests: remove commented out variables
  qemu-iotests: get rid of _full_imgproto_details()
  qemu-doc: Fix "-net van" typo
  libqtest: Fix typo in comments
  unicore32: abort when entering "x 0" on the monitor

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


Compare: https://github.com/qemu/qemu/compare/6fef22d7e650...87ecd4f34082
reply via email to
[Prev in Thread] Current Thread [Next in Thread]