[Spice-devel] [PATCH v3 0/6] Set of patches for further support of VSync

Yuri Benditovich yuri.benditovich at daynix.com
Sat Apr 8 10:46:49 UTC 2017


The main goal is to reduce time in GDI callback (PresentDisplayOnly) and avoid
situation when the processing takes more than 2 seconds causing class driver
watchdog.

1. We offload sending of drawable commands to separate thread (waiting for room in command ring
   may take unpredictable time)
2. In case the usage of device memory is high, allocation of bitmap for rectangle to draw
   also may take unpredictable time (note that single full screen redraw requires >3 MB of space)
   We make drawable objects allocation from GDI callback fast and non-forced and in case they
   fail we provide alternate allocation from OS heap
3. The thread before it sends drawable command down takes care on objects that were allocated from
   OS heap and allocates them from device memory (forced, as now we are not limited by time)
4. We still do not enable VSync automatically, but this can be done for evaluation/testing purpose via
   setting in the driver's registry

Changes from V2:
Fixed coding style
Added checks for allocation result (patch 1)
Removed duplicated code (patch 2)
Splitted commit of non-forced bitmap allocation (patches 4-5)
Added reduced processing in thread after PnP stop received (patch 6)
Extended commit comment (patch 6)
Other cosmetic changes

Changes from V1:
Changed allocations style and coding style
Removed additional synchronization on deallocation upon allocation failure
Removed waiting for thread start on driver startup
Other cosmetic changes

yuri.benditovich at daynix.com (3):
  qxl-wddm-dod: Prepare for failure to allocate memory
  qxl-wddm-dod: PutBytesAlign supports non-forced allocation
  qxl-wddm-dod: Optimize allocation of memory chunks
  qxl-wddm-dod: Dedicated procedure for bitmap creation
  qxl-wddm-dod: Implement non-forced bitmap allocation
  qxl-wddm-dod: Non-forced memory allocations with VSync

 qxldod/QxlDod.cpp | 390 ++++++++++++++++++++++++++++++++++++++++++------------
 qxldod/QxlDod.h   |  18 ++-
 2 files changed, 324 insertions(+), 84 deletions(-)

-- 
2.7.0.windows.1



More information about the Spice-devel mailing list