Commit 92f0ad0b authored by Boris Brezillon's avatar Boris Brezillon Committed by Rob Herring

drm/panfrost: Add a module parameter to expose unstable ioctls

We plan to expose performance counters through 2 driver specific
ioctls until there's a solution to expose them in a generic way.
In order to be able to deprecate those ioctls when this new
infrastructure is in place we add an unsafe module parameter that
will keep those ioctls hidden unless it's set to true (which also
has the effect of tainting the kernel).

All unstable ioctl handlers should use panfrost_unstable_ioctl_check()
to check whether they're supposed to handle the request or reject it
with ENOSYS.
Suggested-by: default avatarEmil Velikov <emil.velikov@collabora.com>
Signed-off-by: default avatarBoris Brezillon <boris.brezillon@collabora.com>
Reviewed-by: default avatarAlyssa Rosenzweig <alyssa@rosenzweig.io>
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20190618081648.17297-3-boris.brezillon@collabora.com
parent dd082cef
...@@ -115,6 +115,8 @@ static inline bool panfrost_model_eq(struct panfrost_device *pfdev, s32 id) ...@@ -115,6 +115,8 @@ static inline bool panfrost_model_eq(struct panfrost_device *pfdev, s32 id)
return !panfrost_model_cmp(pfdev, id); return !panfrost_model_cmp(pfdev, id);
} }
int panfrost_unstable_ioctl_check(void);
int panfrost_device_init(struct panfrost_device *pfdev); int panfrost_device_init(struct panfrost_device *pfdev);
void panfrost_device_fini(struct panfrost_device *pfdev); void panfrost_device_fini(struct panfrost_device *pfdev);
......
...@@ -20,6 +20,9 @@ ...@@ -20,6 +20,9 @@
#include "panfrost_job.h" #include "panfrost_job.h"
#include "panfrost_gpu.h" #include "panfrost_gpu.h"
static bool unstable_ioctls;
module_param_unsafe(unstable_ioctls, bool, 0600);
static int panfrost_ioctl_get_param(struct drm_device *ddev, void *data, struct drm_file *file) static int panfrost_ioctl_get_param(struct drm_device *ddev, void *data, struct drm_file *file)
{ {
struct drm_panfrost_get_param *param = data; struct drm_panfrost_get_param *param = data;
...@@ -297,6 +300,14 @@ static int panfrost_ioctl_get_bo_offset(struct drm_device *dev, void *data, ...@@ -297,6 +300,14 @@ static int panfrost_ioctl_get_bo_offset(struct drm_device *dev, void *data,
return 0; return 0;
} }
int panfrost_unstable_ioctl_check(void)
{
if (!unstable_ioctls)
return -ENOSYS;
return 0;
}
static int static int
panfrost_open(struct drm_device *dev, struct drm_file *file) panfrost_open(struct drm_device *dev, struct drm_file *file)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment