From mboxrd@z Thu Jan 1 00:00:00 1970 From: Will Deacon Subject: Re: [PATCH v2 0/4] SMMUv3 CMD_SYNC optimisation Date: Fri, 13 Oct 2017 20:05:22 +0100 Message-ID: <20171013190521.GD30572@arm.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Robin Murphy Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, sgoutham-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: iommu@lists.linux-foundation.org Hi Robin, On Thu, Aug 31, 2017 at 02:44:24PM +0100, Robin Murphy wrote: > Since Nate reported a reasonable performance boost from the out-of-line > MSI polling in v1 [1], I've now implemented the equivalent for cons > polling as well - that has been boot-tested on D05 with some trivial I/O > and at least doesn't seem to lock up or explode. There's also a little > cosmetic tweaking to make the patches a bit cleaner as a series. > > Robin. > > [1] https://www.mail-archive.com/iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org/msg19657.html > > Robin Murphy (5): > iommu/arm-smmu-v3: Specialise CMD_SYNC handling > iommu/arm-smmu-v3: Forget about cmdq-sync interrupt > iommu/arm-smmu-v3: Use CMD_SYNC completion interrupt > iommu/arm-smmu-v3: Poll for CMD_SYNC outside cmdq lock > iommu/arm-smmu-v3: Use burst-polling for sync completion What's this final mythical patch about? I don't see it in the series. Anyway, the first two patches look fine to me, but this doesn't apply on top of my iommu/devel branch so they will need rebasing. Cheers, Will