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

From: Christian KÃnig
Date: Mon Dec 25 2017 - 03:28:25 EST


Series is Reviewed-by: Christian KÃnig <christian.koenig@xxxxxxx>.

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(-)