On 26/08/2019 23:33, Rob Herring wrote: > It's not entirely clear if this is required, but add a flush of GPU caches > and TLBs before we change an address space to new page tables. > > Fixes: 7282f7645d06 ("drm/panfrost: Implement per FD address spaces") > Cc: Tomeu Vizoso <tomeu.viz...@collabora.com> > Cc: Steven Price <steven.pr...@arm.com> > Cc: Alyssa Rosenzweig <alyssa.rosenzw...@collabora.com> > Cc: David Airlie <airl...@linux.ie> > Cc: Daniel Vetter <dan...@ffwll.ch> > Signed-off-by: Rob Herring <r...@kernel.org>
Equally I'm not sure this is required, but it looks like there might be a path where a AS is reclaimed and then immediately freed (client closes fd) which would need a flush (with wait) to ensure that translations had definitely finished. And it won't do any harm, so: Reviewed-by: Steven Price <steven.pr...@arm.com> Steve > --- > drivers/gpu/drm/panfrost/panfrost_mmu.c | 2 ++ > 1 file changed, 2 insertions(+) > v3: > - New patch > > diff --git a/drivers/gpu/drm/panfrost/panfrost_mmu.c > b/drivers/gpu/drm/panfrost/panfrost_mmu.c > index 3407b00d0a3a..d1ebde3327fe 100644 > --- a/drivers/gpu/drm/panfrost/panfrost_mmu.c > +++ b/drivers/gpu/drm/panfrost/panfrost_mmu.c > @@ -115,6 +115,8 @@ static void panfrost_mmu_enable(struct panfrost_device > *pfdev, struct panfrost_m > u64 transtab = cfg->arm_mali_lpae_cfg.transtab; > u64 memattr = cfg->arm_mali_lpae_cfg.memattr; > > + mmu_hw_do_operation_locked(pfdev, as_nr, 0, ~0UL, AS_COMMAND_FLUSH_MEM); > + > mmu_write(pfdev, AS_TRANSTAB_LO(as_nr), transtab & 0xffffffffUL); > mmu_write(pfdev, AS_TRANSTAB_HI(as_nr), transtab >> 32); > > -- > 2.20.1 > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel