On Mon, Sep 12, 2016 at 11:32 AM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
The conversion to ULONG here is not necessary.>
> Based on a patch by Sandy Stutsman <sstutsma@xxxxxxxxxx>
>
> Signed-off-by: Sameeh Jubran <sameeh@xxxxxxxxxx>
> ---
> qxldod/QxlDod.cpp | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> index d07c7b4..93fd8c6 100755
> --- a/qxldod/QxlDod.cpp
> +++ b/qxldod/QxlDod.cpp
> @@ -2009,17 +2009,17 @@ MapFrameBuffer(
> return STATUS_INVALID_PARAMETER;
> }
>
> - *VirtualAddress = MmMapIoSpace(PhysicalAddress,
> + *VirtualAddress = MmMapIoSpaceEx(PhysicalAddress,
> Length,
> - MmWriteCombined);
> + PAGE_WRITECOMBINE | PAGE_READWRITE);
> if (*VirtualAddress == NULL)
> {
> // The underlying call to MmMapIoSpace failed. This may be because,
> MmWriteCombined
> // isn't supported, so try again with MmNonCached
>
> - *VirtualAddress = MmMapIoSpace(PhysicalAddress,
> + *VirtualAddress = MmMapIoSpaceEx(PhysicalAddress,
> Length,
> - MmNonCached);
> + (ULONG) (PAGE_NOCACHE |
> PAGE_READWRITE));
I can do this change on my own, but I have some minor questions.
> if (*VirtualAddress == NULL)
> {
> DbgPrint(TRACE_LEVEL_ERROR, ("MmMapIoSpace returned a NULL
> buffer when trying to allocate %lu bytes", Length));
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 we can drop this patch if we plan 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.
OT: in the qxldod directory there is qxldod.inf and qxldod.inx file.
It looks like the project uses the .inx file to generate the final .inf
file while the .inf file in git is just an old file. Is it correct?
Should we remove the .inf file from git?
Frediano
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel