<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 12, 2016 at 11:32 AM, 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>
> Based on a patch by Sandy Stutsman <<a href="mailto:sstutsma@redhat.com">sstutsma@redhat.com</a>><br>
><br>
> Signed-off-by: Sameeh Jubran <<a href="mailto:sameeh@daynix.com">sameeh@daynix.com</a>><br>
> ---<br>
>  qxldod/QxlDod.cpp | 8 ++++----<br>
>  1 file changed, 4 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp<br>
> index d07c7b4..93fd8c6 100755<br>
> --- a/qxldod/QxlDod.cpp<br>
> +++ b/qxldod/QxlDod.cpp<br>
> @@ -2009,17 +2009,17 @@ MapFrameBuffer(<br>
>          return STATUS_INVALID_PARAMETER;<br>
>      }<br>
><br>
> -    *VirtualAddress = MmMapIoSpace(PhysicalAddress,<br>
> +    *VirtualAddress = MmMapIoSpaceEx(<wbr>PhysicalAddress,<br>
>                                     Length,<br>
> -                                   MmWriteCombined);<br>
> +                                   PAGE_WRITECOMBINE | PAGE_READWRITE);<br>
>      if (*VirtualAddress == NULL)<br>
>      {<br>
>          // The underlying call to MmMapIoSpace failed. This may be because,<br>
>          MmWriteCombined<br>
>          // isn't supported, so try again with MmNonCached<br>
><br>
> -        *VirtualAddress = MmMapIoSpace(PhysicalAddress,<br>
> +        *VirtualAddress = MmMapIoSpaceEx(<wbr>PhysicalAddress,<br>
>                                         Length,<br>
> -                                       MmNonCached);<br>
> +                                     (ULONG) (PAGE_NOCACHE |<br>
> PAGE_READWRITE));<br>
<br>
</div></div>The conversion to ULONG here is not necessary.<br>
I can do this change on my own, but I have some minor questions.<br>
<span class=""><br>
>          if (*VirtualAddress == NULL)<br>
>          {<br>
>              DbgPrint(TRACE_LEVEL_ERROR, ("MmMapIoSpace returned a NULL<br>
>              buffer when trying to allocate %lu bytes", Length));<br>
<br>
</span>It looks like MmMapIoSpaceEx was introduced with Windows 10 while WDDM was<br>
introduce in Windows 7. Are we going to support only from Windows 10?<br>
Windows 7 supports WDDM but supports also XPDM so it's supported but<br>
users of Windows 8 won't have a QXL driver. Are we never going to support<br>
Windows 8?<br></blockquote><div>I think we can drop this patch if we plan to support Windows 8</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I think the reason for this change is that the new function can specify<br>
additional security, specifically removing the executable bit from page<br>
mappings. Am I right? Perhaps this should be mentioned in the rationale.<br>
<br>
OT: in the qxldod directory there is qxldod.inf and qxldod.inx file.<br>
It looks like the project uses the .inx file to generate the final .inf<br>
file while the .inf file in git is just an old file. Is it correct?<br>
Should we remove the .inf file from git?<br>
<span class="HOEnZb"><font color="#888888"><br>
Frediano<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><font size="4" color="#0b5394" face="times new roman, serif">Respectfully,<br></font><div style="font-size:12.8px;color:rgb(136,136,136)"><font size="4" color="#0b5394" face="times new roman, serif"><b><i>Sameeh Jubran</i></b></font></div><div style="font-size:12.8px;color:rgb(136,136,136)"><i style="color:rgb(7,55,99);font-family:"times new roman",serif;font-size:large"><span style="line-height:15px"><a href="https://il.linkedin.com/pub/sameeh-jubran/87/747/a8a" title="View public profile" name="UNIQUE_ID_SafeHtmlFilter_UNIQUE_ID_SafeHtmlFilter_UNIQUE_ID_SafeHtmlFilter_UNIQUE_ID_SafeHtmlFilter_14e2c1de96f8c195_UNIQUE_ID_SafeHtmlFilter_SafeHtmlFilter_SafeHtmlFilter_webProfileURL" style="color:rgb(17,85,204);margin:0px;padding:0px;border-width:0px;outline:none;vertical-align:baseline;text-decoration:none" target="_blank">Linkedin</a></span></i><br></div><div style="font-size:12.8px;color:rgb(136,136,136)"><font size="4" face="times new roman, serif" color="#073763"><i>Junior Software Engineer @ <a href="http://www.daynix.com" target="_blank">Daynix</a>.</i></font></div></div></div></div></div></div></div>
</div></div>