[patch xf86-video-intel] sna: fix --enable-debug=full

Chris Wilson chris at chris-wilson.co.uk
Fri Apr 17 13:12:00 UTC 2020


Quoting Alexei Podtelezhnikov (2020-04-17 03:44:02)
> First a typo prevents linking
>         src/sna/sna_render.h:164: multiple definition of `tightly_packed'
> 
> Then spurious assertion failure happens
>         sna_accel_flush:17413 assertion '!ret || priv->gpu_bo == NULL' failed
> 
> Both are fixed below.
> 
> Signed-off-by: Alexei Podtelezhnikov <apodtele at gmail.com>
> ---
>  src/sna/compiler.h  | 2 +-
>  src/sna/sna_accel.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/sna/compiler.h b/src/sna/compiler.h
> index 2e579b15..c3d98797 100644
> --- a/src/sna/compiler.h
> +++ b/src/sna/compiler.h
> @@ -50,7 +50,7 @@
>  #define must_check
>  #define constant
>  #define pure
> -#define tighly_packed
> +#define tightly_packed
>  #define flatten
>  #define nonnull
>  #define page_aligned
> diff --git a/src/sna/sna_accel.c b/src/sna/sna_accel.c
> index ee857a14..bf1edec8 100644
> --- a/src/sna/sna_accel.c
> +++ b/src/sna/sna_accel.c
> @@ -17410,7 +17410,7 @@ void sna_accel_flush(struct sna *sna)
>                         assert(!priv->flush);
>                         ret = sna_pixmap_move_to_cpu(priv->pixmap,
>                                                      MOVE_READ | MOVE_WRITE);
> -                       assert(!ret || priv->gpu_bo == NULL);
> +                       assert(ret || priv->gpu_bo == NULL);

That's not spurious. It says "we must succeed in moving the GPU pixmap
to the CPU, or have a good reason like there being no GPU pixmap".
-Chris


More information about the intel-gfx-bugs mailing list