On 2018å¹´08æ??07æ?¥ 15:26, Junwei Zhang wrote: > When create bo from user memory, add it to handle table > for future query. > > Signed-off-by: Junwei Zhang <Jerry.Zhang at amd.com> > --- > amdgpu/amdgpu_bo.c | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c > index 422c7c9..b24e698 100644 > --- a/amdgpu/amdgpu_bo.c > +++ b/amdgpu/amdgpu_bo.c > @@ -556,7 +556,16 @@ int amdgpu_create_bo_from_user_mem(amdgpu_device_handle dev, > bo->alloc_size = size; > bo->handle = args.handle; > > - *buf_handle = bo; > + pthread_mutex_lock(&bo->dev->bo_table_mutex); > + r = handle_table_insert(&bo->dev->bo_handles, bo->handle, bo); > + pthread_mutex_unlock(&bo->dev->bo_table_mutex); > + > + pthread_mutex_init(&bo->cpu_access_mutex, NULL); This line is nothing with patch itself, please separate from it. Regards, David Zhou > + > + if (r) > + amdgpu_bo_free(bo); > + else > + *buf_handle = bo; > > return r; > }