Subject: Re: [PATCH v2 00/12] drm: add check if io_mem_pfn is NULL and cleanup
To: Tan Xiaojun <tanxiaojun@huawei.com>, seanpaul@chromium.org,
        airlied@linux.ie, michel.daenzer@amd.com, alexander.deucher@amd.com,
        nicolai.haehnle@amd.com, thomas.lendacky@amd.com,
        yamada.masahiro@socionext.com, msrb@suse.com, Felix.Kuehling@amd.com,
        dave.jiang@intel.com, gregkh@linuxfoundation.org,
        michael.thayer@oracle.com, hdegoede@redhat.com
Cc: daniel.vetter@intel.com, jani.nikula@linux.intel.com,
        wangzhou1@hisilicon.com, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org
References: <1514173414-28325-1-git-send-email-tanxiaojun@huawei.com>
From: =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
Message-ID: <97ceaf9e-3d04-89a6-062b-6c9170a501a9@amd.com>
Date: Mon, 25 Dec 2017 09:18:10 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.5.0
MIME-Version: 1.0
In-Reply-To: <1514173414-28325-1-git-send-email-tanxiaojun@huawei.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Dec 2017 08:18:27.0150 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 5dbd4465-8f9d-4935-416b-08d54b701522
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR12MB1298
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2164
Lines: 53

Series is Reviewed-by: Christian König <christian.koenig@amd.com>.

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