Noralf Trønnes <nor...@tronnes.org> writes: > (cc affected parties) > > > Den 18.08.2017 09.46, skrev Daniel Vetter: >> On Thu, Aug 17, 2017 at 06:21:30PM +0200, Noralf Trønnes wrote: >>> Reject mapping an imported dma-buf since is's an invalid use-case. >>> >>> Cc: Philipp Zabel <p.za...@pengutronix.de> >>> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> >>> Cc: Sean Paul <seanp...@chromium.org> >>> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> >>> Signed-off-by: Noralf Trønnes <nor...@tronnes.org> >> I think acks from someone using mali would be good too. amdgpu already has >> such checks, so I think on the desktop side we're ok. >> >> Acked-by: Daniel Vetter <daniel.vet...@ffwll.ch> >> >> But I think this one here definitely needs a few more acks. I could break >> uabi if we're unlucky, so let's not rush it. > > Ok, I've CC'ed the affected parties to increase the odds that they look > at this. These are the drivers using drm_gem_dumb_map_offset() > (hopefully I got the list right):
If I understand the affected path right, this would break the PL111+VC4 combination: PL111 makes (dumb) buffers for scanout, and VC4 imports them and uses them for rendering. A vc4 glReadPixels of the window system buffer would map it and fail.
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
