[PATCH] intel: add prime interface for getting/setting a prime bo. (v4)

2012-07-19 Thread Dave Airlie
This adds interfaces for the X driver to use to create a
prime handle from a buffer, and create a bo from a handle.

v2: use Chris's suggested naming (well from at least for consistency)
v3: git commit --amend fail
v4: fix as per Chris's suggestions, group assignments, add get tiling

Signed-off-by: Dave Airlie 
---
 intel/intel_bufmgr.h |4 +++
 intel/intel_bufmgr_gem.c |   63 ++
 2 files changed, 67 insertions(+)

diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
index 9b3a483..2167e43 100644
--- a/intel/intel_bufmgr.h
+++ b/intel/intel_bufmgr.h
@@ -192,6 +192,10 @@ void drm_intel_gem_context_destroy(drm_intel_context *ctx);
 int drm_intel_gem_bo_context_exec(drm_intel_bo *bo, drm_intel_context *ctx,
  int used, unsigned int flags);

+int drm_intel_bo_gem_export_to_prime(drm_intel_bo *bo, int *prime_fd);
+drm_intel_bo *drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr,
+   int prime_fd, int size);
+
 /* drm_intel_bufmgr_fake.c */
 drm_intel_bufmgr *drm_intel_bufmgr_fake_init(int fd,
 unsigned long low_offset,
diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
index 12a3197..eae2199 100644
--- a/intel/intel_bufmgr_gem.c
+++ b/intel/intel_bufmgr_gem.c
@@ -2413,6 +2413,69 @@ drm_intel_gem_bo_get_tiling(drm_intel_bo *bo, uint32_t * 
tiling_mode,
return 0;
 }

+drm_intel_bo *
+drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int 
size)
+{
+   drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
+   int ret;
+   uint32_t handle;
+   drm_intel_bo_gem *bo_gem;
+   struct drm_i915_gem_get_tiling get_tiling;
+
+   ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
+   if (ret) {
+ fprintf(stderr,"ret is %d %d\n", ret, errno);
+   return NULL;
+   }
+
+   bo_gem = calloc(1, sizeof(*bo_gem));
+   if (!bo_gem)
+   return NULL;
+
+   bo_gem->bo.size = size;
+   bo_gem->bo.handle = handle;
+   bo_gem->bo.bufmgr = bufmgr;
+
+   bo_gem->gem_handle = handle;
+
+   atomic_set(&bo_gem->refcount, 1);
+
+   bo_gem->name = "prime";
+   bo_gem->validate_index = -1;
+   bo_gem->reloc_tree_fences = 0;
+   bo_gem->used_as_reloc_target = false;
+   bo_gem->has_error = false;
+   bo_gem->reusable = false;
+
+   DRMINITLISTHEAD(&bo_gem->name_list);
+   DRMINITLISTHEAD(&bo_gem->vma_list);
+
+   VG_CLEAR(get_tiling);
+   get_tiling.handle = bo_gem->gem_handle;
+   ret = drmIoctl(bufmgr_gem->fd,
+  DRM_IOCTL_I915_GEM_GET_TILING,
+  &get_tiling);
+   if (ret != 0) {
+   drm_intel_gem_bo_unreference(&bo_gem->bo);
+   return NULL;
+   }
+   bo_gem->tiling_mode = get_tiling.tiling_mode;
+   bo_gem->swizzle_mode = get_tiling.swizzle_mode;
+   /* XXX stride is unknown */
+   drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+
+   return &bo_gem->bo;
+}
+
+int
+drm_intel_bo_gem_export_to_prime(drm_intel_bo *bo, int *prime_fd)
+{
+   drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
+   drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
+
+   return drmPrimeHandleToFD(bufmgr_gem->fd, bo_gem->gem_handle, 
DRM_CLOEXEC, prime_fd);
+}
+
 static int
 drm_intel_gem_bo_flink(drm_intel_bo *bo, uint32_t * name)
 {
-- 
1.7.10.2



[PATCH] intel: add prime interface for getting/setting a prime bo. (v4)

2012-07-19 Thread Chris Wilson
On Thu, 19 Jul 2012 13:09:49 +1000, Dave Airlie  wrote:
> This adds interfaces for the X driver to use to create a
> prime handle from a buffer, and create a bo from a handle.
> 
> v2: use Chris's suggested naming (well from at least for consistency)
> v3: git commit --amend fail
> v4: fix as per Chris's suggestions, group assignments, add get tiling

The next person to add another create routine gets to refactor the
common initialisation functions. :)

Reviewed-by: Chris Wilson 
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


Re: [PATCH] intel: add prime interface for getting/setting a prime bo. (v4)

2012-07-19 Thread Chris Wilson
On Thu, 19 Jul 2012 13:09:49 +1000, Dave Airlie  wrote:
> This adds interfaces for the X driver to use to create a
> prime handle from a buffer, and create a bo from a handle.
> 
> v2: use Chris's suggested naming (well from at least for consistency)
> v3: git commit --amend fail
> v4: fix as per Chris's suggestions, group assignments, add get tiling

The next person to add another create routine gets to refactor the
common initialisation functions. :)

Reviewed-by: Chris Wilson 
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH] intel: add prime interface for getting/setting a prime bo. (v4)

2012-07-18 Thread Dave Airlie
This adds interfaces for the X driver to use to create a
prime handle from a buffer, and create a bo from a handle.

v2: use Chris's suggested naming (well from at least for consistency)
v3: git commit --amend fail
v4: fix as per Chris's suggestions, group assignments, add get tiling

Signed-off-by: Dave Airlie 
---
 intel/intel_bufmgr.h |4 +++
 intel/intel_bufmgr_gem.c |   63 ++
 2 files changed, 67 insertions(+)

diff --git a/intel/intel_bufmgr.h b/intel/intel_bufmgr.h
index 9b3a483..2167e43 100644
--- a/intel/intel_bufmgr.h
+++ b/intel/intel_bufmgr.h
@@ -192,6 +192,10 @@ void drm_intel_gem_context_destroy(drm_intel_context *ctx);
 int drm_intel_gem_bo_context_exec(drm_intel_bo *bo, drm_intel_context *ctx,
  int used, unsigned int flags);
 
+int drm_intel_bo_gem_export_to_prime(drm_intel_bo *bo, int *prime_fd);
+drm_intel_bo *drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr,
+   int prime_fd, int size);
+
 /* drm_intel_bufmgr_fake.c */
 drm_intel_bufmgr *drm_intel_bufmgr_fake_init(int fd,
 unsigned long low_offset,
diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
index 12a3197..eae2199 100644
--- a/intel/intel_bufmgr_gem.c
+++ b/intel/intel_bufmgr_gem.c
@@ -2413,6 +2413,69 @@ drm_intel_gem_bo_get_tiling(drm_intel_bo *bo, uint32_t * 
tiling_mode,
return 0;
 }
 
+drm_intel_bo *
+drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int 
size)
+{
+   drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bufmgr;
+   int ret;
+   uint32_t handle;
+   drm_intel_bo_gem *bo_gem;
+   struct drm_i915_gem_get_tiling get_tiling;
+
+   ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
+   if (ret) {
+ fprintf(stderr,"ret is %d %d\n", ret, errno);
+   return NULL;
+   }
+
+   bo_gem = calloc(1, sizeof(*bo_gem));
+   if (!bo_gem)
+   return NULL;
+
+   bo_gem->bo.size = size;
+   bo_gem->bo.handle = handle;
+   bo_gem->bo.bufmgr = bufmgr;
+
+   bo_gem->gem_handle = handle;
+
+   atomic_set(&bo_gem->refcount, 1);
+
+   bo_gem->name = "prime";
+   bo_gem->validate_index = -1;
+   bo_gem->reloc_tree_fences = 0;
+   bo_gem->used_as_reloc_target = false;
+   bo_gem->has_error = false;
+   bo_gem->reusable = false;
+
+   DRMINITLISTHEAD(&bo_gem->name_list);
+   DRMINITLISTHEAD(&bo_gem->vma_list);
+
+   VG_CLEAR(get_tiling);
+   get_tiling.handle = bo_gem->gem_handle;
+   ret = drmIoctl(bufmgr_gem->fd,
+  DRM_IOCTL_I915_GEM_GET_TILING,
+  &get_tiling);
+   if (ret != 0) {
+   drm_intel_gem_bo_unreference(&bo_gem->bo);
+   return NULL;
+   }
+   bo_gem->tiling_mode = get_tiling.tiling_mode;
+   bo_gem->swizzle_mode = get_tiling.swizzle_mode;
+   /* XXX stride is unknown */
+   drm_intel_bo_gem_set_in_aperture_size(bufmgr_gem, bo_gem);
+
+   return &bo_gem->bo;
+}
+
+int
+drm_intel_bo_gem_export_to_prime(drm_intel_bo *bo, int *prime_fd)
+{
+   drm_intel_bufmgr_gem *bufmgr_gem = (drm_intel_bufmgr_gem *) bo->bufmgr;
+   drm_intel_bo_gem *bo_gem = (drm_intel_bo_gem *) bo;
+
+   return drmPrimeHandleToFD(bufmgr_gem->fd, bo_gem->gem_handle, 
DRM_CLOEXEC, prime_fd);
+}
+
 static int
 drm_intel_gem_bo_flink(drm_intel_bo *bo, uint32_t * name)
 {
-- 
1.7.10.2

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel