<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 19, 2019 at 8:52 PM Luben Tuikov <<a href="mailto:luben.tuikov@amd.com" target="_blank">luben.tuikov@amd.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2019-11-14 10:34 p.m., Aaron Liu wrote:<br>
> From: Huang Rui <<a href="mailto:ray.huang@amd.com" target="_blank">ray.huang@amd.com</a>><br>
> <br>
> To align the kernel uapi change from Alex:<br>
> <br>
> "Add a flag to the GEM_CREATE ioctl to create encrypted buffers. Buffers with<br>
> this flag set will be created with the TMZ bit set in the PTEs or engines<br>
> accessing them. This is required in order to properly access the data from the<br>
> engines."<br>
> <br>
> We will use GEM_CREATE_ENCRYPTED flag for secure buffer allocation.<br>
> <br>
> Signed-off-by: Huang Rui <<a href="mailto:ray.huang@amd.com" target="_blank">ray.huang@amd.com</a>><br>
> Reviewed-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com" target="_blank">alexander.deucher@amd.com</a>><br>
> ---<br>
>  include/drm/amdgpu_drm.h | 5 +++++<br>
>  1 file changed, 5 insertions(+)<br>
> <br>
> diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h<br>
> index 5c28aa7..1a95e37 100644<br>
> --- a/include/drm/amdgpu_drm.h<br>
> +++ b/include/drm/amdgpu_drm.h<br>
> @@ -141,6 +141,11 @@ extern "C" {<br>
>   * releasing the memory<br>
>   */<br>
>  #define AMDGPU_GEM_CREATE_VRAM_WIPE_ON_RELEASE       (1 << 9)<br>
> +/* Flag that BO will be encrypted and that the TMZ bit should be<br>
> + * set in the PTEs when mapping this buffer via GPUVM or<br>
> + * accessing it with various hw blocks<br>
> + */<br>
> +#define AMDGPU_GEM_CREATE_ENCRYPTED          (1 << 10)<br>
<br>
Style!<br>
TAB char?!<br>
<br>
You have a TAB char between ".._ENCRYPTED" and "(1 << 10)"<br>
Do NOT add/insert TAB chars instead of space to align colunmns!<br>
If when you press Tab key a tab is inserted, as opposed to the line<br>
indented, then DO NOT use this editor.<br>
The Tab key should "indent according to mode" by inserting TAB chars.<br>
If the line is already indented, as this one is, then it should do nothing.<br></blockquote><div><br></div><div>I disagree with this 100%. Tabs or spaces don't matter here from my perspective. I also disagree with your language. It's overly impolite.<br></div><div><br></div><div>Marek</div></div></div>