[PATCH] DRI2: make target_sbc_signed

Jesse Barnes jbarnes at virtuousgeek.org
Thu Mar 4 10:18:33 PST 2010


On Thu, 4 Mar 2010 09:58:57 -0800
Jesse Barnes <jbarnes at virtuousgeek.org> wrote:

> We need to track invalid targets as well as 0 targets, so just make it
> signed so our comparisons work like they should.
> 
> Reported-by: Kristian Høgsberg <krh at bitplanet.net>
> Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> ---
>  hw/xfree86/dri2/dri2.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2.c
> index 301f4fd..55846b1 100644
> --- a/hw/xfree86/dri2/dri2.c
> +++ b/hw/xfree86/dri2/dri2.c
> @@ -62,7 +62,7 @@ typedef struct _DRI2Drawable {
>      ClientPtr		 blockedClient;
>      int			 swap_interval;
>      CARD64		 swap_count;
> -    CARD64		 target_sbc; /* -1 means no SBC wait outstanding */
> +    INT64		 target_sbc; /* -1 means no SBC wait outstanding */
>      CARD64		 last_swap_target; /* most recently queued swap target */
>      int			 swap_limit; /* for N-buffering */
>  } DRI2DrawableRec, *DRI2DrawablePtr;

Damnit, I thought I had compiled this one, but I only compile tested
int64_t.  INT64 is a fiction that Xmd.h would lead you to believe
exists, when it actually doesn't.

I'll re-post yet again.

-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the xorg-devel mailing list