diff for duplicates of <1492036240.7236.80.camel@kernel.crashing.org>

diff --git a/a/1.txt b/N1/1.txt
index d8d959a..6f9e288 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -1,55 +1,45 @@
-On Wed, 2017-04-12 at 15:41 +0100, Lorenzo Pieralisi wrote:
-> > > At least we make the failure explicit, and the driver can take
-> > > alternate (possibly sub-optimal) action if it chooses to do so.
-> > 
-> > The same points apply to things like pgprot_writecombine(),
-> > pgprot_noncached(), pgprot_device().  Then there's also pgprot_nonposted()
-> > that this series also introduces.
-
-No. pgprot_writecombine() silently falling back to simply non-cached is
-ok as we aren't "weakening" the ordering rules silently here. Something
-that is correct with writecombine will also work without. It's just an
-optimisation. Not correctness.
-
-Things like noncached() must of course be honored, and I don't think
-we have a case anywhere where it isn't.
-
-My point with nopost() is that it's never ok to silently downgrade it.
-Code written with the assumption that there is no posting will be
-*incorrect* if posting happens. We do live with that "bug" today indeed
-but once we have that accessors we might start growing more code that
-relies on the specific attribute that things aren't posted and will be
-wrong on all the archs providing the default implementation.
-
-This is why I insist that pgprot_nopost() if it exists globally, should
-return NULL when the semantic cannot be provided. That way there is a
-clear line in the sand. If the driver choses to operate with posted
-non-cached anyway, then make it an explicit driver choice.
-
-> > If ioremap_nopost() is not possible on an architecture, then
-> > pgprot_nonposted() won't be possible either - but you've made no
-> > mention of that so far.
-
-Right. It's not on most in fact.
-
-> > Just like the proposed ioremap_nopost(), pgprot_nonposted() is given a
-> > default implementation that uses pgprot_noncached().  Maybe we should
-> > also make pci_remap_iospace() fail if pgprot_nonposted() is not defined
-> > by the architecture?
-
-Or we *document* that mmap of IO space can result in something that is
-partially non-posted.
-
-> Yes, I was about to mention that and you are right, I should deal with
-> that too unfortunately. BTW, I have not posted the drivers to make the
-> review easier (ie it would add 20 more patches to an already massive
-> patch series - that will be trimmed when the asm-generic include is
-> removed from arches according to this discussion).
-> 
-> Thanks,
-> Lorenzo
-
-_______________________________________________
-linux-arm-kernel mailing list
-linux-arm-kernel@lists.infradead.org
-http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
+T24gV2VkLCAyMDE3LTA0LTEyIGF0IDE1OjQxICswMTAwLCBMb3JlbnpvIFBpZXJhbGlzaSB3cm90
+ZToKPiA+ID4gQXQgbGVhc3Qgd2UgbWFrZSB0aGUgZmFpbHVyZSBleHBsaWNpdCwgYW5kIHRoZSBk
+cml2ZXIgY2FuIHRha2UKPiA+ID4gYWx0ZXJuYXRlIChwb3NzaWJseSBzdWItb3B0aW1hbCkgYWN0
+aW9uIGlmIGl0IGNob29zZXMgdG8gZG8gc28uCj4gPiAKPiA+IFRoZSBzYW1lIHBvaW50cyBhcHBs
+eSB0byB0aGluZ3MgbGlrZSBwZ3Byb3Rfd3JpdGVjb21iaW5lKCksCj4gPiBwZ3Byb3Rfbm9uY2Fj
+aGVkKCksIHBncHJvdF9kZXZpY2UoKS7CoCBUaGVuIHRoZXJlJ3MgYWxzbyBwZ3Byb3Rfbm9ucG9z
+dGVkKCkKPiA+IHRoYXQgdGhpcyBzZXJpZXMgYWxzbyBpbnRyb2R1Y2VzLgoKTm8uIHBncHJvdF93
+cml0ZWNvbWJpbmUoKSBzaWxlbnRseSBmYWxsaW5nIGJhY2sgdG8gc2ltcGx5IG5vbi1jYWNoZWQg
+aXMKb2sgYXMgd2UgYXJlbid0ICJ3ZWFrZW5pbmciIHRoZSBvcmRlcmluZyBydWxlcyBzaWxlbnRs
+eSBoZXJlLiBTb21ldGhpbmcKdGhhdCBpcyBjb3JyZWN0IHdpdGggd3JpdGVjb21iaW5lIHdpbGwg
+YWxzbyB3b3JrIHdpdGhvdXQuIEl0J3MganVzdCBhbgpvcHRpbWlzYXRpb24uIE5vdCBjb3JyZWN0
+bmVzcy4KClRoaW5ncyBsaWtlIG5vbmNhY2hlZCgpIG11c3Qgb2YgY291cnNlIGJlIGhvbm9yZWQs
+IGFuZCBJIGRvbid0IHRoaW5rCndlIGhhdmUgYSBjYXNlIGFueXdoZXJlIHdoZXJlIGl0IGlzbid0
+LgoKTXkgcG9pbnQgd2l0aCBub3Bvc3QoKSBpcyB0aGF0IGl0J3MgbmV2ZXIgb2sgdG8gc2lsZW50
+bHkgZG93bmdyYWRlIGl0LgpDb2RlIHdyaXR0ZW4gd2l0aCB0aGUgYXNzdW1wdGlvbiB0aGF0IHRo
+ZXJlIGlzIG5vIHBvc3Rpbmcgd2lsbCBiZQoqaW5jb3JyZWN0KiBpZiBwb3N0aW5nIGhhcHBlbnMu
+IFdlIGRvIGxpdmUgd2l0aCB0aGF0ICJidWciIHRvZGF5IGluZGVlZApidXQgb25jZSB3ZSBoYXZl
+IHRoYXQgYWNjZXNzb3JzIHdlIG1pZ2h0IHN0YXJ0IGdyb3dpbmcgbW9yZSBjb2RlIHRoYXQKcmVs
+aWVzIG9uIHRoZSBzcGVjaWZpYyBhdHRyaWJ1dGUgdGhhdCB0aGluZ3MgYXJlbid0IHBvc3RlZCBh
+bmQgd2lsbCBiZQp3cm9uZyBvbiBhbGwgdGhlIGFyY2hzIHByb3ZpZGluZyB0aGUgZGVmYXVsdCBp
+bXBsZW1lbnRhdGlvbi4KClRoaXMgaXMgd2h5IEkgaW5zaXN0IHRoYXQgcGdwcm90X25vcG9zdCgp
+IGlmIGl0IGV4aXN0cyBnbG9iYWxseSwgc2hvdWxkCnJldHVybiBOVUxMIHdoZW4gdGhlIHNlbWFu
+dGljIGNhbm5vdCBiZSBwcm92aWRlZC4gVGhhdCB3YXkgdGhlcmUgaXMgYQpjbGVhciBsaW5lIGlu
+IHRoZSBzYW5kLiBJZiB0aGUgZHJpdmVyIGNob3NlcyB0byBvcGVyYXRlIHdpdGggcG9zdGVkCm5v
+bi1jYWNoZWQgYW55d2F5LCB0aGVuIG1ha2UgaXQgYW4gZXhwbGljaXQgZHJpdmVyIGNob2ljZS4K
+Cj4gPiBJZiBpb3JlbWFwX25vcG9zdCgpIGlzIG5vdCBwb3NzaWJsZSBvbiBhbiBhcmNoaXRlY3R1
+cmUsIHRoZW4KPiA+IHBncHJvdF9ub25wb3N0ZWQoKSB3b24ndCBiZSBwb3NzaWJsZSBlaXRoZXIg
+LSBidXQgeW91J3ZlIG1hZGUgbm8KPiA+IG1lbnRpb24gb2YgdGhhdCBzbyBmYXIuCgpSaWdodC4g
+SXQncyBub3Qgb24gbW9zdCBpbiBmYWN0LgoKPiA+IEp1c3QgbGlrZSB0aGUgcHJvcG9zZWQgaW9y
+ZW1hcF9ub3Bvc3QoKSwgcGdwcm90X25vbnBvc3RlZCgpIGlzIGdpdmVuIGEKPiA+IGRlZmF1bHQg
+aW1wbGVtZW50YXRpb24gdGhhdCB1c2VzIHBncHJvdF9ub25jYWNoZWQoKS7CoCBNYXliZSB3ZSBz
+aG91bGQKPiA+IGFsc28gbWFrZSBwY2lfcmVtYXBfaW9zcGFjZSgpIGZhaWwgaWYgcGdwcm90X25v
+bnBvc3RlZCgpIGlzIG5vdCBkZWZpbmVkCj4gPiBieSB0aGUgYXJjaGl0ZWN0dXJlPwoKT3Igd2Ug
+KmRvY3VtZW50KiB0aGF0IG1tYXAgb2YgSU8gc3BhY2UgY2FuIHJlc3VsdCBpbiBzb21ldGhpbmcg
+dGhhdCBpcwpwYXJ0aWFsbHkgbm9uLXBvc3RlZC4KCj4gWWVzLCBJIHdhcyBhYm91dCB0byBtZW50
+aW9uIHRoYXQgYW5kIHlvdSBhcmUgcmlnaHQsIEkgc2hvdWxkIGRlYWwgd2l0aAo+IHRoYXQgdG9v
+IHVuZm9ydHVuYXRlbHkuIEJUVywgSSBoYXZlIG5vdCBwb3N0ZWQgdGhlIGRyaXZlcnMgdG8gbWFr
+ZSB0aGUKPiByZXZpZXcgZWFzaWVyIChpZSBpdCB3b3VsZCBhZGQgMjAgbW9yZSBwYXRjaGVzIHRv
+IGFuIGFscmVhZHkgbWFzc2l2ZQo+IHBhdGNoIHNlcmllcyAtIHRoYXQgd2lsbCBiZSB0cmltbWVk
+IHdoZW4gdGhlIGFzbS1nZW5lcmljIGluY2x1ZGUgaXMKPiByZW1vdmVkIGZyb20gYXJjaGVzIGFj
+Y29yZGluZyB0byB0aGlzIGRpc2N1c3Npb24pLgo+IAo+IFRoYW5rcywKPiBMb3JlbnpvCgpfX19f
+X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2Vy
+bmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0
+cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVs
+Cg==
diff --git a/a/content_digest b/N1/content_digest
index 311bf40..1b17d56 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -34,63 +34,75 @@
   Matt Turner <mattst88@gmail.com>
   Haavard Skinnemoen <hskinnemoen@gmail.com>
   Fenghua Yu <fenghua.yu@intel.com>
- " James Hogan <james.hogan@imgtec.co>\0"
+  James Hogan <james.hogan@imgtec.com>
+  Chris Metcalf <cmetcalf@mellanox.com>
+  Arnd Bergmann <arnd@arndb.de>
+  Heiko Carstens <heiko.carstens@de.ibm.com>
+  Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>
+  Mikael Starvik <starvik@axis.com>
+  Ivan Kokshaysky <ink@jurassic.park.msu.ru>
+  Bjorn Helgaas <bhelgaas@google.com>
+  Stafford Horne <shorne@gmail.com>
+  linux-arm-kernel@lists.infradead.org
+  Richard Henderson <rth@twiddle.net>
+  Chris Zankel <chris@zankel.net>
+  Michal Simek <monstr@monstr.eu>
+  Tony Luck <tony.luck@intel.com>
+  Vineet Gupta <vgupta@synopsys.com>
+  linux-kernel@vger.kernel.org
+  Ralf Baechle <ralf@linux-mips.org>
+  Richard Kuo <rkuo@codeaurora.org>
+  Niklas Cassel <nks@flawful.org>
+  Luis R. Rodriguez <mcgrof@kernel.org>
+  Martin Schwidefsky <schwidefsky@de.ibm.com>
+  Ley Foon Tan <lftan@altera.com>
+ " David S. Miller <davem@davemloft.net>\0"
  "\00:1\0"
  "b\0"
- "On Wed, 2017-04-12 at 15:41 +0100, Lorenzo Pieralisi wrote:\n"
- "> > > At least we make the failure explicit, and the driver can take\n"
- "> > > alternate (possibly sub-optimal) action if it chooses to do so.\n"
- "> > \n"
- "> > The same points apply to things like pgprot_writecombine(),\n"
- "> > pgprot_noncached(), pgprot_device().\302\240 Then there's also pgprot_nonposted()\n"
- "> > that this series also introduces.\n"
- "\n"
- "No. pgprot_writecombine() silently falling back to simply non-cached is\n"
- "ok as we aren't \"weakening\" the ordering rules silently here. Something\n"
- "that is correct with writecombine will also work without. It's just an\n"
- "optimisation. Not correctness.\n"
- "\n"
- "Things like noncached() must of course be honored, and I don't think\n"
- "we have a case anywhere where it isn't.\n"
- "\n"
- "My point with nopost() is that it's never ok to silently downgrade it.\n"
- "Code written with the assumption that there is no posting will be\n"
- "*incorrect* if posting happens. We do live with that \"bug\" today indeed\n"
- "but once we have that accessors we might start growing more code that\n"
- "relies on the specific attribute that things aren't posted and will be\n"
- "wrong on all the archs providing the default implementation.\n"
- "\n"
- "This is why I insist that pgprot_nopost() if it exists globally, should\n"
- "return NULL when the semantic cannot be provided. That way there is a\n"
- "clear line in the sand. If the driver choses to operate with posted\n"
- "non-cached anyway, then make it an explicit driver choice.\n"
- "\n"
- "> > If ioremap_nopost() is not possible on an architecture, then\n"
- "> > pgprot_nonposted() won't be possible either - but you've made no\n"
- "> > mention of that so far.\n"
- "\n"
- "Right. It's not on most in fact.\n"
- "\n"
- "> > Just like the proposed ioremap_nopost(), pgprot_nonposted() is given a\n"
- "> > default implementation that uses pgprot_noncached().\302\240 Maybe we should\n"
- "> > also make pci_remap_iospace() fail if pgprot_nonposted() is not defined\n"
- "> > by the architecture?\n"
- "\n"
- "Or we *document* that mmap of IO space can result in something that is\n"
- "partially non-posted.\n"
- "\n"
- "> Yes, I was about to mention that and you are right, I should deal with\n"
- "> that too unfortunately. BTW, I have not posted the drivers to make the\n"
- "> review easier (ie it would add 20 more patches to an already massive\n"
- "> patch series - that will be trimmed when the asm-generic include is\n"
- "> removed from arches according to this discussion).\n"
- "> \n"
- "> Thanks,\n"
- "> Lorenzo\n"
- "\n"
- "_______________________________________________\n"
- "linux-arm-kernel mailing list\n"
- "linux-arm-kernel@lists.infradead.org\n"
- http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
+ "T24gV2VkLCAyMDE3LTA0LTEyIGF0IDE1OjQxICswMTAwLCBMb3JlbnpvIFBpZXJhbGlzaSB3cm90\n"
+ "ZToKPiA+ID4gQXQgbGVhc3Qgd2UgbWFrZSB0aGUgZmFpbHVyZSBleHBsaWNpdCwgYW5kIHRoZSBk\n"
+ "cml2ZXIgY2FuIHRha2UKPiA+ID4gYWx0ZXJuYXRlIChwb3NzaWJseSBzdWItb3B0aW1hbCkgYWN0\n"
+ "aW9uIGlmIGl0IGNob29zZXMgdG8gZG8gc28uCj4gPiAKPiA+IFRoZSBzYW1lIHBvaW50cyBhcHBs\n"
+ "eSB0byB0aGluZ3MgbGlrZSBwZ3Byb3Rfd3JpdGVjb21iaW5lKCksCj4gPiBwZ3Byb3Rfbm9uY2Fj\n"
+ "aGVkKCksIHBncHJvdF9kZXZpY2UoKS7CoCBUaGVuIHRoZXJlJ3MgYWxzbyBwZ3Byb3Rfbm9ucG9z\n"
+ "dGVkKCkKPiA+IHRoYXQgdGhpcyBzZXJpZXMgYWxzbyBpbnRyb2R1Y2VzLgoKTm8uIHBncHJvdF93\n"
+ "cml0ZWNvbWJpbmUoKSBzaWxlbnRseSBmYWxsaW5nIGJhY2sgdG8gc2ltcGx5IG5vbi1jYWNoZWQg\n"
+ "aXMKb2sgYXMgd2UgYXJlbid0ICJ3ZWFrZW5pbmciIHRoZSBvcmRlcmluZyBydWxlcyBzaWxlbnRs\n"
+ "eSBoZXJlLiBTb21ldGhpbmcKdGhhdCBpcyBjb3JyZWN0IHdpdGggd3JpdGVjb21iaW5lIHdpbGwg\n"
+ "YWxzbyB3b3JrIHdpdGhvdXQuIEl0J3MganVzdCBhbgpvcHRpbWlzYXRpb24uIE5vdCBjb3JyZWN0\n"
+ "bmVzcy4KClRoaW5ncyBsaWtlIG5vbmNhY2hlZCgpIG11c3Qgb2YgY291cnNlIGJlIGhvbm9yZWQs\n"
+ "IGFuZCBJIGRvbid0IHRoaW5rCndlIGhhdmUgYSBjYXNlIGFueXdoZXJlIHdoZXJlIGl0IGlzbid0\n"
+ "LgoKTXkgcG9pbnQgd2l0aCBub3Bvc3QoKSBpcyB0aGF0IGl0J3MgbmV2ZXIgb2sgdG8gc2lsZW50\n"
+ "bHkgZG93bmdyYWRlIGl0LgpDb2RlIHdyaXR0ZW4gd2l0aCB0aGUgYXNzdW1wdGlvbiB0aGF0IHRo\n"
+ "ZXJlIGlzIG5vIHBvc3Rpbmcgd2lsbCBiZQoqaW5jb3JyZWN0KiBpZiBwb3N0aW5nIGhhcHBlbnMu\n"
+ "IFdlIGRvIGxpdmUgd2l0aCB0aGF0ICJidWciIHRvZGF5IGluZGVlZApidXQgb25jZSB3ZSBoYXZl\n"
+ "IHRoYXQgYWNjZXNzb3JzIHdlIG1pZ2h0IHN0YXJ0IGdyb3dpbmcgbW9yZSBjb2RlIHRoYXQKcmVs\n"
+ "aWVzIG9uIHRoZSBzcGVjaWZpYyBhdHRyaWJ1dGUgdGhhdCB0aGluZ3MgYXJlbid0IHBvc3RlZCBh\n"
+ "bmQgd2lsbCBiZQp3cm9uZyBvbiBhbGwgdGhlIGFyY2hzIHByb3ZpZGluZyB0aGUgZGVmYXVsdCBp\n"
+ "bXBsZW1lbnRhdGlvbi4KClRoaXMgaXMgd2h5IEkgaW5zaXN0IHRoYXQgcGdwcm90X25vcG9zdCgp\n"
+ "IGlmIGl0IGV4aXN0cyBnbG9iYWxseSwgc2hvdWxkCnJldHVybiBOVUxMIHdoZW4gdGhlIHNlbWFu\n"
+ "dGljIGNhbm5vdCBiZSBwcm92aWRlZC4gVGhhdCB3YXkgdGhlcmUgaXMgYQpjbGVhciBsaW5lIGlu\n"
+ "IHRoZSBzYW5kLiBJZiB0aGUgZHJpdmVyIGNob3NlcyB0byBvcGVyYXRlIHdpdGggcG9zdGVkCm5v\n"
+ "bi1jYWNoZWQgYW55d2F5LCB0aGVuIG1ha2UgaXQgYW4gZXhwbGljaXQgZHJpdmVyIGNob2ljZS4K\n"
+ "Cj4gPiBJZiBpb3JlbWFwX25vcG9zdCgpIGlzIG5vdCBwb3NzaWJsZSBvbiBhbiBhcmNoaXRlY3R1\n"
+ "cmUsIHRoZW4KPiA+IHBncHJvdF9ub25wb3N0ZWQoKSB3b24ndCBiZSBwb3NzaWJsZSBlaXRoZXIg\n"
+ "LSBidXQgeW91J3ZlIG1hZGUgbm8KPiA+IG1lbnRpb24gb2YgdGhhdCBzbyBmYXIuCgpSaWdodC4g\n"
+ "SXQncyBub3Qgb24gbW9zdCBpbiBmYWN0LgoKPiA+IEp1c3QgbGlrZSB0aGUgcHJvcG9zZWQgaW9y\n"
+ "ZW1hcF9ub3Bvc3QoKSwgcGdwcm90X25vbnBvc3RlZCgpIGlzIGdpdmVuIGEKPiA+IGRlZmF1bHQg\n"
+ "aW1wbGVtZW50YXRpb24gdGhhdCB1c2VzIHBncHJvdF9ub25jYWNoZWQoKS7CoCBNYXliZSB3ZSBz\n"
+ "aG91bGQKPiA+IGFsc28gbWFrZSBwY2lfcmVtYXBfaW9zcGFjZSgpIGZhaWwgaWYgcGdwcm90X25v\n"
+ "bnBvc3RlZCgpIGlzIG5vdCBkZWZpbmVkCj4gPiBieSB0aGUgYXJjaGl0ZWN0dXJlPwoKT3Igd2Ug\n"
+ "KmRvY3VtZW50KiB0aGF0IG1tYXAgb2YgSU8gc3BhY2UgY2FuIHJlc3VsdCBpbiBzb21ldGhpbmcg\n"
+ "dGhhdCBpcwpwYXJ0aWFsbHkgbm9uLXBvc3RlZC4KCj4gWWVzLCBJIHdhcyBhYm91dCB0byBtZW50\n"
+ "aW9uIHRoYXQgYW5kIHlvdSBhcmUgcmlnaHQsIEkgc2hvdWxkIGRlYWwgd2l0aAo+IHRoYXQgdG9v\n"
+ "IHVuZm9ydHVuYXRlbHkuIEJUVywgSSBoYXZlIG5vdCBwb3N0ZWQgdGhlIGRyaXZlcnMgdG8gbWFr\n"
+ "ZSB0aGUKPiByZXZpZXcgZWFzaWVyIChpZSBpdCB3b3VsZCBhZGQgMjAgbW9yZSBwYXRjaGVzIHRv\n"
+ "IGFuIGFscmVhZHkgbWFzc2l2ZQo+IHBhdGNoIHNlcmllcyAtIHRoYXQgd2lsbCBiZSB0cmltbWVk\n"
+ "IHdoZW4gdGhlIGFzbS1nZW5lcmljIGluY2x1ZGUgaXMKPiByZW1vdmVkIGZyb20gYXJjaGVzIGFj\n"
+ "Y29yZGluZyB0byB0aGlzIGRpc2N1c3Npb24pLgo+IAo+IFRoYW5rcywKPiBMb3JlbnpvCgpfX19f\n"
+ "X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1hcm0ta2Vy\n"
+ "bmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0\n"
+ "cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1hcm0ta2VybmVs\n"
+ Cg==
 
-380af893acb59c044b385d2c14385a2aac128ea6aab29c6553585ab39281e167
+a954b43ed8f12ddffe87ec37ab901c96ce955e4bf577c251a20a3b92e90f3c1e
