[Spice-devel] [vd-agent-win32 v2 0/2] Add support for WDDM driver

[Victor Toso]

Hi Javier,

On Tue, Feb 16, 2016 at 10:50:22AM +0100, Javier Celaya wrote:
> El 15/02/16 a las 17:01, Javier Celaya escribió:
> >
> >
> >El 15/02/16 a las 16:38, Victor Toso escribió:
> >>Hi,
> >>
> >>On Fri, Feb 12, 2016 at 02:41:58PM +0100, Javier Celaya wrote:
> >>>This patch set adds support for dynamic resolution changes with the WDDM
> >>>driver on Windows >= 8. It is based on the one sent by Vadim Rozenfeld
> >>>with some improvements:
> >>>
> >>>- It detects both the XDDM driver and the WDDM driver, and uses the
> >>>   correct API.
> >>>- It detects if the client is trying to set the same resolution again,
> >>>   and does nothing. This is the result of the client detecting a primary
> >>>   surface destruction/construction when the resolution changes, and
> >>>   asking for a resolution change again. This bug does not happen with
> >>>   the XDDM driver, so it may be avoidable in some other way.
> >>>- It uses an alternative method to update the list of display modes,
> >>>   that results in a nicer effect when changing the resolution. This
> >>>   method is compatible with the current driver implementation (v0.12),
> >>>   but for best results we have modified the driver to drop the previous
> >>>   method and to improve support of hardware pointers. The source can be
> >>>   found inhttps://github.com/flexvdi/qxl-dod, and precompiled and signed
> >>>   binaries can be downloaded fromhttp://depot.flexvdi.com/guest-tools/.
> >>You might be interested in taking a look at Sandy's work recently pushed
> >>to freedesktop at
> >>https://cgit.freedesktop.org/~teuf/qxl-wddm-dod/
> >>
> >>and thank you for your work on this,
> >>   toso
> >Thank you, Victor. I'll see if these patches are in sync with her job.
> It turns out they are not. In particular, she changed the interface to send
> a custom resolution to the driver. I'm going to try to coordinate with her,
> in case she also has a patch set for the vdagent, so do not push these
> patches upstream yet.
>
> Thanks for pointint this out.

Sure, you are welcome.

Sadly, I don't think Sandy will be working on this anymore. At this
moment I don't know anyone working on this. The work that she has done
is the latest that I'm aware of regarding improving qxl-wddm-dod for
windows 8+.

I consider this a WIP and not yet maintained upstream...

kind regards,
  toso

> >>>Changes from v1:
> >>>- New types and classes have been moved from common/vdcommon.h to
> >>>   vdagent/wddm.h, since they are actually not common to both the vdagent
> >>>   and the vdservice. The code for those classes is in vdagent/wddm.cpp.
> >>>- Some improvements as suggested by Frediano Ziglio.
> >>>
> >>>Javier Celaya (2):
> >>>   Detect and use the new WDDM driver
> >>>   Use CCD API to update the WDDM display modes.
> >>>
> >>>  Makefile.am                |   2 +
> >>>  common/vdcommon.h          |   4 +
> >>>  vdagent/desktop_layout.cpp |  69 ++++++++++++---
> >>>  vdagent/wddm.cpp           |  76 +++++++++++++++++
> >>>  vdagent/wddm.h             | 206 +++++++++++++++++++++++++++++++++++++++++++++
> >>>  5 files changed, 345 insertions(+), 12 deletions(-)
> >>>  create mode 100644 vdagent/wddm.cpp
> >>>  create mode 100644 vdagent/wddm.h
> >>>
> >>>-- 
> >>>2.4.3
> >>>
> >>>_______________________________________________
> >>>Spice-devel mailing list
> >>>Spice-devel at lists.freedesktop.org
> >>>https://lists.freedesktop.org/mailman/listinfo/spice-devel
> >
> >-- 
> >
> >
> >
> ><http://flexvdi.com><http://flexvdi.com>
> >
> >Javier Celaya
> >
> >Software Engineer
> >
> >	
> >
> >j <mailto:javier.celaya at flexvdi.com>avier.celaya at flexvdi.com
> >
> >+34 876 60 00 73
> >
> >@j_celaya
> >
> 
> -- 
> 
> 
> 
> <http://flexvdi.com><http://flexvdi.com>
> 
> Javier Celaya
> 
> Software Engineer
> 
> 	
> 
> j <mailto:javier.celaya at flexvdi.com>avier.celaya at flexvdi.com
> 
> +34 876 60 00 73
> 
> @j_celaya
>