[PATCH 2/2] Revert "drm/panfrost: Use drm_gem_map_offset()"
Daniel Vetter
daniel at ffwll.ch
Wed Aug 7 20:59:52 UTC 2019
On Wed, Aug 07, 2019 at 04:59:51PM +0100, Emil Velikov wrote:
> On Wed, 7 Aug 2019 at 15:53, Sean Paul <sean at poorly.run> wrote:
> >
> > From: Rob Herring <robh at kernel.org>
> >
> > This reverts commit 583bbf46133c726bae277e8f4e32bfba2a528c7f.
> >
> > Turns out we need mmap to work on imported BOs even if the current code
> > is buggy.
> >
> Personally I would have mentioned a use case where imported BOs are used.
>
> > Signed-off-by: Rob Herring <robh at kernel.org>
> > Signed-off-by: Sean Paul <sean at poorly.run>
>
> Regardless of the above nitpick, with the patch order fixed the series is:
> Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
>
> ... in case you haven't picked it already.
Yeah a follow-up patch to add a comment here about why exactly this went
all kaboom, plus which userspace (since panfrost is moving fast) would be
real nice here.
Atm we need to hope someone does a git blame on this before the break this
again, which seems a bit hopeful ...
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
More information about the dri-devel
mailing list