From: Trond Myklebust <trondmy@primarydata.com>
To: "aweits@rit.edu" <aweits@rit.edu>
Cc: bcodding redhat <bcodding@redhat.com>,
"anna.schumaker@netapp.com" <anna.schumaker@netapp.com>,
"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v8 00/11] Fix OPEN/CLOSE races
Date: Tue, 7 Nov 2017 18:35:56 +0000 [thread overview]
Message-ID: <1510079753.7834.7.camel@primarydata.com> (raw)
In-Reply-To: <m2y3ninhxs.fsf@discipline.rit.edu>
T24gVHVlLCAyMDE3LTExLTA3IGF0IDExOjU5IC0wNTAwLCBBbmRyZXcgVyBFbGJsZSB3cm90ZToN
Cj4gVHJvbmQgTXlrbGVidXN0IDx0cm9uZG15QHByaW1hcnlkYXRhLmNvbT4gd3JpdGVzOg0KPiAN
Cj4gPiBPbiBNb24sIDIwMTctMTEtMDYgYXQgMTc6NDYgLTA1MDAsIEFuZHJldyBXIEVsYmxlIHdy
b3RlOg0KPiA+ID4gVHJvbmQgTXlrbGVidXN0IDx0cm9uZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEu
Y29tPiB3cml0ZXM6DQo+ID4gPiANCj4gPiA+ID4gdjg6DQo+ID4gPiA+IC0gUmVhbGx5IGZpeCBj
b21waWxlIGlzc3VlIHdoZW4gQ09ORklHX05GU19WNF8xPW4NCj4gPiA+ID4gLSBuZnNfaW5vZGVf
ZmluZF9zdGF0ZV9hbmRfcmVjb3ZlcigpIHNob3VsZCBhbHNvIHRyeSB0byBtYXRjaA0KPiA+ID4g
PiB0aGUNCj4gPiA+ID4gICBvcGVuX3N0YXRlaWQuDQo+ID4gPiANCj4gPiA+IFNlZWluZyBhIGxv
dCBvZiBURVNUX1NUQVRFSUQncyBmb3IgdGhlIGludmFsaWQgc3RhdGVpZCBnbyBvdmVyDQo+ID4g
PiB0aGUNCj4gPiA+IHdpcmUgLSB0aGlzIGFsc28gbWFrZXMgdGhlIHNlcnZlciBoaXQgdGhpcyBx
dWl0ZSBhIGJpdDoNCj4gPiA+IA0KPiA+ID4gcHJfd2Fybl9yYXRlbGltaXRlZCgiTkZTRDogY2xp
ZW50ICVzIHRlc3Rpbmcgc3RhdGUgSUQgIg0KPiA+ID4gICAgICAgICAgICAgICAgICAgICAgICAg
IndpdGggaW5jb3JyZWN0IGNsaWVudCBJRFxuIiwgYWRkcl9zdHIpOw0KPiA+ID4gDQo+ID4gDQo+
ID4gSSdtIG5vdCBzZWVpbmcgdGhhdCBhdCBhbGwuIENhbiB5b3UgcGxlYXNlIGVsYWJvcmF0ZSBv
biB3aGljaA0KPiA+IHNlcnZlcg0KPiA+IGVycm9ycyBhcmUgdHJpZ2dlcmluZyB0aGlzPyBJJ2Qg
bm90IGV4cGVjdCB0byBldmVyIHNlZSBURVNUX1NUQVRFSUQNCj4gPiBvbg0KPiA+IGEgbm9ybWFs
IHJ1bi4NCj4gDQo+IEhvcGVmdWxseSB0aGlzIGhlbHBzLg0KPiANCj4gTm90IGNvbXBsZXRlbHkg
c3VyZSB0aGlzIGlzIGNvcnJlY3QsIGJ1dCB0aGlzIG1ha2VzIHRoZSBjbGllbnQgbm8NCj4gbG9u
Z2VyDQo+IHRlc3QgdGhlIF9pbnZhbGlkXyBzdGF0ZWlkOg0KPiANCj4gZGlmZiAtLWdpdCBhL2Zz
L25mcy9uZnM0cHJvYy5jIGIvZnMvbmZzL25mczRwcm9jLmMNCj4gaW5kZXggNmVjY2JkMWI5OGNj
Li43ZGM5MTRhNjM1NmYgMTAwNjQ0DQo+IC0tLSBhL2ZzL25mcy9uZnM0cHJvYy5jDQo+ICsrKyBi
L2ZzL25mcy9uZnM0cHJvYy5jDQo+IEBAIC0xNDM2LDYgKzE0MzYsOCBAQCBzdGF0aWMgdm9pZA0K
PiBuZnNfY2xlYXJfb3Blbl9zdGF0ZWlkX2xvY2tlZChzdHJ1Y3QgbmZzNF9zdGF0ZSAqc3RhdGUs
DQo+ICAgICAgICAgICAgICAgICBuZnNfcmVzeW5jX29wZW5fc3RhdGVpZF9sb2NrZWQoc3RhdGUp
Ow0KPiAgICAgICAgICAgICAgICAgZ290byBvdXQ7DQo+ICAgICAgICAgfQ0KPiArICAgICAgIGlm
ICh0ZXN0X2JpdChORlNfT1BFTl9TVEFURSwgJnN0YXRlLT5mbGFncykgPT0gMCkNCj4gKyAgICAg
ICAgICAgICAgIHN0YXRlaWQtPnR5cGUgPSBORlM0X0lOVkFMSURfU1RBVEVJRF9UWVBFOw0KPiAg
ICAgICAgIGlmICh0ZXN0X2JpdChORlNfREVMRUdBVEVEX1NUQVRFLCAmc3RhdGUtPmZsYWdzKSA9
PSAwKQ0KPiAgICAgICAgICAgICAgICAgbmZzNF9zdGF0ZWlkX2NvcHkoJnN0YXRlLT5zdGF0ZWlk
LCBzdGF0ZWlkKTsNCj4gICAgICAgICBuZnM0X3N0YXRlaWRfY29weSgmc3RhdGUtPm9wZW5fc3Rh
dGVpZCwgc3RhdGVpZCk7DQo+IA0KPiANCj4gVGhlIFRFU1RfU1RBVEVJRCdzIGFyZSBiZWluZyBp
c3N1ZWQgZnJvbSB0aGUgImZyZWVtZSIgcGF0aCBvZg0KPiB1cGRhdGVfb3Blbl9zdGF0ZWlkKCkg
dmlhIG5mczRfb3BlbmRhdGFfdG9fbmZzNF9zdGF0ZSgpLiBJIHB1dCBhDQo+IHRyYWNlcG9pbnQN
Cj4gdGhlcmU6DQoNCkFoLCBJIHNlZS4gQ2FuIHdlIGluIHRoYXQgY2FzZSBwbGVhc2UgcmF0aGVy
IGRvIHRoZSBmb2xsb3dpbmc/DQoNCjg8LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0NCkZyb20gZTNlZDIxYzE5NGY0MjA3Y2YzYTAwYjYyNGZl
ODZiOWE5YjE4OGMxMyBNb24gU2VwIDE3IDAwOjAwOjAwIDIwMDENCkZyb206IFRyb25kIE15a2xl
YnVzdCA8dHJvbmQubXlrbGVidXN0QHByaW1hcnlkYXRhLmNvbT4NCkRhdGU6IFR1ZSwgNyBOb3Yg
MjAxNyAxMzoxMDo0NiAtMDUwMA0KU3ViamVjdDogW1BBVENIIDEvMl0gTkZTdjQ6IG5mc19zZXRf
b3Blbl9zdGF0ZWlkIG11c3Qgbm90IHRyaWdnZXIgc3RhdGUNCiByZWNvdmVyeSBmb3IgY2xvc2Vk
IHN0YXRlDQoNCkluIG5mc19zZXRfb3Blbl9zdGF0ZWlkX2xvY2tlZCwgd2UgbXVzdCBpZ25vcmUg
c3RhdGVpZHMgZnJvbSBjbG9zZWQgc3RhdGUuDQoNClJlcG9ydGVkLWJ5OiBBbmRyZXcgVyBFbGJs
ZSA8YXdlaXRzQHJpdC5lZHU+DQpTaWduZWQtb2ZmLWJ5OiBUcm9uZCBNeWtsZWJ1c3QgPHRyb25k
Lm15a2xlYnVzdEBwcmltYXJ5ZGF0YS5jb20+DQotLS0NCiBmcy9uZnMvbmZzNHByb2MuYyB8IDMg
KystDQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwgMSBkZWxldGlvbigtKQ0KDQpk
aWZmIC0tZ2l0IGEvZnMvbmZzL25mczRwcm9jLmMgYi9mcy9uZnMvbmZzNHByb2MuYw0KaW5kZXgg
ZDUzNmQ0ZWQ5OTBmLi4wYWE4NDYxZTczNDEgMTAwNjQ0DQotLS0gYS9mcy9uZnMvbmZzNHByb2Mu
Yw0KKysrIGIvZnMvbmZzL25mczRwcm9jLmMNCkBAIC0xNTQxLDcgKzE1NDEsOCBAQCBzdGF0aWMg
dm9pZCBuZnNfc2V0X29wZW5fc3RhdGVpZF9sb2NrZWQoc3RydWN0IG5mczRfc3RhdGUgKnN0YXRl
LA0KIAkJd3JpdGVfc2VxbG9jaygmc3RhdGUtPnNlcWxvY2spOw0KIAl9DQogDQotCWlmICghbmZz
NF9zdGF0ZWlkX21hdGNoX290aGVyKHN0YXRlaWQsICZzdGF0ZS0+b3Blbl9zdGF0ZWlkKSkgew0K
KwlpZiAodGVzdF9iaXQoTkZTX09QRU5fU1RBVEUsICZzdGF0ZS0+ZmxhZ3MpICYmDQorCSAgICAh
bmZzNF9zdGF0ZWlkX21hdGNoX290aGVyKHN0YXRlaWQsICZzdGF0ZS0+b3Blbl9zdGF0ZWlkKSkg
ew0KIAkJbmZzNF9zdGF0ZWlkX2NvcHkoZnJlZW1lLCAmc3RhdGUtPm9wZW5fc3RhdGVpZCk7DQog
CQluZnNfdGVzdF9hbmRfY2xlYXJfYWxsX29wZW5fc3RhdGVpZChzdGF0ZSk7DQogCX0NCi0tIA0K
Mi4xMy42DQoNCi0tIA0KVHJvbmQgTXlrbGVidXN0DQpMaW51eCBORlMgY2xpZW50IG1haW50YWlu
ZXIsIFByaW1hcnlEYXRhDQp0cm9uZC5teWtsZWJ1c3RAcHJpbWFyeWRhdGEuY29tDQo=
next prev parent reply other threads:[~2017-11-07 18:35 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-06 20:28 [PATCH v8 00/11] Fix OPEN/CLOSE races Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 01/11] NFSv4: Fix OPEN / CLOSE race Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 02/11] NFSv4: Add a tracepoint to document open stateid updates Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 03/11] NFSv4: Fix open create exclusive when the server reboots Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 04/11] NFS: Fix a typo in nfs_rename() Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 05/11] NFSv4: Retry CLOSE and DELEGRETURN on NFS4ERR_OLD_STATEID Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 06/11] NFSv4: Don't try to CLOSE if the stateid 'other' field has changed Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 07/11] pNFS: Retry NFS4ERR_OLD_STATEID errors in layoutreturn-on-close Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 08/11] NFSv4: Retry NFS4ERR_OLD_STATEID errors in layoutreturn Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 09/11] NFSv4: cleanup nfs4_close_done Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 10/11] NFSv4: Clean up nfs4_delegreturn_done Trond Myklebust
2017-11-06 20:28 ` [PATCH v8 11/11] NFSv4: Check the open stateid when searching for expired state Trond Myklebust
2017-11-06 22:46 ` [PATCH v8 00/11] Fix OPEN/CLOSE races Andrew W Elble
2017-11-06 22:50 ` Trond Myklebust
2017-11-06 23:13 ` Andrew W Elble
2017-11-07 16:59 ` Andrew W Elble
2017-11-07 18:35 ` Trond Myklebust [this message]
2017-11-07 19:00 ` Trond Myklebust
2017-11-07 23:03 ` Andrew W Elble
2017-11-07 23:44 ` Trond Myklebust
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=1510079753.7834.7.camel@primarydata.com \
--to=trondmy@primarydata.com \
--cc=anna.schumaker@netapp.com \
--cc=aweits@rit.edu \
--cc=bcodding@redhat.com \
--cc=linux-nfs@vger.kernel.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.