Re: [PATCH] drm/amdgpu: move CS secure flag next the structs where it's used

[Christian König]

Am 27.11.19 um 21:56 schrieb Alex Deucher:

So it's not mixed up with the CTX stuff.

Signed-off-by: Alex Deucher 


Reviewed-by: Christian König 


---
  include/uapi/drm/amdgpu_drm.h | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/uapi/drm/amdgpu_drm.h b/include/uapi/drm/amdgpu_drm.h
index f75c6957064d..918ac3548cd3 100644
--- a/include/uapi/drm/amdgpu_drm.h
+++ b/include/uapi/drm/amdgpu_drm.h
@@ -207,9 +207,6 @@ union drm_amdgpu_bo_list {
  #define AMDGPU_CTX_OP_QUERY_STATE 3
  #define AMDGPU_CTX_OP_QUERY_STATE24
  
-/* Flag the command submission as secure */

-#define AMDGPU_CS_FLAGS_SECURE  (1 << 0)
-
  /* GPU reset status */
  #define AMDGPU_CTX_NO_RESET   0
  /* this the context caused it */
@@ -559,6 +556,9 @@ struct drm_amdgpu_cs_chunk {
__u64   chunk_data;
  };
  
+/* Flag the command submission as secure */

+#define AMDGPU_CS_FLAGS_SECURE  (1 << 0)
+
  struct drm_amdgpu_cs_in {
/** Rendering context id */
__u32   ctx_id;


___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

RE: [PATCH] drm/amdgpu: move CS secure flag next the structs where it's used

[Liu, Zhan]

> -Original Message-
> From: amd-gfx  On Behalf Of Alex
> Deucher
> Sent: 2019/November/27, Wednesday 3:57 PM
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander 
> Subject: [PATCH] drm/amdgpu: move CS secure flag next the structs where it's
> used
> 
> So it's not mixed up with the CTX stuff.
> 
> Signed-off-by: Alex Deucher 

Reviewed-by: Zhan Liu 

> ---
>  include/uapi/drm/amdgpu_drm.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/include/uapi/drm/amdgpu_drm.h
> b/include/uapi/drm/amdgpu_drm.h index f75c6957064d..918ac3548cd3
> 100644
> --- a/include/uapi/drm/amdgpu_drm.h
> +++ b/include/uapi/drm/amdgpu_drm.h
> @@ -207,9 +207,6 @@ union drm_amdgpu_bo_list {
>  #define AMDGPU_CTX_OP_QUERY_STATE3
>  #define AMDGPU_CTX_OP_QUERY_STATE2   4
> 
> -/* Flag the command submission as secure */
> -#define AMDGPU_CS_FLAGS_SECURE  (1 << 0)
> -
>  /* GPU reset status */
>  #define AMDGPU_CTX_NO_RESET  0
>  /* this the context caused it */
> @@ -559,6 +556,9 @@ struct drm_amdgpu_cs_chunk {
>   __u64   chunk_data;
>  };
> 
> +/* Flag the command submission as secure */
> +#define AMDGPU_CS_FLAGS_SECURE  (1 << 0)
> +
>  struct drm_amdgpu_cs_in {
>   /** Rendering context id */
>   __u32   ctx_id;
> --
> 2.23.0
> 
> ___
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.fre
> edesktop.org%2Fmailman%2Flistinfo%2Famd-
> gfxdata=02%7C01%7Czhan.liu%40amd.com%7C8c7b0eddf0c34548540
> a08d7737c57b2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C6
> 37104850267345545sdata=3Q3%2BLLpiH7ekvDFgp1zmrpGwvXQj1RD
> 2aKe6yap56Uw%3Dreserved=0
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx