Re:Re: [PATCH] drm/amdgpu: Reduce a lock scope in amdgpu_amdkfd_gpuvm_free_memory_of_gpu()

From: èåå
Date: Sun Apr 19 2020 - 21:05:14 EST




From: Markus Elfring <Markus.Elfring@xxxxxx>
Date: 2020-04-19 17:05:07
To: Bernard Zhao <bernard@xxxxxxxx>,amd-gfx@xxxxxxxxxxxxxxxxxxxxx,dri-devel@xxxxxxxxxxxxxxxxxxxxx,Alex Deucher <alexander.deucher@xxxxxxx>,"Christian KÃnig" <christian.koenig@xxxxxxx>,Chunming Zhou <David1.Zhou@xxxxxxx>,"Felix KÃhling" <Felix.Kuehling@xxxxxxx>
Cc: kernel@xxxxxxxx,linux-kernel@xxxxxxxxxxxxxxx,Daniel Vetter <daniel@xxxxxxxx>,David Airlie <airlied@xxxxxxxx>
Subject: Re: [PATCH] drm/amdgpu: Reduce a lock scope in amdgpu_amdkfd_gpuvm_free_memory_of_gpu()>> Maybe we could reduce the mutex_lock(&mem->lock)`s protected code area,
>> and noneed to protect pr_debug.
>
>I suggest to improve the commit message.
>Would you like to adjust the patch subject?
>
>Do you imagine that data synchronisation should evolve in other ways?
>
>Regards,
>Markus

Sure, I will modify the code and adjust this patch subject. I will submit it again.