[Spice-devel] [PATCH qxl-wddm-dod v4 03/10] Use MmMapIoSpaceEx instead of MmMapIoSpace

Javier Celaya javier.celaya at flexvdi.com
Tue Sep 20 13:55:54 UTC 2016


Hi,
El mar, 20-09-2016 a las 12:27 +0200, Christophe Fergeau escribió:
> On Tue, Sep 20, 2016 at 01:14:10PM +0300, Sameeh Jubran wrote:
> > On Mon, Sep 12, 2016 at 11:32 AM, Frediano Ziglio <
> > fziglio at redhat.com>
> > wrote:
> > > It looks like MmMapIoSpaceEx was introduced with Windows 10 while
> > > WDDM was
> > > introduce in Windows 7. Are we going to support only from Windows
> > > 10?
> > > Windows 7 supports WDDM but supports also XPDM so it's supported
> > > but
> > > users of Windows 8 won't have a QXL driver. Are we never going to
> > > support
> > > Windows 8?
> > > 
> > 
> > > 
> > > I think the reason for this change is that the new function can
> > > specify
> > > additional security, specifically removing the executable bit
> > > from page
> > > mappings. Am I right? Perhaps this should be mentioned in the
> > > rationale.
> > 
> > I think we can drop this patch if we plan to support Windows 8
> 
> It would be easier to make that decision if there was a rationale for
> the change in the commit log, or if you could answer Frediano's
> question
> as to _why_ this commit is changing from one to the other function.
> 
> Christophe

This patch also removes support for Windows 2012 server, which we plan
to use as a VDI desktop. So, we would also like to have this patch not
applied, unless there is a good reason (which I don't see either).

_______________________________________________
Spice-devel mailing list
Spice-devel at lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel




More information about the Spice-devel mailing list