From: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>
To: Robin Murphy <robin.murphy-5wv7dgnIgG8@public.gmane.org>
Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
sgoutham-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v2 0/4] SMMUv3 CMD_SYNC optimisation
Date: Mon, 16 Oct 2017 16:02:09 +0100 [thread overview]
Message-ID: <20171016150208.GA10500@arm.com> (raw)
In-Reply-To: <20171013190521.GD30572-5wv7dgnIgG8@public.gmane.org>
On Fri, Oct 13, 2017 at 08:05:21PM +0100, Will Deacon wrote:
> 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.
No idea what I was doing wrong on Friday, but the first two patches apply
cleanly now, so I'll push them out in a bit.
Sorry for the noise,
Will
WARNING: multiple messages have this Message-ID (diff)
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/4] SMMUv3 CMD_SYNC optimisation
Date: Mon, 16 Oct 2017 16:02:09 +0100 [thread overview]
Message-ID: <20171016150208.GA10500@arm.com> (raw)
In-Reply-To: <20171013190521.GD30572@arm.com>
On Fri, Oct 13, 2017 at 08:05:21PM +0100, Will Deacon wrote:
> 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 at lists.linux-foundation.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.
No idea what I was doing wrong on Friday, but the first two patches apply
cleanly now, so I'll push them out in a bit.
Sorry for the noise,
Will
next prev parent reply other threads:[~2017-10-16 15:02 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-31 13:44 [PATCH v2 0/4] SMMUv3 CMD_SYNC optimisation Robin Murphy
2017-08-31 13:44 ` Robin Murphy
[not found] ` <cover.1504182142.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2017-08-31 13:44 ` [PATCH v2 1/4] iommu/arm-smmu-v3: Specialise CMD_SYNC handling Robin Murphy
2017-08-31 13:44 ` Robin Murphy
2017-08-31 13:44 ` [PATCH v2 2/4] iommu/arm-smmu-v3: Forget about cmdq-sync interrupt Robin Murphy
2017-08-31 13:44 ` Robin Murphy
2017-08-31 13:44 ` [PATCH v2 3/4] iommu/arm-smmu-v3: Use CMD_SYNC completion MSI Robin Murphy
2017-08-31 13:44 ` Robin Murphy
[not found] ` <dbf0ce00f8e249c64f3d2041acd8d91818178e52.1504182142.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2017-10-13 18:32 ` Will Deacon
2017-10-13 18:32 ` Will Deacon
[not found] ` <20171013183237.GA30572-5wv7dgnIgG8@public.gmane.org>
2017-10-16 12:25 ` Robin Murphy
2017-10-16 12:25 ` Robin Murphy
2017-08-31 13:44 ` [PATCH v2 4/4] iommu/arm-smmu-v3: Poll for CMD_SYNC outside cmdq lock Robin Murphy
2017-08-31 13:44 ` Robin Murphy
[not found] ` <ff239173e47dfa0fc76eaa2a25b3cbcfe8dce5e6.1504182142.git.robin.murphy-5wv7dgnIgG8@public.gmane.org>
2017-10-13 18:59 ` Will Deacon
2017-10-13 18:59 ` Will Deacon
[not found] ` <20171013185917.GB30572-5wv7dgnIgG8@public.gmane.org>
2017-10-16 13:12 ` Robin Murphy
2017-10-16 13:12 ` Robin Murphy
2017-08-31 13:44 ` [RFT] iommu/arm-smmu-v3: Use burst-polling for sync completion Robin Murphy
2017-08-31 13:44 ` Robin Murphy
2017-10-13 19:05 ` [PATCH v2 0/4] SMMUv3 CMD_SYNC optimisation Will Deacon
2017-10-13 19:05 ` Will Deacon
[not found] ` <20171013190521.GD30572-5wv7dgnIgG8@public.gmane.org>
2017-10-16 13:18 ` Robin Murphy
2017-10-16 13:18 ` Robin Murphy
2017-10-16 15:02 ` Will Deacon [this message]
2017-10-16 15:02 ` Will Deacon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171016150208.GA10500@arm.com \
--to=will.deacon-5wv7dgnigg8@public.gmane.org \
--cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=robin.murphy-5wv7dgnIgG8@public.gmane.org \
--cc=sgoutham-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.