linux-kernel - Re: [PATCH v2 00/12] drm: add check if io_mem

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 25 Dec 2017 09:18:10 +0100
From:   Christian König <christian.koenig@....com>
To:     Tan Xiaojun <tanxiaojun@...wei.com>, seanpaul@...omium.org,
        airlied@...ux.ie, michel.daenzer@....com,
        alexander.deucher@....com, nicolai.haehnle@....com,
        thomas.lendacky@....com, yamada.masahiro@...ionext.com,
        msrb@...e.com, Felix.Kuehling@....com, dave.jiang@...el.com,
        gregkh@...uxfoundation.org, michael.thayer@...cle.com,
        hdegoede@...hat.com
Cc:     daniel.vetter@...el.com, jani.nikula@...ux.intel.com,
        wangzhou1@...ilicon.com, dri-devel@...ts.freedesktop.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 00/12] drm: add check if io_mem_pfn is NULL and cleanup

Series is Reviewed-by: Christian König <christian.koenig@....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(-)
>