linux-kernel - Re: [PATCH v3 00/17] Cross-architecture definitions of relaxed MMIO accessors

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20141031110951.GB10778@ulmo.nvidia.com>
Date:	Fri, 31 Oct 2014 12:09:53 +0100
From:	Thierry Reding <thierry.reding@...il.com>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	Will Deacon <will.deacon@....com>,
	"linux-arch@...r.kernel.org" <linux-arch@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"benh@...nel.crashing.org" <benh@...nel.crashing.org>,
	"chris@...kel.net" <chris@...kel.net>,
	"cmetcalf@...era.com" <cmetcalf@...era.com>,
	"davem@...emloft.net" <davem@...emloft.net>,
	"deller@....de" <deller@....de>,
	"dhowells@...hat.com" <dhowells@...hat.com>,
	"geert@...ux-m68k.org" <geert@...ux-m68k.org>,
	"heiko.carstens@...ibm.com" <heiko.carstens@...ibm.com>,
	"hpa@...or.com" <hpa@...or.com>,
	"jcmvbkbc@...il.com" <jcmvbkbc@...il.com>,
	"jesper.nilsson@...s.com" <jesper.nilsson@...s.com>,
	"mingo@...hat.com" <mingo@...hat.com>,
	"monstr@...str.eu" <monstr@...str.eu>,
	"paulmck@...ux.vnet.ibm.com" <paulmck@...ux.vnet.ibm.com>,
	"rdunlap@...radead.org" <rdunlap@...radead.org>,
	"sam@...nborg.org" <sam@...nborg.org>,
	"schwidefsky@...ibm.com" <schwidefsky@...ibm.com>,
	"starvik@...s.com" <starvik@...s.com>,
	"takata@...ux-m32r.org" <takata@...ux-m32r.org>,
	"tglx@...utronix.de" <tglx@...utronix.de>,
	"tony.luck@...el.com" <tony.luck@...el.com>,
	"daniel.thompson@...aro.org" <daniel.thompson@...aro.org>,
	"broonie@...aro.org" <broonie@...aro.org>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>
Subject: Re: [PATCH v3 00/17] Cross-architecture definitions of relaxed MMIO
 accessors

On Thu, Oct 30, 2014 at 09:04:54PM +0100, Arnd Bergmann wrote:
> On Thursday 30 October 2014 16:59:07 Will Deacon wrote:
> > On Thu, Sep 25, 2014 at 02:15:10PM +0100, Arnd Bergmann wrote:
> > > On Wednesday 24 September 2014 18:17:19 Will Deacon wrote:
> > > > Hello everybody,
> > > > 
> > > > This is version three of the series I've originally posted here:
> > > > 
> > > >   v1: https://lkml.org/lkml/2014/4/17/269
> > > >   v2: https://lkml.org/lkml/2014/5/22/468
> > > > 
> > > > This is basically just a rebase on top of 3.17-rc6, minus the alpha patch
> > > > (which was merged into mainline).
> > > > 
> > > > I looked at reworking the non-relaxed accessors to imply mmiowb, but it
> > > > quickly got messy as some architectures (e.g. mips) deliberately keep
> > > > mmiowb and readX/writeX separate whilst others (e.g. powerpc) don't trust
> > > > drivers to get mmiowb correct, so add barriers to both. Given that
> > > > arm/arm64/x86 don't care about mmiowb, I've left that as an exercise for
> > > > an architecture that does care.
> > > > 
> > > > In order to get this lot merged, we probably want to merge the asm-generic
> > > > patch (1/17) first, so Acks would be much appreciated on the architecture
> > > > bits.
> > > > 
> > > > As before, I've included the original cover letter below, as that describes
> > > > what I'm trying to do in more detail.
> > > > 
> > > 
> > > I've now applied the parts of your series that are required to have
> > > every architecture provide all the 'relaxed' accessors to the
> > > asm-generic tree, on top of Thierry's series.
> > 
> > Since these didn't make it for 3.18, would you like me to repost the series,
> > or do you already have a branch suitable for 3.19?
> 
> I still need to figure out how to do this right in combination with Thierry's
> patches. Last time I had his patches first and yours on top, and I didn't
> manage to rebase your series when his broke.

Would it help if we merged both series so that we have to rebase only
once? I'd really like to get the asm-generic/io.h series merged soon
because 64-bit Tegra support depends on it and we'd really like to get
started on that.

Thierry

Content of type "application/pgp-signature" skipped