Re: [PATCH xf86-video-intel] uxa: only call intel_sync_close when built with HAVE_DRI3

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

 



On Fri, Sep 02, 2016 at 08:58:22PM +1000, Jonathan Gray wrote:
> Avoid calling a function only built with dri3, fixes an undefined
> symbol crash when opting into uxa reported by Walter Alejandro Iglesias
> when running OpenBSD.
> 
> Signed-off-by: Jonathan Gray <jsg@xxxxxxxxx>
> ---
>  src/uxa/intel_driver.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/uxa/intel_driver.c b/src/uxa/intel_driver.c
> index 3703c41..9fb05c4 100644
> --- a/src/uxa/intel_driver.c
> +++ b/src/uxa/intel_driver.c
> @@ -1191,7 +1191,9 @@ static Bool I830CloseScreen(CLOSE_SCREEN_ARGS_DECL)
>  		intel->dri3 = DRI_NONE;
>  	}
>  
> +#if HAVE_DRI3
>  	intel_sync_close(screen);
> +#endif

In src/uxa/intel.h, we have

#if HAVE_DRI3
Bool intel_sync_init(ScreenPtr screen);
void intel_sync_close(ScreenPtr screen);
#else
static inline Bool intel_sync_init(ScreenPtr screen) { return 0; }
static inline void intel_sync_close(ScreenPtr screen) { }
#endif

Ideas?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux