linux-kernel - Re: [PATCH drm-misc-next 0/5] PowerVR VM fixes

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]

Message-ID: <d8c45d6c1b4ecbb182c4a16cb06def520450efa4.camel@imgtec.com>
Date:   Mon, 27 Nov 2023 10:53:54 +0000
From:   Frank Binns <Frank.Binns@...tec.com>
To:     "daniel@...ll.ch" <daniel@...ll.ch>,
        Donald Robson <Donald.Robson@...tec.com>,
        Matt Coster <Matt.Coster@...tec.com>,
        "mripard@...nel.org" <mripard@...nel.org>,
        "airlied@...il.com" <airlied@...il.com>,
        "dakr@...hat.com" <dakr@...hat.com>
CC:     "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH drm-misc-next 0/5] PowerVR VM fixes

Hi,

Thank you for the patches.

On Sat, 2023-11-25 at 00:36 +0100, Danilo Krummrich wrote:
> Hi,
> 
> Some major GPUVM changes landed just before v8 of the PowerVR series. Since v8
> went in rather quickly (review process was finished otherwise) I haven't had the
> chance to review the subsequent code changes.
> 
> Hence, this series with a few fixes in this context. Plus a minor GPUVM patch to
> make the drm_gpuvm_prepare_* helpers useful for PowerVR.
> 
> - Danilo
> 
> 
> Danilo Krummrich (5):
>   drm/imagination: vm: prevent duplicate drm_gpuvm_bo instances
>   drm/imagination: vm: check for drm_gpuvm_range_valid()
>   drm/imagination: vm: fix drm_gpuvm reference count
>   drm/gpuvm: fall back to drm_exec_lock_obj()
>   drm/imagination: vm: make use of GPUVM's drm_exec helper
> 
>  drivers/gpu/drm/drm_gpuvm.c          | 36 ++++++++++++++++++++--
>  drivers/gpu/drm/imagination/pvr_vm.c | 46 +++++++++++++++-------------
>  drivers/gpu/drm/imagination/pvr_vm.h |  3 +-
>  include/drm/drm_gpuvm.h              | 23 ++------------
>  4 files changed, 63 insertions(+), 45 deletions(-)
> 
> 
> base-commit: 46990918f35c1bf6e367cf8e0423e7344fec9fcb

For the series:

Tested-by: Frank Binns <frank.binns@...tec.com>

I'll leave it to Donald to do the review.

Thanks
Frank