<html><body><div style="font-family: times new roman, new york, times, serif; font-size: 12pt; color: #000000"><div>Acked-by: Frediano Ziglio <fziglio@redhat.com></div><div><br></div><div>Frediano<br></div><div><br></div><hr id="zwchr"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><b>From: </b>"Yuri Benditovich" <yuri.benditovich@daynix.com><br><b>To: </b>"Frediano Ziglio" <fziglio@redhat.com><br><b>Cc: </b>spice-devel@lists.freedesktop.org, "Yan Vugenfirer" <yan@daynix.com><br><b>Sent: </b>Wednesday, January 25, 2017 5:45:23 PM<br><b>Subject: </b>Re: [Spice-devel] [PATCH] qxl-wddm-dod: Fix for screen not shown after driver disabled<br><div><br></div><div dir="ltr">We use this name as video mode that the driver uses and passes to the BasicVideo is expected to be supported by BIOS or UEFI. "VGA Compatible" sounds like conformance to some specification, which is not a case. Name of the field "<span style="font-size:12.8px">SupportNonVGA" related to this feature is also inclear and seems at least inverted. </span><span style="font-size:12.8px">The difference between VGA mode and QXL mode is that first uses video modes declared by the BIOS and changes them via BIOS calls, when the seconds does not.</span><div><div><div><br></div></div></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 25, 2017 at 6:26 PM, Frediano Ziglio <span dir="ltr"><<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">><br>
> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1411340" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1411340</a><br>
> Do not set SupportNonVGA field for device rev.4 and higher.<br>
> Then the class driver will not call miniport's procedure<br>
> StopDeviceAndReleasePostDisplayOwnership upon PnP stop.<br>
> QXL device modes are not compatible with VGA ones; the<br>
> driver in this procedure returns display information related<br>
> to QXL mode which later used by BasicDisplay driver and causes<br>
> it to show the screen incorrectly or fail to show it at all.<br>
><br>
> Signed-off-by: Yuri Benditovich <<a href="mailto:yuri.benditovich@daynix.com" target="_blank">yuri.benditovich@daynix.com</a>><br>
> ---<br>
> qxldod/QxlDod.cpp | 2 +-<br>
> qxldod/QxlDod.h | 2 ++<br>
> 2 files changed, 3 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp<br>
> index 7adbec4..001cd67 100755<br>
> --- a/qxldod/QxlDod.cpp<br>
> +++ b/qxldod/QxlDod.cpp<br>
> @@ -411,7 +411,7 @@ NTSTATUS QxlDod::QueryAdapterInfo(_In_ CONST<br>
> DXGKARG_QUERYADAPTERINFO* pQueryAda<br>
> pDriverCaps->PointerCaps.Monochrome = 1;<br>
> pDriverCaps->PointerCaps.Color = 1;<br>
><br>
> - pDriverCaps->SupportNonVGA = TRUE;<br>
> + pDriverCaps->SupportNonVGA = m_pHWDevice->IsBIOSCompatible();<br>
><br>
> DbgPrint(TRACE_LEVEL_VERBOSE, ("<--- %s 1\n", __FUNCTION__));<br>
> return STATUS_SUCCESS;<br>
> diff --git a/qxldod/QxlDod.h b/qxldod/QxlDod.h<br>
> index 4df4c61..8df1fcf 100755<br>
> --- a/qxldod/QxlDod.h<br>
> +++ b/qxldod/QxlDod.h<br>
> @@ -263,6 +263,7 @@ public:<br>
> virtual NTSTATUS Escape(_In_ CONST DXGKARG_ESCAPE* pEscap) = 0;<br>
> NTSTATUS AcquireDisplayInfo(DXGK_DISPLAY_INFORMATION& DispInfo);<br>
> ULONG GetId(void) { return m_Id; }<br>
> + virtual BOOLEAN IsBIOSCompatible() { return TRUE; }<br>
> protected:<br>
> virtual NTSTATUS GetModeList(DXGK_DISPLAY_INFORMATION* pDispInfo) = 0;<br>
> protected:<br>
> @@ -480,6 +481,7 @@ public:<br>
> NTSTATUS SetPointerShape(_In_ CONST DXGKARG_SETPOINTERSHAPE*<br>
> pSetPointerShape);<br>
> NTSTATUS SetPointerPosition(_In_ CONST DXGKARG_SETPOINTERPOSITION*<br>
> pSetPointerPosition);<br>
> NTSTATUS Escape(_In_ CONST DXGKARG_ESCAPE* pEscap);<br>
> + BOOLEAN IsBIOSCompatible() { return FALSE; }<br>
> protected:<br>
> NTSTATUS GetModeList(DXGK_DISPLAY_INFORMATION* pDispInfo);<br>
> VOID BltBits (BLT_INFO* pDst,<br>
<br>
</div></div>Why you used IsBIOSCompatible name instead of IsVGACompatible ?<br>
<span class="HOEnZb"><span data-mce-style="color: #888888;" style="color: #888888;"><br>
Frediano<br>
</span></span></blockquote></div><br></div>
</blockquote><div><br></div></div></body></html>