drm/amdgpu: extend AMDGPU_CTX_PRIORITY_NORMAL comment
authorEmil Velikov <emil.velikov@collabora.com>
Fri, 14 Jun 2019 17:33:35 +0000 (18:33 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 16 Jul 2019 18:08:30 +0000 (13:08 -0500)
Currently the AMDGPU_CTX_PRIORITY_* defines are used in both
drm_amdgpu_ctx_in::priority and drm_amdgpu_sched_in::priority.

Extend the comment to mention the CAP_SYS_NICE or DRM_MASTER requirement
is only applicable with the former.

Cc: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Cc: Christian König <christian.koenig@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
include/uapi/drm/amdgpu_drm.h

index d799858b9e5345d046333a5a580dcf23eaa498c9..11cc573229627a918061ed548ca6af496f900b61 100644 (file)
@@ -219,7 +219,10 @@ union drm_amdgpu_bo_list {
 #define AMDGPU_CTX_PRIORITY_VERY_LOW    -1023
 #define AMDGPU_CTX_PRIORITY_LOW         -512
 #define AMDGPU_CTX_PRIORITY_NORMAL      0
-/* Selecting a priority above NORMAL requires CAP_SYS_NICE or DRM_MASTER */
+/*
+ * When used in struct drm_amdgpu_ctx_in, a priority above NORMAL requires
+ * CAP_SYS_NICE or DRM_MASTER
+*/
 #define AMDGPU_CTX_PRIORITY_HIGH        512
 #define AMDGPU_CTX_PRIORITY_VERY_HIGH   1023
 
@@ -229,6 +232,7 @@ struct drm_amdgpu_ctx_in {
        /** For future use, no flags defined so far */
        __u32   flags;
        __u32   ctx_id;
+       /** AMDGPU_CTX_PRIORITY_* */
        __s32   priority;
 };
 
@@ -281,6 +285,7 @@ struct drm_amdgpu_sched_in {
        /* AMDGPU_SCHED_OP_* */
        __u32   op;
        __u32   fd;
+       /** AMDGPU_CTX_PRIORITY_* */
        __s32   priority;
        __u32   ctx_id;
 };