On 10/16/2017 02:43 AM, Christian König wrote:
Am 15.10.2017 um 22:11 schrieb Mark Thompson:
This is not in libva 2.0, so it shouldn't be enabled yet.

Signed-off-by: Mark Thompson <s...@jkqxz.net>
---
On 15/10/17 20:06, Leo Liu wrote:
On 2017-10-15 07:23 AM, Mark Thompson wrote:
This reverts commit c4ed39f85b1ebd062eaa51880fcc79cfbcb4e5c3.
Can we just leave the code there, and gated by "#if 0" for now? so it would be easier for testing purpose.
Sure; alternative patch enclosing. (I was mostly thinking that people might prefer not to have the disabled code appearing in releases, but if that's fine then this is indeed more convenient for testing.)

I agree with Leo that this is the better approach.

Feel free to add my Acked-by to the patch.

BTW: Mark if you don't have commit rights yet, feel free to request them. I and I think Leo as well will certainly support that.
Yep. Agreed.

So Mark please rebase and tag rb/ab for all your uncommitted patches, and then you could send them to me. I will commit them for you for now.

Leo


Regards,
Christian.

Thanks,

- Mark


  src/gallium/state_trackers/va/context.c | 2 +-
  src/gallium/state_trackers/va/surface.c | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/gallium/state_trackers/va/context.c b/src/gallium/state_trackers/va/context.c
index 1207499a78..78e1f19ab7 100644
--- a/src/gallium/state_trackers/va/context.c
+++ b/src/gallium/state_trackers/va/context.c
@@ -89,7 +89,7 @@ static struct VADriverVTable vtable =
     &vlVaQuerySurfaceAttributes,
     &vlVaAcquireBufferHandle,
     &vlVaReleaseBufferHandle,
-#if VA_CHECK_VERSION(1, 0, 0)
+#if 0
     &vlVaExportSurfaceHandle,
  #endif
  };
diff --git a/src/gallium/state_trackers/va/surface.c b/src/gallium/state_trackers/va/surface.c
index 76e562717a..ccdeabc3ad 100644
--- a/src/gallium/state_trackers/va/surface.c
+++ b/src/gallium/state_trackers/va/surface.c
@@ -903,7 +903,7 @@ vlVaQueryVideoProcPipelineCaps(VADriverContextP ctx, VAContextID context,
     return VA_STATUS_SUCCESS;
  }
  -#if VA_CHECK_VERSION(1, 0, 0)
+#if 0
  VAStatus
  vlVaExportSurfaceHandle(VADriverContextP ctx,
                          VASurfaceID surface_id,



_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to