* [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
@ 2020-01-20 16:43 Imre Deak
2020-01-21 18:52 ` Emil Velikov
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Imre Deak @ 2020-01-20 16:43 UTC (permalink / raw)
To: intel-gfx, dri-devel
Platforms without a HW detiler doesn't support the get_tiling IOCTL.
Fix the drm_intel_bo_gem_create_from_* functions assuming the default
no-tiling, no-swizzling setting for the GEM buffer in this case.
Signed-off-by: Imre Deak <imre.deak@intel.com>
---
intel/intel_bufmgr_gem.c | 42 +++++++++++++++++++++++++---------------
1 file changed, 26 insertions(+), 16 deletions(-)
diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
index fbf48730..fc249ef1 100644
--- a/intel/intel_bufmgr_gem.c
+++ b/intel/intel_bufmgr_gem.c
@@ -1069,6 +1069,27 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
tiling_mode, stride, size, flags);
}
+static int get_tiling_mode(drm_intel_bufmgr_gem *bufmgr_gem,
+ uint32_t gem_handle,
+ uint32_t *tiling_mode,
+ uint32_t *swizzle_mode)
+{
+ struct drm_i915_gem_get_tiling get_tiling;
+ int ret;
+
+ memclear(get_tiling);
+ ret = drmIoctl(bufmgr_gem->fd,
+ DRM_IOCTL_I915_GEM_GET_TILING,
+ &get_tiling);
+ if (ret != 0 && errno != EOPNOTSUPP)
+ return ret;
+
+ *tiling_mode = get_tiling.tiling_mode;
+ *swizzle_mode = get_tiling.swizzle_mode;
+
+ return 0;
+}
+
/**
* Returns a drm_intel_bo wrapping the given buffer object handle.
*
@@ -1084,7 +1105,6 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
drm_intel_bo_gem *bo_gem;
int ret;
struct drm_gem_open open_arg;
- struct drm_i915_gem_get_tiling get_tiling;
/* At the moment most applications only have a few named bo.
* For instance, in a DRI client only the render buffers passed
@@ -1146,16 +1166,11 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
HASH_ADD(name_hh, bufmgr_gem->name_table,
global_name, sizeof(bo_gem->global_name), bo_gem);
- memclear(get_tiling);
- get_tiling.handle = bo_gem->gem_handle;
- ret = drmIoctl(bufmgr_gem->fd,
- DRM_IOCTL_I915_GEM_GET_TILING,
- &get_tiling);
+ ret = get_tiling_mode(bufmgr_gem, bo_gem->gem_handle,
+ &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
if (ret != 0)
goto err_unref;
- 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, 0);
DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
@@ -2634,7 +2649,6 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
int ret;
uint32_t handle;
drm_intel_bo_gem *bo_gem;
- struct drm_i915_gem_get_tiling get_tiling;
pthread_mutex_lock(&bufmgr_gem->lock);
ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
@@ -2688,15 +2702,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
bo_gem->has_error = false;
bo_gem->reusable = false;
- memclear(get_tiling);
- get_tiling.handle = bo_gem->gem_handle;
- if (drmIoctl(bufmgr_gem->fd,
- DRM_IOCTL_I915_GEM_GET_TILING,
- &get_tiling))
+ ret = get_tiling_mode(bufmgr_gem, handle,
+ &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
+ if (ret)
goto err;
- 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, 0);
--
2.23.1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
2020-01-20 16:43 [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling Imre Deak
@ 2020-01-21 18:52 ` Emil Velikov
2020-01-21 22:40 ` Eric Engestrom
2020-01-22 9:31 ` [PATCH libdrm v2] " Imre Deak
2 siblings, 0 replies; 6+ messages in thread
From: Emil Velikov @ 2020-01-21 18:52 UTC (permalink / raw)
To: Imre Deak; +Cc: Intel Graphics Development, ML dri-devel
On Mon, 20 Jan 2020 at 16:43, Imre Deak <imre.deak@intel.com> wrote:
>
> Platforms without a HW detiler doesn't support the get_tiling IOCTL.
> Fix the drm_intel_bo_gem_create_from_* functions assuming the default
> no-tiling, no-swizzling setting for the GEM buffer in this case.
>
> Signed-off-by: Imre Deak <imre.deak@intel.com>
Seems like it this can happen only in vgpu cases.... which explains
why it was not raised earlier. Regardless:
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Aside: might want to do similar patch for mesa. Be that for
classic/i965, gallium/iris and/or the Vulkan ANV driver.
-Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
2020-01-20 16:43 [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling Imre Deak
2020-01-21 18:52 ` Emil Velikov
@ 2020-01-21 22:40 ` Eric Engestrom
2020-01-21 23:38 ` Imre Deak
2020-01-22 9:31 ` [PATCH libdrm v2] " Imre Deak
2 siblings, 1 reply; 6+ messages in thread
From: Eric Engestrom @ 2020-01-21 22:40 UTC (permalink / raw)
To: Imre Deak; +Cc: intel-gfx, dri-devel
On Monday, 2020-01-20 18:43:43 +0200, Imre Deak wrote:
> Platforms without a HW detiler doesn't support the get_tiling IOCTL.
> Fix the drm_intel_bo_gem_create_from_* functions assuming the default
> no-tiling, no-swizzling setting for the GEM buffer in this case.
>
> Signed-off-by: Imre Deak <imre.deak@intel.com>
> ---
> intel/intel_bufmgr_gem.c | 42 +++++++++++++++++++++++++---------------
> 1 file changed, 26 insertions(+), 16 deletions(-)
>
> diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> index fbf48730..fc249ef1 100644
> --- a/intel/intel_bufmgr_gem.c
> +++ b/intel/intel_bufmgr_gem.c
> @@ -1069,6 +1069,27 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
> tiling_mode, stride, size, flags);
> }
>
> +static int get_tiling_mode(drm_intel_bufmgr_gem *bufmgr_gem,
> + uint32_t gem_handle,
> + uint32_t *tiling_mode,
> + uint32_t *swizzle_mode)
> +{
> + struct drm_i915_gem_get_tiling get_tiling;
> + int ret;
> +
> + memclear(get_tiling);
> + ret = drmIoctl(bufmgr_gem->fd,
> + DRM_IOCTL_I915_GEM_GET_TILING,
> + &get_tiling);
You're missing `get_tiling.handle = gem_handle;`
Or better yet, just initialise `get_tiling` and get rid of the memclear():
struct drm_i915_gem_get_tiling get_tiling = {
.handle = gem_handle,
};
With either fix:
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
FYI, I've posted the following MR for the equivalent Mesa changes:
https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3497
> + if (ret != 0 && errno != EOPNOTSUPP)
> + return ret;
> +
> + *tiling_mode = get_tiling.tiling_mode;
> + *swizzle_mode = get_tiling.swizzle_mode;
> +
> + return 0;
> +}
> +
> /**
> * Returns a drm_intel_bo wrapping the given buffer object handle.
> *
> @@ -1084,7 +1105,6 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> drm_intel_bo_gem *bo_gem;
> int ret;
> struct drm_gem_open open_arg;
> - struct drm_i915_gem_get_tiling get_tiling;
>
> /* At the moment most applications only have a few named bo.
> * For instance, in a DRI client only the render buffers passed
> @@ -1146,16 +1166,11 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> HASH_ADD(name_hh, bufmgr_gem->name_table,
> global_name, sizeof(bo_gem->global_name), bo_gem);
>
> - memclear(get_tiling);
> - get_tiling.handle = bo_gem->gem_handle;
> - ret = drmIoctl(bufmgr_gem->fd,
> - DRM_IOCTL_I915_GEM_GET_TILING,
> - &get_tiling);
> + ret = get_tiling_mode(bufmgr_gem, bo_gem->gem_handle,
> + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> if (ret != 0)
> goto err_unref;
>
> - 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, 0);
> DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
> @@ -2634,7 +2649,6 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> int ret;
> uint32_t handle;
> drm_intel_bo_gem *bo_gem;
> - struct drm_i915_gem_get_tiling get_tiling;
>
> pthread_mutex_lock(&bufmgr_gem->lock);
> ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
> @@ -2688,15 +2702,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> bo_gem->has_error = false;
> bo_gem->reusable = false;
>
> - memclear(get_tiling);
> - get_tiling.handle = bo_gem->gem_handle;
> - if (drmIoctl(bufmgr_gem->fd,
> - DRM_IOCTL_I915_GEM_GET_TILING,
> - &get_tiling))
> + ret = get_tiling_mode(bufmgr_gem, handle,
> + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> + if (ret)
> goto err;
>
> - 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, 0);
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
2020-01-21 22:40 ` Eric Engestrom
@ 2020-01-21 23:38 ` Imre Deak
0 siblings, 0 replies; 6+ messages in thread
From: Imre Deak @ 2020-01-21 23:38 UTC (permalink / raw)
To: Eric Engestrom; +Cc: intel-gfx, dri-devel
On Tue, Jan 21, 2020 at 10:40:48PM +0000, Eric Engestrom wrote:
> On Monday, 2020-01-20 18:43:43 +0200, Imre Deak wrote:
> > Platforms without a HW detiler doesn't support the get_tiling IOCTL.
> > Fix the drm_intel_bo_gem_create_from_* functions assuming the default
> > no-tiling, no-swizzling setting for the GEM buffer in this case.
> >
> > Signed-off-by: Imre Deak <imre.deak@intel.com>
> > ---
> > intel/intel_bufmgr_gem.c | 42 +++++++++++++++++++++++++---------------
> > 1 file changed, 26 insertions(+), 16 deletions(-)
> >
> > diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> > index fbf48730..fc249ef1 100644
> > --- a/intel/intel_bufmgr_gem.c
> > +++ b/intel/intel_bufmgr_gem.c
> > @@ -1069,6 +1069,27 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
> > tiling_mode, stride, size, flags);
> > }
> >
> > +static int get_tiling_mode(drm_intel_bufmgr_gem *bufmgr_gem,
> > + uint32_t gem_handle,
> > + uint32_t *tiling_mode,
> > + uint32_t *swizzle_mode)
> > +{
> > + struct drm_i915_gem_get_tiling get_tiling;
> > + int ret;
> > +
> > + memclear(get_tiling);
> > + ret = drmIoctl(bufmgr_gem->fd,
> > + DRM_IOCTL_I915_GEM_GET_TILING,
> > + &get_tiling);
>
> You're missing `get_tiling.handle = gem_handle;`
Err, thanks for catching that, will resend fixing it.
> Or better yet, just initialise `get_tiling` and get rid of the memclear():
> struct drm_i915_gem_get_tiling get_tiling = {
> .handle = gem_handle,
> };
>
> With either fix:
> Reviewed-by: Eric Engestrom <eric@engestrom.ch>
>
> FYI, I've posted the following MR for the equivalent Mesa changes:
> https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3497
>
> > + if (ret != 0 && errno != EOPNOTSUPP)
> > + return ret;
> > +
> > + *tiling_mode = get_tiling.tiling_mode;
> > + *swizzle_mode = get_tiling.swizzle_mode;
> > +
> > + return 0;
> > +}
> > +
> > /**
> > * Returns a drm_intel_bo wrapping the given buffer object handle.
> > *
> > @@ -1084,7 +1105,6 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> > drm_intel_bo_gem *bo_gem;
> > int ret;
> > struct drm_gem_open open_arg;
> > - struct drm_i915_gem_get_tiling get_tiling;
> >
> > /* At the moment most applications only have a few named bo.
> > * For instance, in a DRI client only the render buffers passed
> > @@ -1146,16 +1166,11 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> > HASH_ADD(name_hh, bufmgr_gem->name_table,
> > global_name, sizeof(bo_gem->global_name), bo_gem);
> >
> > - memclear(get_tiling);
> > - get_tiling.handle = bo_gem->gem_handle;
> > - ret = drmIoctl(bufmgr_gem->fd,
> > - DRM_IOCTL_I915_GEM_GET_TILING,
> > - &get_tiling);
> > + ret = get_tiling_mode(bufmgr_gem, bo_gem->gem_handle,
> > + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> > if (ret != 0)
> > goto err_unref;
> >
> > - 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, 0);
> > DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
> > @@ -2634,7 +2649,6 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> > int ret;
> > uint32_t handle;
> > drm_intel_bo_gem *bo_gem;
> > - struct drm_i915_gem_get_tiling get_tiling;
> >
> > pthread_mutex_lock(&bufmgr_gem->lock);
> > ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
> > @@ -2688,15 +2702,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> > bo_gem->has_error = false;
> > bo_gem->reusable = false;
> >
> > - memclear(get_tiling);
> > - get_tiling.handle = bo_gem->gem_handle;
> > - if (drmIoctl(bufmgr_gem->fd,
> > - DRM_IOCTL_I915_GEM_GET_TILING,
> > - &get_tiling))
> > + ret = get_tiling_mode(bufmgr_gem, handle,
> > + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> > + if (ret)
> > goto err;
> >
> > - 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, 0);
> >
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH libdrm v2] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
2020-01-20 16:43 [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling Imre Deak
2020-01-21 18:52 ` Emil Velikov
2020-01-21 22:40 ` Eric Engestrom
@ 2020-01-22 9:31 ` Imre Deak
2020-01-28 13:38 ` [Intel-gfx] " Imre Deak
2 siblings, 1 reply; 6+ messages in thread
From: Imre Deak @ 2020-01-22 9:31 UTC (permalink / raw)
To: intel-gfx, dri-devel; +Cc: Eric Engestrom, Emil Velikov
Platforms without a HW detiler doesn't support the get_tiling IOCTL.
Fix the drm_intel_bo_gem_create_from_* functions assuming the default
no-tiling, no-swizzling setting for the GEM buffer in this case.
v2:
- Add the missing gem handle IOCTL parameter. (Eric)
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
---
intel/intel_bufmgr_gem.c | 43 +++++++++++++++++++++++++---------------
1 file changed, 27 insertions(+), 16 deletions(-)
diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
index fbf48730..ade13a4c 100644
--- a/intel/intel_bufmgr_gem.c
+++ b/intel/intel_bufmgr_gem.c
@@ -1069,6 +1069,28 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
tiling_mode, stride, size, flags);
}
+static int get_tiling_mode(drm_intel_bufmgr_gem *bufmgr_gem,
+ uint32_t gem_handle,
+ uint32_t *tiling_mode,
+ uint32_t *swizzle_mode)
+{
+ struct drm_i915_gem_get_tiling get_tiling = {
+ .handle = gem_handle,
+ };
+ int ret;
+
+ ret = drmIoctl(bufmgr_gem->fd,
+ DRM_IOCTL_I915_GEM_GET_TILING,
+ &get_tiling);
+ if (ret != 0 && errno != EOPNOTSUPP)
+ return ret;
+
+ *tiling_mode = get_tiling.tiling_mode;
+ *swizzle_mode = get_tiling.swizzle_mode;
+
+ return 0;
+}
+
/**
* Returns a drm_intel_bo wrapping the given buffer object handle.
*
@@ -1084,7 +1106,6 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
drm_intel_bo_gem *bo_gem;
int ret;
struct drm_gem_open open_arg;
- struct drm_i915_gem_get_tiling get_tiling;
/* At the moment most applications only have a few named bo.
* For instance, in a DRI client only the render buffers passed
@@ -1146,16 +1167,11 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
HASH_ADD(name_hh, bufmgr_gem->name_table,
global_name, sizeof(bo_gem->global_name), bo_gem);
- memclear(get_tiling);
- get_tiling.handle = bo_gem->gem_handle;
- ret = drmIoctl(bufmgr_gem->fd,
- DRM_IOCTL_I915_GEM_GET_TILING,
- &get_tiling);
+ ret = get_tiling_mode(bufmgr_gem, bo_gem->gem_handle,
+ &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
if (ret != 0)
goto err_unref;
- 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, 0);
DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
@@ -2634,7 +2650,6 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
int ret;
uint32_t handle;
drm_intel_bo_gem *bo_gem;
- struct drm_i915_gem_get_tiling get_tiling;
pthread_mutex_lock(&bufmgr_gem->lock);
ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
@@ -2688,15 +2703,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
bo_gem->has_error = false;
bo_gem->reusable = false;
- memclear(get_tiling);
- get_tiling.handle = bo_gem->gem_handle;
- if (drmIoctl(bufmgr_gem->fd,
- DRM_IOCTL_I915_GEM_GET_TILING,
- &get_tiling))
+ ret = get_tiling_mode(bufmgr_gem, handle,
+ &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
+ if (ret)
goto err;
- 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, 0);
--
2.23.1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Intel-gfx] [PATCH libdrm v2] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling
2020-01-22 9:31 ` [PATCH libdrm v2] " Imre Deak
@ 2020-01-28 13:38 ` Imre Deak
0 siblings, 0 replies; 6+ messages in thread
From: Imre Deak @ 2020-01-28 13:38 UTC (permalink / raw)
To: intel-gfx, dri-devel, Eric Engestrom, Emil Velikov,
Juha-Pekka Heikkilä
On Wed, Jan 22, 2020 at 11:31:22AM +0200, Imre Deak wrote:
> Platforms without a HW detiler doesn't support the get_tiling IOCTL.
> Fix the drm_intel_bo_gem_create_from_* functions assuming the default
> no-tiling, no-swizzling setting for the GEM buffer in this case.
>
> v2:
> - Add the missing gem handle IOCTL parameter. (Eric)
>
> Signed-off-by: Imre Deak <imre.deak@intel.com>
> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
> Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Thanks for the reviewes, pushed to
gitlab.freedesktop.org:mesa/drm.git master
> ---
> intel/intel_bufmgr_gem.c | 43 +++++++++++++++++++++++++---------------
> 1 file changed, 27 insertions(+), 16 deletions(-)
>
> diff --git a/intel/intel_bufmgr_gem.c b/intel/intel_bufmgr_gem.c
> index fbf48730..ade13a4c 100644
> --- a/intel/intel_bufmgr_gem.c
> +++ b/intel/intel_bufmgr_gem.c
> @@ -1069,6 +1069,28 @@ check_bo_alloc_userptr(drm_intel_bufmgr *bufmgr,
> tiling_mode, stride, size, flags);
> }
>
> +static int get_tiling_mode(drm_intel_bufmgr_gem *bufmgr_gem,
> + uint32_t gem_handle,
> + uint32_t *tiling_mode,
> + uint32_t *swizzle_mode)
> +{
> + struct drm_i915_gem_get_tiling get_tiling = {
> + .handle = gem_handle,
> + };
> + int ret;
> +
> + ret = drmIoctl(bufmgr_gem->fd,
> + DRM_IOCTL_I915_GEM_GET_TILING,
> + &get_tiling);
> + if (ret != 0 && errno != EOPNOTSUPP)
> + return ret;
> +
> + *tiling_mode = get_tiling.tiling_mode;
> + *swizzle_mode = get_tiling.swizzle_mode;
> +
> + return 0;
> +}
> +
> /**
> * Returns a drm_intel_bo wrapping the given buffer object handle.
> *
> @@ -1084,7 +1106,6 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> drm_intel_bo_gem *bo_gem;
> int ret;
> struct drm_gem_open open_arg;
> - struct drm_i915_gem_get_tiling get_tiling;
>
> /* At the moment most applications only have a few named bo.
> * For instance, in a DRI client only the render buffers passed
> @@ -1146,16 +1167,11 @@ drm_intel_bo_gem_create_from_name(drm_intel_bufmgr *bufmgr,
> HASH_ADD(name_hh, bufmgr_gem->name_table,
> global_name, sizeof(bo_gem->global_name), bo_gem);
>
> - memclear(get_tiling);
> - get_tiling.handle = bo_gem->gem_handle;
> - ret = drmIoctl(bufmgr_gem->fd,
> - DRM_IOCTL_I915_GEM_GET_TILING,
> - &get_tiling);
> + ret = get_tiling_mode(bufmgr_gem, bo_gem->gem_handle,
> + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> if (ret != 0)
> goto err_unref;
>
> - 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, 0);
> DBG("bo_create_from_handle: %d (%s)\n", handle, bo_gem->name);
> @@ -2634,7 +2650,6 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> int ret;
> uint32_t handle;
> drm_intel_bo_gem *bo_gem;
> - struct drm_i915_gem_get_tiling get_tiling;
>
> pthread_mutex_lock(&bufmgr_gem->lock);
> ret = drmPrimeFDToHandle(bufmgr_gem->fd, prime_fd, &handle);
> @@ -2688,15 +2703,11 @@ drm_intel_bo_gem_create_from_prime(drm_intel_bufmgr *bufmgr, int prime_fd, int s
> bo_gem->has_error = false;
> bo_gem->reusable = false;
>
> - memclear(get_tiling);
> - get_tiling.handle = bo_gem->gem_handle;
> - if (drmIoctl(bufmgr_gem->fd,
> - DRM_IOCTL_I915_GEM_GET_TILING,
> - &get_tiling))
> + ret = get_tiling_mode(bufmgr_gem, handle,
> + &bo_gem->tiling_mode, &bo_gem->swizzle_mode);
> + if (ret)
> goto err;
>
> - 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, 0);
>
> --
> 2.23.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-01-28 13:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-20 16:43 [PATCH libdrm] intel: drm_intel_bo_gem_create_from_* on platforms w/o HW tiling Imre Deak
2020-01-21 18:52 ` Emil Velikov
2020-01-21 22:40 ` Eric Engestrom
2020-01-21 23:38 ` Imre Deak
2020-01-22 9:31 ` [PATCH libdrm v2] " Imre Deak
2020-01-28 13:38 ` [Intel-gfx] " Imre Deak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).