On Mon, Feb 08, 2016 at 08:56:06PM +0200, Gabriel Feceoru wrote:
> Some platforms have ring BSD available but no BSD2.
> Because of the current verification, tests involving ring BSD1
> will be skipped if no BSD2 is available.
>
> Decoupling the checks will allow running the BSD1 specific tests
> on these platforms.
>
> Signed-off-by: Gabriel Feceoru <gabriel.feceoru@xxxxxxxxx>
Applied, thanks.
-Daniel
> ---
> lib/ioctl_wrappers.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
> index f371bf0..dfabde7 100644
> --- a/lib/ioctl_wrappers.c
> +++ b/lib/ioctl_wrappers.c
> @@ -1348,8 +1348,10 @@ void gem_require_ring(int fd, int ring_id)
>
> /* silly ABI, the kernel thinks everyone who has BSD also has BSD2 */
> if ((ring_id & ~(3<<13)) == I915_EXEC_BSD) {
> - if (ring_id & (3 << 13))
> + if (ring_id & (2 << 13))
> igt_require(gem_has_bsd2(fd));
> + if (ring_id & (1 << 13))
> + igt_require(gem_has_bsd(fd))
> }
> }
>
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
