From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Stuebner Subject: Re: [PATCH v3 03/43] drm/rockchip: Respect page offset for PRIME mmap calls Date: Sun, 18 Feb 2018 11:53:49 +0100 Message-ID: <3167776.tAZEmiy1TT@phil> References: <20180130202913.28724-1-thierry.escande@collabora.com> <20180130202913.28724-4-thierry.escande@collabora.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20180130202913.28724-4-thierry.escande@collabora.com> Sender: linux-kernel-owner@vger.kernel.org To: linux-rockchip@lists.infradead.org Cc: Thierry Escande , Archit Taneja , Inki Dae , Thierry Reding , Sandy Huang , Sean Paul , David Airlie , Tomasz Figa , Enric Balletbo i Serra , Zain Wang , Lin Huang , Douglas Anderson , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Yakir Yang , =?ISO-8859-1?Q?=D8rjan?= Eide , Mark Yao , Haixia Shi List-Id: linux-rockchip.vger.kernel.org Am Dienstag, 30. Januar 2018, 21:28:33 CET schrieb Thierry Escande: > From: Ørjan Eide > > When mapping external DMA-bufs through the PRIME mmap call, we might be > given an offset which has to be respected. However for the internal DRM > GEM mmap path, we have to ignore the fake mmap offset used to identify > the buffer only. Currently the code always zeroes out vma->vm_pgoff, > which breaks the former. > > This patch fixes the problem by moving the vm_pgoff assignment to a > function that is used only for GEM mmap path, so that the PRIME path > retains the original offset. > > Cc: Daniel Kurtz > Signed-off-by: Ørjan Eide > Signed-off-by: Tomasz Figa > Signed-off-by: Sean Paul > Signed-off-by: Thierry Escande > Tested-by: Heiko Stuebner applied to drm-misc. So I've picked up the "easy" patches that I have read somewhat often and also tested myself using the lima driver on some Rockchip socs (rk3036 + mali400 and rk3328 + mali450). I'll try to also look at the rest but no guarantees on timing as they look a lot more involved in real graphics-related stuff :-) Thanks Heiko