All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <1462733173.3006.7.camel@intel.com>

diff --git a/a/1.txt b/N1/1.txt
index 2af45e3..4e6c765 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -1,19 +1,28 @@
-T24gU3VuLCAyMDE2LTA1LTA4IGF0IDAxOjUyIC0wNzAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90
-ZToNCj4gT24gRnJpLCBNYXkgMDYsIDIwMTYgYXQgMDM6NTM6MDlQTSAtMDYwMCwgVmlzaGFsIFZl
-cm1hIHdyb3RlOg0KPiA+IA0KPiA+IEZyb206IE1hdHRoZXcgV2lsY294IDxtYXR0aGV3LnIud2ls
-Y294QGludGVsLmNvbT4NCj4gPiANCj4gPiBkYXhfY2xlYXJfc2VjdG9ycygpIGNhbm5vdCBoYW5k
-bGUgcG9pc29uZWQgYmxvY2tzLsKgwqBUaGVzZSBtdXN0IGJlDQo+ID4gemVyb2VkIHVzaW5nIHRo
-ZSBCSU8gaW50ZXJmYWNlIGluc3RlYWQuwqDCoENvbnZlcnQgZXh0MiBhbmQgWEZTIHRvDQo+ID4g
-dXNlDQo+ID4gb25seSBzYl9pc3N1ZV96ZXJvdXQoKS4NCj4gPiANCj4gPiBTaWduZWQtb2ZmLWJ5
-OiBNYXR0aGV3IFdpbGNveCA8bWF0dGhldy5yLndpbGNveEBpbnRlbC5jb20+DQo+ID4gW3Zpc2hh
-bDogQWxzbyByZW1vdmUgdGhlIGRheF9jbGVhcl9zZWN0b3JzIGZ1bmN0aW9uIGVudGlyZWx5XQ0K
-PiA+IFNpZ25lZC1vZmYtYnk6IFZpc2hhbCBWZXJtYSA8dmlzaGFsLmwudmVybWFAaW50ZWwuY29t
-Pg0KPiBKdXN0IHRvIG1ha2Ugc3VyZTrCoMKgdGhlIGV4aXN0aW5nIHNiX2lzc3VlX3plcm91dCBh
-cyBpbiA0LjYtcmMNCj4gaXMgYWxyZWFkeSBkb2luZyB0aGUgcmlnaHQgdGhpbmcgZm9yIERBWD/C
-oMKgSSd2ZSBnb3QgYSBwZW5kaW5nDQo+IHBhdGNoc2V0DQo+IGZvciBYRlMgdGhhdCBpbnRyb2R1
-Y2VzIGFub3RoZXIgZGF4X2NsZWFyX3NlY3RvcnMgdXNlcnMsIGJ1dCBpZiBpdCdzDQo+IGFscmVh
-ZHkgc2FmZSB0byB1c2UgYmxrZGV2X2lzc3VlX3plcm9vdXQgSSBjYW4gc3dpdGNoIHRvIHRoYXQg
-YW5kDQo+IGF2b2lkDQo+IHRoZSBtZXJnZSBjb25mbGljdC4NCg0KSSBiZWxpZXZlIHNvIC0gSmFu
-IGhhcyBtb3ZlZCBhbGwgdW53cml0dGVuIGV4dGVudCBjb252ZXJzaW9ucyBvdXQgb2YNCkRBWCB3
-aXRoIGhpcyBwYXRjaCBzZXQsIGFuZCBJIGJlbGlldmUgemVyb2luZyB0aHJvdWdoIHRoZSBkcml2
-ZXIgaXMNCmFsd2F5cyBmaW5lLiBSb3NzIG9yIEphbiBjb3VsZCBjb25maXJtIHRob3VnaC7CoA==
+On Sun, 2016-05-08 at 01:52 -0700, Christoph Hellwig wrote:
+> On Fri, May 06, 2016 at 03:53:09PM -0600, Vishal Verma wrote:
+> > 
+> > From: Matthew Wilcox <matthew.r.wilcox@intel.com>
+> > 
+> > dax_clear_sectors() cannot handle poisoned blocks.  These must be
+> > zeroed using the BIO interface instead.  Convert ext2 and XFS to
+> > use
+> > only sb_issue_zerout().
+> > 
+> > Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
+> > [vishal: Also remove the dax_clear_sectors function entirely]
+> > Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
+> Just to make sure:  the existing sb_issue_zerout as in 4.6-rc
+> is already doing the right thing for DAX?  I've got a pending
+> patchset
+> for XFS that introduces another dax_clear_sectors users, but if it's
+> already safe to use blkdev_issue_zeroout I can switch to that and
+> avoid
+> the merge conflict.
+
+I believe so - Jan has moved all unwritten extent conversions out of
+DAX with his patch set, and I believe zeroing through the driver is
+always fine. Ross or Jan could confirm though. 
+_______________________________________________
+xfs mailing list
+xfs@oss.sgi.com
+http://oss.sgi.com/mailman/listinfo/xfs
diff --git a/a/content_digest b/N1/content_digest
index 1cd4d65..bbf95b6 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -5,44 +5,52 @@
  "Subject\0Re: [PATCH v5 3/5] dax: use sb_issue_zerout instead of calling dax_clear_sectors\0"
  "Date\0Sun, 8 May 2016 18:46:13 +0000\0"
  "To\0hch@infradead.org <hch@infradead.org>\0"
- "Cc\0linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>"
-  linux-block@vger.kernel.org <linux-block@vger.kernel.org>
-  xfs@oss.sgi.com <xfs@oss.sgi.com>
+ "Cc\0axboe@fb.com <axboe@fb.com>"
+  jack@suse.cz <jack@suse.cz>
+  boaz@plexistor.com <boaz@plexistor.com>
   linux-nvdimm@ml01.01.org <linux-nvdimm@ml01.01.org>
-  jmoyer@redhat.com <jmoyer@redhat.com>
+  linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org>
+  xfs@oss.sgi.com <xfs@oss.sgi.com>
+  linux-block@vger.kernel.org <linux-block@vger.kernel.org>
   linux-mm@kvack.org <linux-mm@kvack.org>
+  jmoyer@redhat.com <jmoyer@redhat.com>
+  Wilcox
+  Matthew R <matthew.r.wilcox@intel.com>
+  ross.zwisler@linux.intel.com <ross.zwisler@linux.intel.com>
+  linux-fsdevel@vger.kernel.org <linux-fsdevel@vger.kernel.org>
   Williams
   Dan J <dan.j.williams@intel.com>
-  axboe@fb.com <axboe@fb.com>
-  akpm@linux-foundation.org <akpm@linux-foundation.org>
-  linux-fsdevel@vger.kernel.org <linux-fsdevel@vger.kernel.org>
-  ross.zwisler@linux.intel.com <ross.zwisler@linux.intel.com>
   linux-ext4@vger.kernel.org <linux-ext4@vger.kernel.org>
-  boaz@plexistor.com <boaz@plexistor.com>
-  Wilcox
-  Matthew R <matthew.r.wilcox@intel.com>
-  david@fromorbit.com <david@fromorbit.com>
- " jack@suse.cz <jack@suse.cz>\0"
+ " akpm@linux-foundation.org <akpm@linux-foundation.org>\0"
  "\00:1\0"
  "b\0"
- "T24gU3VuLCAyMDE2LTA1LTA4IGF0IDAxOjUyIC0wNzAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90\n"
- "ZToNCj4gT24gRnJpLCBNYXkgMDYsIDIwMTYgYXQgMDM6NTM6MDlQTSAtMDYwMCwgVmlzaGFsIFZl\n"
- "cm1hIHdyb3RlOg0KPiA+IA0KPiA+IEZyb206IE1hdHRoZXcgV2lsY294IDxtYXR0aGV3LnIud2ls\n"
- "Y294QGludGVsLmNvbT4NCj4gPiANCj4gPiBkYXhfY2xlYXJfc2VjdG9ycygpIGNhbm5vdCBoYW5k\n"
- "bGUgcG9pc29uZWQgYmxvY2tzLsKgwqBUaGVzZSBtdXN0IGJlDQo+ID4gemVyb2VkIHVzaW5nIHRo\n"
- "ZSBCSU8gaW50ZXJmYWNlIGluc3RlYWQuwqDCoENvbnZlcnQgZXh0MiBhbmQgWEZTIHRvDQo+ID4g\n"
- "dXNlDQo+ID4gb25seSBzYl9pc3N1ZV96ZXJvdXQoKS4NCj4gPiANCj4gPiBTaWduZWQtb2ZmLWJ5\n"
- "OiBNYXR0aGV3IFdpbGNveCA8bWF0dGhldy5yLndpbGNveEBpbnRlbC5jb20+DQo+ID4gW3Zpc2hh\n"
- "bDogQWxzbyByZW1vdmUgdGhlIGRheF9jbGVhcl9zZWN0b3JzIGZ1bmN0aW9uIGVudGlyZWx5XQ0K\n"
- "PiA+IFNpZ25lZC1vZmYtYnk6IFZpc2hhbCBWZXJtYSA8dmlzaGFsLmwudmVybWFAaW50ZWwuY29t\n"
- "Pg0KPiBKdXN0IHRvIG1ha2Ugc3VyZTrCoMKgdGhlIGV4aXN0aW5nIHNiX2lzc3VlX3plcm91dCBh\n"
- "cyBpbiA0LjYtcmMNCj4gaXMgYWxyZWFkeSBkb2luZyB0aGUgcmlnaHQgdGhpbmcgZm9yIERBWD/C\n"
- "oMKgSSd2ZSBnb3QgYSBwZW5kaW5nDQo+IHBhdGNoc2V0DQo+IGZvciBYRlMgdGhhdCBpbnRyb2R1\n"
- "Y2VzIGFub3RoZXIgZGF4X2NsZWFyX3NlY3RvcnMgdXNlcnMsIGJ1dCBpZiBpdCdzDQo+IGFscmVh\n"
- "ZHkgc2FmZSB0byB1c2UgYmxrZGV2X2lzc3VlX3plcm9vdXQgSSBjYW4gc3dpdGNoIHRvIHRoYXQg\n"
- "YW5kDQo+IGF2b2lkDQo+IHRoZSBtZXJnZSBjb25mbGljdC4NCg0KSSBiZWxpZXZlIHNvIC0gSmFu\n"
- "IGhhcyBtb3ZlZCBhbGwgdW53cml0dGVuIGV4dGVudCBjb252ZXJzaW9ucyBvdXQgb2YNCkRBWCB3\n"
- "aXRoIGhpcyBwYXRjaCBzZXQsIGFuZCBJIGJlbGlldmUgemVyb2luZyB0aHJvdWdoIHRoZSBkcml2\n"
- ZXIgaXMNCmFsd2F5cyBmaW5lLiBSb3NzIG9yIEphbiBjb3VsZCBjb25maXJtIHRob3VnaC7CoA==
+ "On Sun, 2016-05-08 at 01:52 -0700, Christoph Hellwig wrote:\n"
+ "> On Fri, May 06, 2016 at 03:53:09PM -0600, Vishal Verma wrote:\n"
+ "> > \n"
+ "> > From: Matthew Wilcox <matthew.r.wilcox@intel.com>\n"
+ "> > \n"
+ "> > dax_clear_sectors() cannot handle poisoned blocks.\302\240\302\240These must be\n"
+ "> > zeroed using the BIO interface instead.\302\240\302\240Convert ext2 and XFS to\n"
+ "> > use\n"
+ "> > only sb_issue_zerout().\n"
+ "> > \n"
+ "> > Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>\n"
+ "> > [vishal: Also remove the dax_clear_sectors function entirely]\n"
+ "> > Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>\n"
+ "> Just to make sure:\302\240\302\240the existing sb_issue_zerout as in 4.6-rc\n"
+ "> is already doing the right thing for DAX?\302\240\302\240I've got a pending\n"
+ "> patchset\n"
+ "> for XFS that introduces another dax_clear_sectors users, but if it's\n"
+ "> already safe to use blkdev_issue_zeroout I can switch to that and\n"
+ "> avoid\n"
+ "> the merge conflict.\n"
+ "\n"
+ "I believe so - Jan has moved all unwritten extent conversions out of\n"
+ "DAX with his patch set, and I believe zeroing through the driver is\n"
+ "always fine. Ross or Jan could confirm though.\302\240\n"
+ "_______________________________________________\n"
+ "xfs mailing list\n"
+ "xfs@oss.sgi.com\n"
+ http://oss.sgi.com/mailman/listinfo/xfs
 
-682802f1aa7d196ccfec3dd227b3080349b4bf9737251e7f688468b58eed84d7
+f4bc3b24fb0de376cb1db019168845af08365bbfc34c847ca0450c15abd617b3

diff --git a/a/1.txt b/N2/1.txt
index 2af45e3..ebd415a 100644
--- a/a/1.txt
+++ b/N2/1.txt
@@ -1,19 +1,24 @@
-T24gU3VuLCAyMDE2LTA1LTA4IGF0IDAxOjUyIC0wNzAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90
-ZToNCj4gT24gRnJpLCBNYXkgMDYsIDIwMTYgYXQgMDM6NTM6MDlQTSAtMDYwMCwgVmlzaGFsIFZl
-cm1hIHdyb3RlOg0KPiA+IA0KPiA+IEZyb206IE1hdHRoZXcgV2lsY294IDxtYXR0aGV3LnIud2ls
-Y294QGludGVsLmNvbT4NCj4gPiANCj4gPiBkYXhfY2xlYXJfc2VjdG9ycygpIGNhbm5vdCBoYW5k
-bGUgcG9pc29uZWQgYmxvY2tzLsKgwqBUaGVzZSBtdXN0IGJlDQo+ID4gemVyb2VkIHVzaW5nIHRo
-ZSBCSU8gaW50ZXJmYWNlIGluc3RlYWQuwqDCoENvbnZlcnQgZXh0MiBhbmQgWEZTIHRvDQo+ID4g
-dXNlDQo+ID4gb25seSBzYl9pc3N1ZV96ZXJvdXQoKS4NCj4gPiANCj4gPiBTaWduZWQtb2ZmLWJ5
-OiBNYXR0aGV3IFdpbGNveCA8bWF0dGhldy5yLndpbGNveEBpbnRlbC5jb20+DQo+ID4gW3Zpc2hh
-bDogQWxzbyByZW1vdmUgdGhlIGRheF9jbGVhcl9zZWN0b3JzIGZ1bmN0aW9uIGVudGlyZWx5XQ0K
-PiA+IFNpZ25lZC1vZmYtYnk6IFZpc2hhbCBWZXJtYSA8dmlzaGFsLmwudmVybWFAaW50ZWwuY29t
-Pg0KPiBKdXN0IHRvIG1ha2Ugc3VyZTrCoMKgdGhlIGV4aXN0aW5nIHNiX2lzc3VlX3plcm91dCBh
-cyBpbiA0LjYtcmMNCj4gaXMgYWxyZWFkeSBkb2luZyB0aGUgcmlnaHQgdGhpbmcgZm9yIERBWD/C
-oMKgSSd2ZSBnb3QgYSBwZW5kaW5nDQo+IHBhdGNoc2V0DQo+IGZvciBYRlMgdGhhdCBpbnRyb2R1
-Y2VzIGFub3RoZXIgZGF4X2NsZWFyX3NlY3RvcnMgdXNlcnMsIGJ1dCBpZiBpdCdzDQo+IGFscmVh
-ZHkgc2FmZSB0byB1c2UgYmxrZGV2X2lzc3VlX3plcm9vdXQgSSBjYW4gc3dpdGNoIHRvIHRoYXQg
-YW5kDQo+IGF2b2lkDQo+IHRoZSBtZXJnZSBjb25mbGljdC4NCg0KSSBiZWxpZXZlIHNvIC0gSmFu
-IGhhcyBtb3ZlZCBhbGwgdW53cml0dGVuIGV4dGVudCBjb252ZXJzaW9ucyBvdXQgb2YNCkRBWCB3
-aXRoIGhpcyBwYXRjaCBzZXQsIGFuZCBJIGJlbGlldmUgemVyb2luZyB0aHJvdWdoIHRoZSBkcml2
-ZXIgaXMNCmFsd2F5cyBmaW5lLiBSb3NzIG9yIEphbiBjb3VsZCBjb25maXJtIHRob3VnaC7CoA==
+On Sun, 2016-05-08 at 01:52 -0700, Christoph Hellwig wrote:
+> On Fri, May 06, 2016 at 03:53:09PM -0600, Vishal Verma wrote:
+> > 
+> > From: Matthew Wilcox <matthew.r.wilcox@intel.com>
+> > 
+> > dax_clear_sectors() cannot handle poisoned blocks.  These must be
+> > zeroed using the BIO interface instead.  Convert ext2 and XFS to
+> > use
+> > only sb_issue_zerout().
+> > 
+> > Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
+> > [vishal: Also remove the dax_clear_sectors function entirely]
+> > Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
+> Just to make sure:  the existing sb_issue_zerout as in 4.6-rc
+> is already doing the right thing for DAX?  I've got a pending
+> patchset
+> for XFS that introduces another dax_clear_sectors users, but if it's
+> already safe to use blkdev_issue_zeroout I can switch to that and
+> avoid
+> the merge conflict.
+
+I believe so - Jan has moved all unwritten extent conversions out of
+DAX with his patch set, and I believe zeroing through the driver is
+always fine. Ross or Jan could confirm though.
diff --git a/a/content_digest b/N2/content_digest
index 1cd4d65..51d8589 100644
--- a/a/content_digest
+++ b/N2/content_digest
@@ -25,24 +25,29 @@
  " jack@suse.cz <jack@suse.cz>\0"
  "\00:1\0"
  "b\0"
- "T24gU3VuLCAyMDE2LTA1LTA4IGF0IDAxOjUyIC0wNzAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90\n"
- "ZToNCj4gT24gRnJpLCBNYXkgMDYsIDIwMTYgYXQgMDM6NTM6MDlQTSAtMDYwMCwgVmlzaGFsIFZl\n"
- "cm1hIHdyb3RlOg0KPiA+IA0KPiA+IEZyb206IE1hdHRoZXcgV2lsY294IDxtYXR0aGV3LnIud2ls\n"
- "Y294QGludGVsLmNvbT4NCj4gPiANCj4gPiBkYXhfY2xlYXJfc2VjdG9ycygpIGNhbm5vdCBoYW5k\n"
- "bGUgcG9pc29uZWQgYmxvY2tzLsKgwqBUaGVzZSBtdXN0IGJlDQo+ID4gemVyb2VkIHVzaW5nIHRo\n"
- "ZSBCSU8gaW50ZXJmYWNlIGluc3RlYWQuwqDCoENvbnZlcnQgZXh0MiBhbmQgWEZTIHRvDQo+ID4g\n"
- "dXNlDQo+ID4gb25seSBzYl9pc3N1ZV96ZXJvdXQoKS4NCj4gPiANCj4gPiBTaWduZWQtb2ZmLWJ5\n"
- "OiBNYXR0aGV3IFdpbGNveCA8bWF0dGhldy5yLndpbGNveEBpbnRlbC5jb20+DQo+ID4gW3Zpc2hh\n"
- "bDogQWxzbyByZW1vdmUgdGhlIGRheF9jbGVhcl9zZWN0b3JzIGZ1bmN0aW9uIGVudGlyZWx5XQ0K\n"
- "PiA+IFNpZ25lZC1vZmYtYnk6IFZpc2hhbCBWZXJtYSA8dmlzaGFsLmwudmVybWFAaW50ZWwuY29t\n"
- "Pg0KPiBKdXN0IHRvIG1ha2Ugc3VyZTrCoMKgdGhlIGV4aXN0aW5nIHNiX2lzc3VlX3plcm91dCBh\n"
- "cyBpbiA0LjYtcmMNCj4gaXMgYWxyZWFkeSBkb2luZyB0aGUgcmlnaHQgdGhpbmcgZm9yIERBWD/C\n"
- "oMKgSSd2ZSBnb3QgYSBwZW5kaW5nDQo+IHBhdGNoc2V0DQo+IGZvciBYRlMgdGhhdCBpbnRyb2R1\n"
- "Y2VzIGFub3RoZXIgZGF4X2NsZWFyX3NlY3RvcnMgdXNlcnMsIGJ1dCBpZiBpdCdzDQo+IGFscmVh\n"
- "ZHkgc2FmZSB0byB1c2UgYmxrZGV2X2lzc3VlX3plcm9vdXQgSSBjYW4gc3dpdGNoIHRvIHRoYXQg\n"
- "YW5kDQo+IGF2b2lkDQo+IHRoZSBtZXJnZSBjb25mbGljdC4NCg0KSSBiZWxpZXZlIHNvIC0gSmFu\n"
- "IGhhcyBtb3ZlZCBhbGwgdW53cml0dGVuIGV4dGVudCBjb252ZXJzaW9ucyBvdXQgb2YNCkRBWCB3\n"
- "aXRoIGhpcyBwYXRjaCBzZXQsIGFuZCBJIGJlbGlldmUgemVyb2luZyB0aHJvdWdoIHRoZSBkcml2\n"
- ZXIgaXMNCmFsd2F5cyBmaW5lLiBSb3NzIG9yIEphbiBjb3VsZCBjb25maXJtIHRob3VnaC7CoA==
+ "On Sun, 2016-05-08 at 01:52 -0700, Christoph Hellwig wrote:\n"
+ "> On Fri, May 06, 2016 at 03:53:09PM -0600, Vishal Verma wrote:\n"
+ "> > \n"
+ "> > From: Matthew Wilcox <matthew.r.wilcox@intel.com>\n"
+ "> > \n"
+ "> > dax_clear_sectors() cannot handle poisoned blocks.\302\240\302\240These must be\n"
+ "> > zeroed using the BIO interface instead.\302\240\302\240Convert ext2 and XFS to\n"
+ "> > use\n"
+ "> > only sb_issue_zerout().\n"
+ "> > \n"
+ "> > Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>\n"
+ "> > [vishal: Also remove the dax_clear_sectors function entirely]\n"
+ "> > Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>\n"
+ "> Just to make sure:\302\240\302\240the existing sb_issue_zerout as in 4.6-rc\n"
+ "> is already doing the right thing for DAX?\302\240\302\240I've got a pending\n"
+ "> patchset\n"
+ "> for XFS that introduces another dax_clear_sectors users, but if it's\n"
+ "> already safe to use blkdev_issue_zeroout I can switch to that and\n"
+ "> avoid\n"
+ "> the merge conflict.\n"
+ "\n"
+ "I believe so - Jan has moved all unwritten extent conversions out of\n"
+ "DAX with his patch set, and I believe zeroing through the driver is\n"
+ always fine. Ross or Jan could confirm though.
 
-682802f1aa7d196ccfec3dd227b3080349b4bf9737251e7f688468b58eed84d7
+805ba727eb7b2ea024414ae8487ccd894756fcf6b421108603697fc2dc84c0ca

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.