From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 4F35B7CDC for ; Tue, 26 Apr 2016 09:59:01 -0500 (CDT) Received: from cuda.sgi.com (cuda1.sgi.com [192.48.157.11]) by relay1.corp.sgi.com (Postfix) with ESMTP id D5B738F8035 for ; Tue, 26 Apr 2016 07:58:57 -0700 (PDT) Received: from mail.kernel.org ([198.145.29.136]) by cuda.sgi.com with ESMTP id hQ0ZaIolKr8QVOLv (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Tue, 26 Apr 2016 07:58:55 -0700 (PDT) Message-ID: <1461682731.26226.20.camel@kernel.org> Subject: Re: [PATCH v2 5/5] dax: handle media errors in dax_do_io From: Vishal Verma Date: Tue, 26 Apr 2016 08:58:51 -0600 In-Reply-To: <20160426004155.GF18496@dastard> References: <1459303190-20072-1-git-send-email-vishal.l.verma@intel.com> <1459303190-20072-6-git-send-email-vishal.l.verma@intel.com> <20160420205923.GA24797@infradead.org> <1461434916.3695.7.camel@intel.com> <20160425083114.GA27556@infradead.org> <1461604476.3106.12.camel@intel.com> <20160425232552.GD18496@dastard> <1461628381.1421.24.camel@intel.com> <20160426004155.GF18496@dastard> Mime-Version: 1.0 List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Dave Chinner , "Verma, Vishal L" Cc: "axboe@fb.com" , "jack@suse.cz" , "linux-nvdimm@ml01.01.org" , "linux-kernel@vger.kernel.org" , "xfs@oss.sgi.com" , "hch@infradead.org" , "linux-mm@kvack.org" , "linux-block@vger.kernel.org" , "viro@zeniv.linux.org.uk" , "linux-fsdevel@vger.kernel.org" , "akpm@linux-foundation.org" , "linux-ext4@vger.kernel.org" , "Wilcox, Matthew R" T24gVHVlLCAyMDE2LTA0LTI2IGF0IDEwOjQxICsxMDAwLCBEYXZlIENoaW5uZXIgd3JvdGU6Cj4g PD4KCj4gPiBUaGUgYXBwbGljYXRpb24gZG9lc24ndCBoYXZlIHRvIHNjYW4gdGhlIGVudGlyZSBm aWxlc3lzdGVtLCBidXQKPiA+IHByZXN1bWFibHkgaXQga25vd3Mgd2hhdCBmaWxlcyBpdCAnb3du cycsIGFuZCBkb2VzIGEgZmllbWFwIGZvcgo+ID4gdGhvc2UuCj4gWW91J3JlIGFzc3VtaW5nIHRo YXQgb25seSB0aGUgREFYIGF3YXJlIGFwcGxpY2F0aW9uIGFjY2Vzc2VzIGl0J3MKPiBmaWxlcy7C oMKgdXNlcnMsIGJhY2t1cCBwcm9ncmFtcywgZGF0YSByZXBsaWNhdG9ycywgZmlsZXlzdGVtCj4g cmUtb3JnYW5pc2VycyAoZS5nLsKgwqBkZWZyYWdtZW50ZXJzKSBldGMgYWxsIG1heSBhY2Nlc3Mg dGhlIGZpbGVzIGFuZAo+IHRoZXkgbWF5IHRocm93IGVycm9ycy4gV2hhdCB0aGVuPwoKSW4gdGhp cyBzY2VuYXJpbywgYmFja3VwIGFwcGxpY2F0aW9ucyBldGMgdGhhdCB0cnkgdG8gcmVhZCB0aGF0 IGRhdGEKYmVmb3JlIGl0IGhhcyBiZWVuIHJlcGxhY2VkIHdpbGwganVzdCBoaXQgdGhlIGVycm9y cyBhbmQgZmFpbC4uCgo+wqAKCjw+Cgo+ID4gVGhlIGRhdGEgdGhhdCB3YXMgbG9zdCBpcyBnb25l IC0tIHRoaXMgYXNzdW1lcyB0aGUgYXBwbGljYXRpb24gaGFzCj4gPiBzb21lCj4gPiBhYmlsaXR5 IHRvIHJlY292ZXIgdXNpbmcgYSBqb3VybmFsL2xvZyBvciBvdGhlciByZWR1bmRhbmN5IC0geWVz LAo+ID4gYXQgdGhlCj4gPiBhcHBsaWNhdGlvbiBsYXllci4gSWYgaXQgZG9lc24ndCBoYXZlIHRo aXMgc29ydCBvZiBjYXBhYmlsaXR5LCB0aGUKPiA+IG9ubHkKPiA+IG9wdGlvbiBpcyB0byByZXN0 b3JlIGZpbGVzIGZyb20gYSBiYWNrdXAvbWlycm9yLgo+IFNvIHRoZSBhcmNoaXRlY3R1cmUgaGFz IGEgYnVpbHQgaW4gYXNzdW1wdGlvbiB0aGF0IG9ubHkgdXNlcnNwYWNlCj4gY2FuIGhhbmRsZSBk YXRhIGxvc3M/Cj4gCj4gV2hhdCBhYm91dCBmaWxlc3l0c2VtcyBsaWtlIE5PVkEsIHRoYXQgdXNl IGxvZyBzdHJ1Y3R1cmVkIGRlc2lnbiB0bwo+IHByb3ZpZGUgREFYIHcvIHVwZGF0ZSBhdG9taWNp dHkgYW5kIGNhbiBwb3RlbnRpYWxseSBhbHNvIHByb3ZpZGUKPiByZWR1bmRhbmN5L3JlcGFpciB0 aHJvdWdoIHRoZSBzYW1lIG1lY2hhbmlzbXM/IFdvbid0IHBtZW0gbmF0aXZlCj4gZmlsZXN5c3Rl bXMgd2l0aCBidWlsdCBpbiBkYXRhIHByb3RlY3Rpb24gZmVhdHVyZXMgbGlrZSB0aGlzIHJlbW92 ZQo+IHRoZSBuZWVkIGZvciBhZGRpbmcgYWxsIHRoaXMgdG8gdXNlcnNwYWNlIGFwcGxpY2F0aW9u cz8KPiAKPiBJZiBzbywgc2hvdWxkbid0IHRoYXQgYmUgdGhlIGZvY3VzIG9mIGRldmVsb3BtZW50 IHJhaHRlciB0aGFuCj4gcGxhY2luZyB0aGUgYnVyZGVuIG9uIHVzZXJzcGFjZSBhcHBzIHRvIGhh bmRsZSBzdG9yYWdlIHJlcGFpcgo+IHNpdHVhdGlvbnM/CgpBZ3JlZWQgdGhhdCBmaWxlIHN5c3Rl bXMgbGlrZSBOT1ZBIGNhbiBiZSBkZXNpZ25lZCB0byBoYW5kbGUgdGhpcwpiZXR0ZXIsIGJ1dCBo YXZlbid0IHlvdSBzYWlkIGluIHRoZSBwYXN0IHRoYXQgaXQgbWF5IHRha2UgeWVhcnMgZm9yIGEK bmV3IGZpbGUgc3lzdGVtIHRvIGJlY29tZSBwcm9kdWN0aW9uIHJlYWR5LCBhbmQgdGhhdCBEQVgg aXMgdGhlIHVudGlsLQp0aGVuIHNvbHV0aW9uIHRoYXQgZ2V0cyB1cyBtb3N0IG9mIHRoZSB3YXkg dGhlcmUuLiBJIHRoaW5rIHdlIGp1c3Qgd2FudAp0byBlbnN1cmUgdGhhdCBjdXJyZW50LURBWCBo YXMgc29tZSB3YXkgdG8gZGVhbCB3aXRoIGVycm9ycywgYW5kIHRoZXNlCnBhdGNoZXMgcHJvdmlk ZSBhbiBhZG1pbi1pbnRlcnZlbnRpb24gcmVjb3ZlcnkgcGF0aCBhbmQgcG9zc2libHkKYW5vdGhl ciBpZiB0aGUgYXBwIHdhbnRzIHRvIHRyeSBzb21ldGhpbmcgZmFuY3kgZm9yIHJlY292ZXJ5LgoK PD4KPiAKPiA+wqAKPiA+IFRvIHN1bW1hcml6ZSwgdGhlIHR3byBjYXNlcyB3ZSB3YW50IHRvIGhh bmRsZSBhcmU6Cj4gPiAxLiBBcHBsaWNhdGlvbiBoYXMgaW5idWlsdCByZWNvdmVyeToKPiA+IMKg IC0gaGl0cyBiYWRibG9jawo+ID4gwqAgLSBmaWd1cmVzIG91dCBpdCBpcyBhYmxlIHRvIHJlY292 ZXIgdGhlIGRhdGEKPiA+IMKgIC0gaGFuZGxlcyBTSUdCVVMgb3IgRUlPCj4gPiDCoCAtIGRvZXMg YSAoc2VjdG9yIGFsaWduZWQpIHdyaXRlKCkgdG8gcmVzdG9yZSB0aGUgZGF0YQo+IFRoZSAiZmln dXJlcyBvdXQiIHN0ZXAgaGVyZSBpcyB3aGVyZSA+OTUlIG9mIHRoZSB3b3JrIHdlJ2QgaGF2ZSB0 bwo+IGRvIGlzLiBBbmQgdGhhdCdzIGluIGZpbGVzeXN0ZW0gYW5kIGJsb2NrIGxheWVyIGNvZGUs IG5vdAo+IHVzZXJzcGFjZSwgYW5kIHVzZXJzcGFjZSBjYW4ndCBkbyB0aGF0IHdvcmsgaW4gYSBz aWduYWwgaGFuZGxlci4KPiBBbmQgaXTCoMKgY2FuIHN0aWxsIGZhbGwgZG93biB0byB0aGUgc2Vj b25kIGNhc2Ugd2hlbiB0aGUgYXBwbGljYXRpb24KPiBkb2Vzbid0IGhhdmUgYW5vdGhlciBjb3B5 IG9mIHRoZSBkYXRhIHNvbWV3aGVyZS4KCkFoIHdoZW4gSSBzYWlkICJmaWd1cmVzIG91dCIgSSB3 YXMgb25seSB0aGlua2luZyBpZiB0aGUgYXBwbGljYXRpb24gaGFzCnNvbWUgcmVkdW5kYW5jeS9q b3VyYW5sbGluZywgYW5kIGlmIGl0IGNhbiByZWNvdmVyIHVzaW5nIHRoYXQgLS0gbm90CmFkZGl0 aW9uYWwgcmVjb3ZlcnkgbWVjaGFuaXNtcyBhdCB0aGUgYmxvY2svZnMgbGF5ZXIuCgo+IAo+IEZX SVcsIHdlIGRvbid0IGhhdmUgYSBEQVggZW5hYmxlZCBmaWxlc3lzdGVtIHRoYXQgY2FuIGRvCj4g cmV2ZXJzZSBibG9jayBtYXBwaW5nLCBzbyB3ZSdyZSBhIHllYXIgb3IgdHdvIGF3YXkgZnJvbSB0 aGlzIGJlaW5nIGEKPiB3b3JrYWJsZSBwcm9kdWN0aW9uIHNvbHV0aW9uIGZyb20gdGhlIGZpbGVz eXN0ZW0gcGVyc3BlY3RpdmUuIEFuZAo+IEFGQUlDVCwgaXQncyBub3QgZXZlbiBvbiB0aGUgcm9h ZG1hcCBmb3IgZG0vbWQgbGF5ZXJzLgo+IAo+ID4gCj4gPiAyLiBBcHBsaWNhdGlvbiBkb2Vzbid0 IGhhdmUgYW55IGluYnVpbHQgcmVjb3ZlcnkgbWVjaGFuaXNtCj4gPiDCoCAtIGhpdHMgYmFkYmxv Y2sKPiA+IMKgIC0gZ2V0cyBTSUdCVVMgKG9yIEVJTykgYW5kIGNyYXNoZXMKPiA+IMKgIC0gU3lz YWRtaW4gcmVzdG9yZXMgZmlsZSBmcm9tIGJhY2t1cAo+IFdoaWNoIGlzIG5vIGRpZmZlcmVudCB0 byBhbiBleGlzdGluZyBub24tREFYIGFwcGxpY2F0aW9uIGdldHRpbmcgYW4KPiBFSU8vc2lnYnVz IGZyb20gY3VycmVudCBzdG9yYWdlIHRlY2hub2xvZ2llcy4KPiAKPiBFeGNlcHQ6IGluIHRoZSBl eGlzdGluZyBzdG9yYWdlIHN0YWNrLCByZWR1bmRhbmN5IGFuZCBjb3JyZWN0aW9uIGhhcwo+IGFs cmVhZHkgaGFkIHRvIGhhdmUgZmFpbGVkIGZvciB0aGUgYXBwbGljYXRpb24gdG8gc2VlIHN1Y2gg YW4gZXJyb3IuCj4gSGVuY2UgdGhpcyBpcyBub3JtYWxseSBjb25zaWRlcmVkIGEgRFIgY2FzZSBh cyB0aGVyZSdzIGhhZCB0byBiZQo+IGNhc2NhZGluZyBmYWlsdXJlcyAoZS5nLsKgwqBtdWx0aXBs ZSBkaXNrIGZhaWx1cmVzIGluIGEgUkFJRCkgdG8gZ2V0Cj4gdG8gdGhpcyBzdGFnZSwgbm90IGEg c2luZ2xlIGVycm9yIGluIGEgc2luZ2xlIHNlY3RvciBpbgo+IG5vbi1yZWR1bmRhbnQgc3RvcmFn ZS4KPiAKPiBXZSBuZWVkIHNvbWUgZm9ybSBvZiByZWR1bmRhbmN5IGFuZCBjb3JyZWN0aW9uIGlu IHRoZSBQTUVNIHN0YWNrIHRvCj4gcHJldmVudCBzaW5nbGUgc2VjdG9yIGVycm9ycyBmcm9tIHRh a2luZyBkb3duIHNlcnZpY2VzIHVudGlsIGFuCj4gYWRtaW5pc3RyYXRvciBjYW4gY29ycmVjdCB0 aGUgcHJvYmxlbS4gSSdtIHRyeWluZyB0byB1bmRlcnN0YW5kCj4gd2hlcmUgdGhpcyBpcyBzdXBw b3NlZCB0byBmaXQgaW50byB0aGUgcGljdHVyZSAtIGF0IHRoaXMgcG9pbnQgSQo+IHJlYWxseSBk b24ndCB0aGluayB1c2Vyc3BhY2UgYXBwbGljYXRpb25zIGFyZSBnb2luZyB0byBiZSBhYmxlIHRv IGRvCj4gdGhpcyByZWxpYWJseS4uLi4KCkFncmVlZCB0aGF0IHRoZSBwbWVtIHN0YWNrIGNvdWxk IHVzZSBtb3JlIHJlZHVuZGFuY3kgYW5kIGVycm9yCmNvcnJlY3Rpb24sIHBlcmhhcHMgZW5hYmxp bmcgbWQtcmFpZCB0byByYWlkIHBtZW0gZGV2aWNlcyBhbmQgdGhlbgplbmFibGUgREFYIG9uIHRv cCBvZiB0aGF0IGFuZCB3ZSdsbCBoYXZlIGEgYmV0dGVyIGNoYW5jZSB0byBoYW5kbGUKZXJyb3Jz LCBidXQgdGhhdCBsZXZlbCBvZiByZWNvdmVyeSBpc24ndCB3aGF0IHRoZXNlIHBhdGNoZXMgYXJl IGFpbWluZwpmb3IgLS0gdGhhdCBpcyBvYnZpb3VzbHkgYSBsb25nZXIgdGVybSBlZmZvcnQuIFRo ZXNlIHNpbXBseSBhaW0gdG8KcHJvdmlkZSB0aGF0IGRpc2FzdGVyIHJlY292ZXJ5IHBhdGggd2hl biBhIHNpbmdsZSBzZWN0b3IgZmFpbHVyZSBkb2VzCnRha2UgZG93biB0aGUgc2VydmljZS4KClRv ZGF5LCBvbiBhIGRheCBlbmFibGVkIGZpbGVzeXN0ZW0sIGlmL3doZW4gdGhlIGFwcCBoaXRzIGFu IGVycm9yIGFuZApjcmFzaGVzLCBkYXggaXMgc2ltcGx5IGRpc2FibGVkIHRpbGwgdGhlIGVycm9y cyBhcmUgZ29uZS4gVGhpcyBpcwpvYnZpb3VzbHkgbGVzcyB0aGFuIGlkZWFsLiAoVGhpcyB3YXMg ZG9uZSBiZWNhdXNlIHRoZXJlIGlzIGN1cnJlbnRseSBubwp3YXkgZm9yIGEgREFYIGZpbGUgc3lz dGVtIHRvIHNlbmQgYW55IElPIC0gbW1hcCBvciBvdGhlcndpc2UgLSB0aHJvdWdoCnRoZSBkcml2 ZXIsIGluY2x1ZGluZyB6ZXJvaW5nIG9mIG5ldyBmcyBibG9ja3MpLiBUaGVzZSBwYXRjaGVzIGVu YWJsZQp0aGUgRFIgcGF0aCBieSBhbGxvd2luZyBzb21lIG5vbi1tbWFwIElPIChtb3N0IGltcG9y dGFudGx5IHplcm9pbmcpIHRvCmdvIHRocm91Z2ggdGhlIGRyaXZlciB3aGljaCBjYW4gdGVsbCB0 aGUgZGV2aWNlIHRvIGRvIHNvbWUgcmVtYXBwaW5nCmV0Yy4KClNvLCB5ZXMsIHRoaXMgaXMgdmVy eSBtdWNoIGEgRFIgY2FzZSBpbiBvdXIgY3VycmVudCBwbWVtK2RheAphcmNoaXRlY3R1cmUsIGFu ZCB3ZSBzaG91bGQgcHJvYmFibHkgZGVzaWduIG1vcmUgcm9idXN0IGhhbmRsaW5nIGF0IHRoZQpi bG9jay9tZC9mcyBsYXllciwgYnV0IHdpdGggdGhlc2UsIHlvdSBhdCBsZWFzdCBnZXQgdG8gY3Jh c2ggdGhlIGFwcCwKZGVsZXRlIGl0cyBmaWxlcyBhbmQgcmVzdG9yZSB0aGVtIGZyb20gb3V0LW9m LWJhbmQgYmFja3VwcyBhbmQgY29udGludWUKd2l0aCBEQVguCgo+IAo+IENoZWVycywKPiAKPiBE YXZlLgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KeGZz IG1haWxpbmcgbGlzdAp4ZnNAb3NzLnNnaS5jb20KaHR0cDovL29zcy5zZ2kuY29tL21haWxtYW4v bGlzdGluZm8veGZzCg==