[Spice-devel] [PATCH 0/8] cleanups and allow checked builds of qxl driver

[Arnon Gilboa]

ack for all, but in 5/8 pls ifdef EngDebugBreak();

Alon Levy wrote:
> This is part 1 of 4 for the S3 support:
>  1. cleanup + checked build (this one) + FreeSurface bugfix (Yonit).
>  2. QXL_IO_LOG for miniport (seperated because it includes an
>     snprintf implementation and so is large)
>  3. cleanup after QXL_IO_LOG (including helper debug routines)
>  4. S3 implmenetation via QXL_UPDATE_MEM, punting, and copy of surfaces to
>     ram & back.
>
> Alon Levy (8):
>   display: use WAIT_FOR_EVENT everywhere
>   display: fix broken debug in checked
>   display/res: turn two asserts in CursorCacheRemove to debug prints
>   display/driver: DebugPrint: check for log_level being set (allow
>     using DebugPrint with pdev early)
>   display/res: better error message for FreeMem assert
>   treewide: staticfy some functions, small indentation fixes
>   display/surface: CreateDeviceBitmap: remove redundant FreeSurface
>   display/driver: DebugPrintV: check for pdev->Res before dereferencing
>     it
>
>  display/driver.c  |   16 +++++++----
>  display/quic.c    |    4 +++
>  display/res.c     |   68 +++++++++++++++++++---------------------------------
>  display/surface.c |    7 ++---
>  display/surface.h |    4 ---
>  miniport/qxl.c    |    4 +-
>  6 files changed, 44 insertions(+), 59 deletions(-)
>
>