Re: [PATCH 1/3] i-g-t: check whether kernel has dual bsd ring

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

 



On Thu, Aug 7, 2014 at 12:49 AM, Zhipeng Gong <zhipeng.gong@xxxxxxxxx> wrote:
> Signed-off-by: Zhipeng Gong <zhipeng.gong@xxxxxxxxx>
> ---
>  lib/ioctl_wrappers.c | 16 ++++++++++++++++
>  lib/ioctl_wrappers.h |  1 +
>  2 files changed, 17 insertions(+)
>
> diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
> index c4e1080..03163a0 100644
> --- a/lib/ioctl_wrappers.c
> +++ b/lib/ioctl_wrappers.c
> @@ -754,6 +754,22 @@ bool gem_has_vebox(int fd)
>         return gem_has_enable_ring(fd,LOCAL_I915_PARAM_HAS_VEBOX);
>  }
>
> +#define LOCAL_I915_PARAM_HAS_BSD2 29
> +/**
> + * gem_has_bsd2:
> + * @fd: open i915 drm file descriptor
> + *
> + * Feature test macro to query whether the BSD2 ring is available. This is simply
> + * a specific version of gem_has_enable_ring() for the BSD2 ring.
> + *
> + * Note that recent Bspec calls this the VCS ring for Video Command Submission.
> + *
> + * Returns: Whether the BSD ring is avaible or not.
> + */
> +bool gem_has_bsd2(int fd)
> +{
> +       return gem_has_enable_ring(fd,I915_PARAM_HAS_BSD2);

You add a LOCAL_ but execute without it. Compilation fails without
libdrm patches I just sent:
http://lists.freedesktop.org/archives/intel-gfx/2014-December/056568.html
http://lists.freedesktop.org/archives/intel-gfx/2014-December/056569.html

So, or use the local or remove it considering that it is on libdrm.
Thomas, how do you prefer it?

With that fixed feel free to use:
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
Tested-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

> +}
>  /**
>   * gem_available_aperture_size:
>   * @fd: open i915 drm file descriptor
> diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
> index 310d82e..2979634 100644
> --- a/lib/ioctl_wrappers.h
> +++ b/lib/ioctl_wrappers.h
> @@ -83,6 +83,7 @@ bool gem_has_enable_ring(int fd,int param);
>  bool gem_has_bsd(int fd);
>  bool gem_has_blt(int fd);
>  bool gem_has_vebox(int fd);
> +bool gem_has_bsd2(int fd);
>  bool gem_uses_aliasing_ppgtt(int fd);
>  int gem_available_fences(int fd);
>  uint64_t gem_available_aperture_size(int fd);
> --
> 2.0.3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://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