>
> Part of source image mapped by PresentDisplayOnly
> should be big enough to cover all rectangles being
> transferred.
>
> Signed-off-by: Sameeh Jubran <sameeh@xxxxxxxxxx>
> Signed-off-by: Dmitry Fleytman <dmitry@xxxxxxxxxx>
> ---
> qxldod/QxlDod.cpp | 28 +++++++++++++++++++++++-----
> qxldod/QxlDod.h | 2 ++
> 2 files changed, 25 insertions(+), 5 deletions(-)
>
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> index 577b4ff..1f54174 100755
> --- a/qxldod/QxlDod.cpp
> +++ b/qxldod/QxlDod.cpp
> @@ -3772,12 +3772,11 @@ QxlDevice::ExecutePresentDisplayOnly(
> ctx->Mdl = NULL;
> ctx->DisplaySource = this;
>
> - // Alternate between synch and asynch execution, for demonstrating
> - // that a real hardware implementation can do either
> -
> + // Source bitmap is in user mode, must be locked under __try/__except
> + // and mapped to kernel space before use.
> {
> - // Map Source into kernel space, as Blt will be executed by system
> worker thread
> - UINT sizeToMap = ctx->SrcPitch * ctx->SrcHeight;
> + LONG maxHeight = GetMaxSourceMappingHeight(ctx->Moves,
> ctx->NumMoves, ctx->DirtyRect, ctx->NumDirtyRects);
> + UINT sizeToMap = ctx->SrcPitch * maxHeight;
>
> PMDL mdl = IoAllocateMdl((PVOID)SrcAddr, sizeToMap, FALSE, FALSE,
> NULL);
> if(!mdl)
> @@ -4691,6 +4690,25 @@ void QxlDevice::SetMonitorConfig(QXLHead *
> monitor_config)
> AsyncIo(QXL_IO_MONITORS_CONFIG_ASYNC, 0);
> }
>
> +LONG QxlDevice::GetMaxSourceMappingHeight(D3DKMT_MOVE_RECT* Moves, ULONG
> NumMoves, RECT* DirtyRects, ULONG NumDirtyRects)
> +{
> + LONG maxHeight = 0;
> + if (Moves != NULL) {
> + for (UINT i = 0; i < NumMoves; i++) {
> + const POINT& pSourcePoint = Moves[i].SourcePoint;
> + const RECT& pDestRect = Moves[i].DestRect;
> + maxHeight = MAX(maxHeight, pDestRect.bottom - pDestRect.top +
> pSourcePoint.y);
> + }
> + }
> + if (DirtyRects != NULL) {
> + for (UINT i = 0; i < NumDirtyRects; i++) {
> + const RECT& pDirtyRect = DirtyRects[i];
> + maxHeight = MAX(maxHeight, pDirtyRect.bottom);
> + }
> + }
> + return maxHeight;
> +}
> +
> QXL_PAGED
> NTSTATUS QxlDevice::Escape(_In_ CONST DXGKARG_ESCAPE* pEscape)
> {
> diff --git a/qxldod/QxlDod.h b/qxldod/QxlDod.h
> index ac33faa..f4a115b 100755
> --- a/qxldod/QxlDod.h
> +++ b/qxldod/QxlDod.h
> @@ -647,6 +647,8 @@ private:
> QXL_PAGED
> void SetMonitorConfig(QXLHead* monitor_config);
>
> + static LONG GetMaxSourceMappingHeight(D3DKMT_MOVE_RECT* Moves, ULONG
> NumMoves, RECT* DirtyRects, ULONG NumDirtyRects);
> +
> private:
> PUCHAR m_IoBase;
> BOOLEAN m_IoMapped;
Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel