Re: [PATCH v2 00/12] drm: add check if io_mem_pfn is NULL and cleanup

2017-12-25 Thread Tan Xiaojun
On 2017/12/25 16:18, Christian König wrote:
> Series is Reviewed-by: Christian König .
> 
> I'm going to pick that up for 4.16.
> 
> Thanks for the cleanup,
> Christian.
> 

Thank you very much.
Xiaojun.

> Am 25.12.2017 um 04:43 schrieb Tan Xiaojun:
>> I found an OOPS when I used the mainline kernel for graphical tests in 
>> Hisilicon
>> D05, I do not know how to solve this problem until I saw your discussion on 
>> this
>> issue a month ago:
>>
>> https://lists.freedesktop.org/archives/dri-devel/2017-November/159046.html
>>
>> And my problem can be solved perfectly by your solution.
>>
>> This is important for me, I want to solve this problem as soon as possible. 
>> So
>> I follow the result of your discussion, make and send these patches below.
>>
>> If anything is not good, please point it out, thanks.
>>
>> Change logs of v2:
>>   * add new function to instead of ttm_bo_default_io_mem_pfn() and
>> do some cleanup.
>>
>> Tan Xiaojun (12):
>>drm/ttm: add ttm_bo_io_mem_pfn to check io_mem_pfn
>>drm/ast: remove the default io_mem_pfn set
>>drm/bochs: remove the default io_mem_pfn set
>>drm/cirrus: remove the default io_mem_pfn set
>>drm/mgag200: remove the default io_mem_pfn set
>>drm/nouveau: remove the default io_mem_pfn set
>>drm/qxl: remove the default io_mem_pfn set
>>drm/radeon: remove the default io_mem_pfn set
>>drm/virtio: remove the default io_mem_pfn set
>>drm/vmwgfx: remove the default io_mem_pfn set
>>staging: remove the default io_mem_pfn set
>>drm/ttm: remove ttm_bo_default_io_mem_pfn
>>
>>   drivers/gpu/drm/ast/ast_ttm.c  |  1 -
>>   drivers/gpu/drm/bochs/bochs_mm.c   |  1 -
>>   drivers/gpu/drm/cirrus/cirrus_ttm.c|  1 -
>>   drivers/gpu/drm/mgag200/mgag200_ttm.c  |  1 -
>>   drivers/gpu/drm/nouveau/nouveau_bo.c   |  1 -
>>   drivers/gpu/drm/qxl/qxl_ttm.c  |  1 -
>>   drivers/gpu/drm/radeon/radeon_ttm.c|  1 -
>>   drivers/gpu/drm/ttm/ttm_bo_vm.c| 22 +-
>>   drivers/gpu/drm/virtio/virtgpu_ttm.c   |  1 -
>>   drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |  1 -
>>   drivers/staging/vboxvideo/vbox_ttm.c   |  1 -
>>   include/drm/ttm/ttm_bo_api.h   | 11 ---
>>   12 files changed, 13 insertions(+), 30 deletions(-)
>>
> 
> 
> 




Re: [PATCH v2 00/12] drm: add check if io_mem_pfn is NULL and cleanup

2017-12-25 Thread Christian König

Series is Reviewed-by: Christian König .

I'm going to pick that up for 4.16.

Thanks for the cleanup,
Christian.

Am 25.12.2017 um 04:43 schrieb Tan Xiaojun:

I found an OOPS when I used the mainline kernel for graphical tests in Hisilicon
D05, I do not know how to solve this problem until I saw your discussion on this
issue a month ago:

https://lists.freedesktop.org/archives/dri-devel/2017-November/159046.html

And my problem can be solved perfectly by your solution.

This is important for me, I want to solve this problem as soon as possible. So
I follow the result of your discussion, make and send these patches below.

If anything is not good, please point it out, thanks.

Change logs of v2:
  * add new function to instead of ttm_bo_default_io_mem_pfn() and
do some cleanup.

Tan Xiaojun (12):
   drm/ttm: add ttm_bo_io_mem_pfn to check io_mem_pfn
   drm/ast: remove the default io_mem_pfn set
   drm/bochs: remove the default io_mem_pfn set
   drm/cirrus: remove the default io_mem_pfn set
   drm/mgag200: remove the default io_mem_pfn set
   drm/nouveau: remove the default io_mem_pfn set
   drm/qxl: remove the default io_mem_pfn set
   drm/radeon: remove the default io_mem_pfn set
   drm/virtio: remove the default io_mem_pfn set
   drm/vmwgfx: remove the default io_mem_pfn set
   staging: remove the default io_mem_pfn set
   drm/ttm: remove ttm_bo_default_io_mem_pfn

  drivers/gpu/drm/ast/ast_ttm.c  |  1 -
  drivers/gpu/drm/bochs/bochs_mm.c   |  1 -
  drivers/gpu/drm/cirrus/cirrus_ttm.c|  1 -
  drivers/gpu/drm/mgag200/mgag200_ttm.c  |  1 -
  drivers/gpu/drm/nouveau/nouveau_bo.c   |  1 -
  drivers/gpu/drm/qxl/qxl_ttm.c  |  1 -
  drivers/gpu/drm/radeon/radeon_ttm.c|  1 -
  drivers/gpu/drm/ttm/ttm_bo_vm.c| 22 +-
  drivers/gpu/drm/virtio/virtgpu_ttm.c   |  1 -
  drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |  1 -
  drivers/staging/vboxvideo/vbox_ttm.c   |  1 -
  include/drm/ttm/ttm_bo_api.h   | 11 ---
  12 files changed, 13 insertions(+), 30 deletions(-)





[PATCH v2 00/12] drm: add check if io_mem_pfn is NULL and cleanup

2017-12-24 Thread Tan Xiaojun
I found an OOPS when I used the mainline kernel for graphical tests in Hisilicon
D05, I do not know how to solve this problem until I saw your discussion on this
issue a month ago:

https://lists.freedesktop.org/archives/dri-devel/2017-November/159046.html

And my problem can be solved perfectly by your solution.

This is important for me, I want to solve this problem as soon as possible. So
I follow the result of your discussion, make and send these patches below.

If anything is not good, please point it out, thanks.

Change logs of v2:
 * add new function to instead of ttm_bo_default_io_mem_pfn() and
   do some cleanup.

Tan Xiaojun (12):
  drm/ttm: add ttm_bo_io_mem_pfn to check io_mem_pfn
  drm/ast: remove the default io_mem_pfn set
  drm/bochs: remove the default io_mem_pfn set
  drm/cirrus: remove the default io_mem_pfn set
  drm/mgag200: remove the default io_mem_pfn set
  drm/nouveau: remove the default io_mem_pfn set
  drm/qxl: remove the default io_mem_pfn set
  drm/radeon: remove the default io_mem_pfn set
  drm/virtio: remove the default io_mem_pfn set
  drm/vmwgfx: remove the default io_mem_pfn set
  staging: remove the default io_mem_pfn set
  drm/ttm: remove ttm_bo_default_io_mem_pfn

 drivers/gpu/drm/ast/ast_ttm.c  |  1 -
 drivers/gpu/drm/bochs/bochs_mm.c   |  1 -
 drivers/gpu/drm/cirrus/cirrus_ttm.c|  1 -
 drivers/gpu/drm/mgag200/mgag200_ttm.c  |  1 -
 drivers/gpu/drm/nouveau/nouveau_bo.c   |  1 -
 drivers/gpu/drm/qxl/qxl_ttm.c  |  1 -
 drivers/gpu/drm/radeon/radeon_ttm.c|  1 -
 drivers/gpu/drm/ttm/ttm_bo_vm.c| 22 +-
 drivers/gpu/drm/virtio/virtgpu_ttm.c   |  1 -
 drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c |  1 -
 drivers/staging/vboxvideo/vbox_ttm.c   |  1 -
 include/drm/ttm/ttm_bo_api.h   | 11 ---
 12 files changed, 13 insertions(+), 30 deletions(-)

-- 
2.7.4