Re: [PATCH qxl-wddm-dod v2 23/25] Fixing monitor flicker on resolution change

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> Passing the flag QXL_SURF_FLAG_KEEP_DATA to the new created surface
> causes weird flickering, thus this should be set to 0.
> 
> Signed-off-by: Sameeh Jubran <sameeh@xxxxxxxxxx>
> ---
>  qxldod/QxlDod.cpp | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/qxldod/QxlDod.cpp b/qxldod/QxlDod.cpp
> index e869578..a085ba5 100755
> --- a/qxldod/QxlDod.cpp
> +++ b/qxldod/QxlDod.cpp
> @@ -3434,13 +3434,13 @@ void
> QxlDevice::CreatePrimarySurface(PVIDEO_MODE_INFORMATION pModeInfo)
>      primary_surface_create->width = pModeInfo->VisScreenWidth;
>      primary_surface_create->height = pModeInfo->VisScreenHeight;
>      primary_surface_create->stride = pModeInfo->ScreenStride;
> -
> -    primary_surface_create->mem = PA( m_RamStart, m_MainMemSlot);
> -
> -    primary_surface_create->flags = QXL_SURF_FLAG_KEEP_DATA;
> -    primary_surface_create->type = QXL_SURF_TYPE_PRIMARY;
> -    DbgPrint(TRACE_LEVEL_VERBOSE, ("<--> %s format = %d, width = %d, height
> = %d, stride = %d\n", __FUNCTION__, pModeInfo->BitsPerPlane,
> pModeInfo->VisScreenWidth, pModeInfo->VisScreenHeight,
> -                                     pModeInfo->ScreenStride));
> +
> +    primary_surface_create->mem = PA( m_RamStart, m_MainMemSlot);
> +
> +    primary_surface_create->flags = 0;
> +    primary_surface_create->type = QXL_SURF_TYPE_PRIMARY;

This is the single line changed, other are just space changes
and should be removed.

> +    DbgPrint(TRACE_LEVEL_VERBOSE, ("<--> %s format = %d, width = %d, height
> = %d, stride = %d\n", __FUNCTION__, pModeInfo->BitsPerPlane,
> pModeInfo->VisScreenWidth, pModeInfo->VisScreenHeight,
> +                                     pModeInfo->ScreenStride));
>  //    AsyncIo(QXL_IO_CREATE_PRIMARY_ASYNC, 0);
>      SyncIo(QXL_IO_CREATE_PRIMARY, 0);
>      DbgPrint(TRACE_LEVEL_VERBOSE, ("<--- %s\n", __FUNCTION__));

Beside that,

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

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




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]