[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] a703d3: qxl: switch display on entering VGA
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] a703d3: qxl: switch display on entering VGA |
Date: |
Tue, 11 Apr 2017 03:00:12 -0700 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: a703d3aef5991b72a5a45880e7491232b8032f09
https://github.com/qemu/qemu/commit/a703d3aef5991b72a5a45880e7491232b8032f09
Author: Marc-André Lureau <address@hidden>
Date: 2017-04-07 (Fri, 07 Apr 2017)
Changed paths:
M hw/display/qxl.c
Log Message:
-----------
qxl: switch display on entering VGA
Since commit cd958edb1fae85d, same size console resize is skipped. This
change broke QXL incoming migration in VGA mode,
qemu_spice_display_switch() is no longer called during qxl_post_load(),
because default message surface is of the same size, and during
displaychangelistener registration, PCIQXLDevice.mode is
QXL_MODE_UNDEFINED. This triggers a later crash on refresh:
==2634== Invalid read of size 4
==3516== at 0x65F3050: pixman_image_get_data (in
/usr/lib64/libpixman-1.so.0.34.0)
==3516== by 0x6F0CEB: qemu_spice_create_update (spice-display.c:215)
==3516== by 0x6F1CC7: qemu_spice_display_refresh (spice-display.c:502)
==3516== by 0x58CF77: display_refresh (qxl.c:1948)
==3516== by 0x6E8084: do_safe_dpy_refresh (console.c:1591)
==3516== by 0x6E80D5: dpy_refresh (console.c:1604)
==3516== by 0x6E4508: gui_update (console.c:201)
==3516== by 0x81898E: timerlist_run_timers (qemu-timer.c:536)
==3516== by 0x8189D6: qemu_clock_run_timers (qemu-timer.c:547)
==3516== by 0x818D98: qemu_clock_run_all_timers (qemu-timer.c:662)
==3516== by 0x81952A: main_loop_wait (main-loop.c:514)
==3516== by 0x4ADD29: main_loop (vl.c:1898)
One way to solve this is to explicitely call qemu_spice_display_switch()
on entering VGA mode, which is called during qxl_post_load().
Fixes:
"null pointer access on migration resume of systemrescuecd boot menu with
qxl-vga"
https://bugs.launchpad.net/qemu/+bug/1679126
https://bugzilla.redhat.com/show_bug.cgi?id=1438566
Signed-off-by: Marc-André Lureau <address@hidden>
Message-id: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
Commit: 86dbcdd9c7590d06db89ca256c5eaf0b4aba8858
https://github.com/qemu/qemu/commit/86dbcdd9c7590d06db89ca256c5eaf0b4aba8858
Author: Gerd Hoffmann <address@hidden>
Date: 2017-04-11 (Tue, 11 Apr 2017)
Changed paths:
M hw/display/qxl.c
M hw/display/qxl.h
Log Message:
-----------
qxl: add migration blocker to avoid pre-save assert
Cc: address@hidden
Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Message-id: address@hidden
Commit: 17fa24b79ceaa8258e415822cf2e2cb23cf3facb
https://github.com/qemu/qemu/commit/17fa24b79ceaa8258e415822cf2e2cb23cf3facb
Author: Peter Maydell <address@hidden>
Date: 2017-04-11 (Tue, 11 Apr 2017)
Changed paths:
M hw/display/qxl.c
M hw/display/qxl.h
Log Message:
-----------
Merge remote-tracking branch 'remotes/kraxel/tags/pull-fixes-20170411-1' into
staging
qxl: bugfixes.
# gpg: Signature made Tue 11 Apr 2017 08:00:00 BST
# gpg: using RSA key 0x4CB6D8EED3E87138
# gpg: Good signature from "Gerd Hoffmann (work) <address@hidden>"
# gpg: aka "Gerd Hoffmann <address@hidden>"
# gpg: aka "Gerd Hoffmann (private) <address@hidden>"
# Primary key fingerprint: A032 8CFF B93A 17A7 9901 FE7D 4CB6 D8EE D3E8 7138
* remotes/kraxel/tags/pull-fixes-20170411-1:
qxl: add migration blocker to avoid pre-save assert
qxl: switch display on entering VGA
Signed-off-by: Peter Maydell <address@hidden>
Compare: https://github.com/qemu/qemu/compare/c322fdd41ad5...17fa24b79cea
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] a703d3: qxl: switch display on entering VGA,
GitHub <=