On Wed, Nov 25, 2020 at 05:29:39PM +0000, Robin Murphy wrote: > The only user of tlb_flush_leaf is a particularly hairy corner of the > Arm short-descriptor code, which wants a synchronous invalidation to > minimise the races inherent in trying to split a large page mapping. > This is already far enough into "here be dragons" territory that no > sensible caller should ever hit it, and thus it really doesn't need > optimising. Although using tlb_flush_walk there may technically be > more heavyweight than needed, it does the job and saves everyone else > having to carry around useless baggage. > > Signed-off-by: Robin Murphy <robin.murphy@xxxxxxx> > --- > > Reviewing the Mediatek TLB optimisation patches just left me thinking > "why do we even have this?"... Panfrost folks, this has zero functional > impact to you, merely wants an ack for straying outside drivers/iommu. Thanks, this looks good to me, but I'll defer queuing it until the last minute so that I can merge all the iommu component branches together first and then apply this on top. Should happen next week. Will _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
