[PATCH] EXA: ModifyPixmapHeader_mixed fixes.
Maarten Maathuis
madman2003 at gmail.com
Tue Dec 8 09:52:30 PST 2009
You might still want to set to deferred pixmap to NULL if it currently
is this pixmap.
2009/12/8 Michel Dänzer <michel at daenzer.net>:
> From: Michel Dänzer <daenzer at vmware.com>
>
> * Better detection of dimension changes.
> * Make sure to re-create the system memory copy when the pixmap dimensions
> change (e.g. the screen pixmap on screen resize).
> * Clear the valid regions.
>
> Signed-off-by: Michel Dänzer <daenzer at vmware.com>
> ---
>
> Maarten, note that I don't think there's any point in dealing with deferred
> pixmaps here after all, since the GPU copy contents won't be considered valid
> anyway.
>
>
> exa/exa_mixed.c | 40 +++++++++++++++++++++++++++++++++++++---
> 1 files changed, 37 insertions(+), 3 deletions(-)
>
> diff --git a/exa/exa_mixed.c b/exa/exa_mixed.c
> index 7dc426f..e55ce33 100644
> --- a/exa/exa_mixed.c
> +++ b/exa/exa_mixed.c
> @@ -135,17 +135,51 @@ exaModifyPixmapHeader_mixed(PixmapPtr pPixmap, int width, int height, int depth,
> pExaPixmap->score = EXA_PIXMAP_SCORE_PINNED;
> }
>
> - if (pExaPixmap->driverPriv) {
> - if (width > 0 && height > 0 && bitsPerPixel > 0) {
> + has_gpu_copy = exaPixmapHasGpuCopy(pPixmap);
> +
> + if (width <= 0)
> + width = pPixmap->drawable.width;
> +
> + if (height <= 0)
> + height = pPixmap->drawable.height;
> +
> + if (bitsPerPixel <= 0) {
> + if (depth <= 0)
> + bitsPerPixel = pPixmap->drawable.bitsPerPixel;
> + else
> + bitsPerPixel = BitsPerPixel(depth);
> + }
> +
> + if (depth <= 0)
> + depth = pPixmap->drawable.depth;
> +
> + if (width != pPixmap->drawable.width ||
> + height != pPixmap->drawable.height ||
> + devKind != pPixmap->devKind ||
> + depth != pPixmap->drawable.depth ||
> + bitsPerPixel != pPixmap->drawable.bitsPerPixel) {
> + if (pExaPixmap->driverPriv) {
> exaSetFbPitch(pExaScr, pExaPixmap,
> width, height, bitsPerPixel);
>
> exaSetAccelBlock(pExaScr, pExaPixmap,
> width, height, bitsPerPixel);
> + REGION_EMPTY(pScreen, &pExaPixmap->validFB);
> }
> +
> + /* Need to re-create system copy if there's also a GPU copy */
> + if (has_gpu_copy && pExaPixmap->sys_ptr) {
> + free(pExaPixmap->sys_ptr);
> + pExaPixmap->sys_ptr = NULL;
> + pExaPixmap->sys_pitch = devKind > 0 ? devKind :
> + PixmapBytePad(width, depth);
> + DamageUnregister(&pPixmap->drawable, pExaPixmap->pDamage);
> + DamageDestroy(pExaPixmap->pDamage);
> + pExaPixmap->pDamage = NULL;
> + REGION_EMPTY(pScreen, &pExaPixmap->validSys);
> + }
> }
>
> - has_gpu_copy = exaPixmapHasGpuCopy(pPixmap);
> if (has_gpu_copy) {
> pPixmap->devPrivate.ptr = pExaPixmap->fb_ptr;
> pPixmap->devKind = pExaPixmap->fb_pitch;
> --
> 1.6.4.3
>
> _______________________________________________
> xorg-devel mailing list
> xorg-devel at lists.x.org
> http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list