From: "Myklebust, Trond" <Trond.Myklebust@netapp.com>
To: David Howells <dhowells@redhat.com>
Cc: "Schumaker, Bryan" <Bryan.Schumaker@netapp.com>,
"sct@redhat.com" <sct@redhat.com>,
"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>,
"steved@redhat.com" <steved@redhat.com>,
"jlayton@redhat.com" <jlayton@redhat.com>
Subject: Re: [PATCH 1/2] NFS: Fix a number of bugs in the idmapper
Date: Tue, 31 Jul 2012 14:48:38 +0000 [thread overview]
Message-ID: <1343746117.5528.3.camel@lade.trondhjem.org> (raw)
In-Reply-To: <20120731144234.6664.85664.stgit@warthog.procyon.org.uk>
T24gVHVlLCAyMDEyLTA3LTMxIGF0IDE1OjQyICswMTAwLCBEYXZpZCBIb3dlbGxzIHdyb3RlOg0K
PiBGaXggYSBudW1iZXIgb2YgYnVncyBpbiB0aGUgTkZTIGlkbWFwcGVyIGNvZGU6DQo+IA0KPiAg
KDEpIE9ubHkgcmVnaXN0ZXJlZCBrZXkgdHlwZXMgY2FuIGJlIHBhc3NlZCB0byB0aGUgY29yZSBr
ZXlzIGNvZGUsIHNvDQo+ICAgICAgcmVnaXN0ZXIgdGhlIGxlZ2FjeSBpZG1hcHBlciBrZXkgdHlw
ZS4NCj4gDQo+ICAgICAgVGhpcyBpcyBhIHJlcXVpcmVtZW50IGJlY2F1c2UgdGhlIHVucmVnaXN0
ZXIgZnVuY3Rpb24gY2xlYW5zIHVwIGtleXMNCj4gICAgICBiZWxvbmdpbmcgdG8gdGhhdCBrZXkg
dHlwZSBzbyB0aGF0IHRoZXJlIGFyZW4ndCBkYW5nbGluZyBwb2ludGVycyB0byB0aGUNCj4gICAg
ICBtb2R1bGUgbGVmdCBiZWhpbmQgLSBpbmNsdWRpbmcgdGhlIGtleS0+dHlwZSBwb2ludGVyLg0K
PiANCj4gICgyKSBSZW5hbWUgdGhlIGxlZ2FjeSBrZXkgdHlwZS4gIFlvdSBjYW4ndCBoYXZlIHR3
byBrZXkgdHlwZXMgd2l0aCB0aGUgc2FtZQ0KPiAgICAgIG5hbWUsIGFuZCAoMSkgd291bGQgb3Ro
ZXJ3aXNlIHJlcXVpcmUgdGhhdC4NCj4gDQo+ICAoMykgY29tcGxldGVfcmVxdWVzdF9rZXkoKSBt
dXN0IGJlIGNhbGxlZCBpbiB0aGUgZXJyb3IgcGF0aCBvZg0KPiAgICAgIG5mc19pZG1hcF9sZWdh
Y3lfdXBjYWxsKCkuDQo+IA0KPiAgKDQpIFRoZXJlIGlzIG9uZSBpZG1hcCBzdHJ1Y3QgZm9yIGVh
Y2ggbmZzX2NsaWVudCBzdHJ1Y3QuICBUaGlzIG1lYW5zIHRoYXQNCj4gICAgICBpZG1hcC0+aWRt
YXBfa2V5X2NvbnMgaXMgc2hhcmVkIHdpdGhvdXQgdGhlIHVzZSBvZiBhIGxvY2suICBUaGlzIGlz
IGENCj4gICAgICBwcm9ibGVtIGJlY2F1c2Uga2V5X2luc3RhbnRpYXRlX2FuZF9saW5rKCkgLSBh
cyBjYWxsZWQgaW5kaXJlY3RseSBieQ0KPiAgICAgIGlkbWFwX3BpcGVfZG93bmNhbGwoKSAtIHJl
bGVhc2VzIGFueW9uZSB3YWl0aW5nIGZvciB0aGUga2V5IHRvIGJlDQo+ICAgICAgaW5zdGFudGlh
dGVkLg0KPiANCj4gICAgICBXaGF0IGhhcHBlbnMgaXMgdGhhdCBpZG1hcF9waXBlX2Rvd25jYWxs
KCkgcnVubmluZyBpbiB0aGUgcnBjLmlkbWFwZA0KPiAgICAgIHRocmVhZCwgcmVsZWFzZXMgdGhl
IE5GUyBmaWxlc3lzdGVtIGluIHdoYXRldmVyIHRocmVhZCB0aGF0IGlzIHJ1bm5pbmcgaW4NCj4g
ICAgICB0byBjb250aW51ZS4gIFRoaXMgbWF5IHRoZW4gbWFrZSBhbm90aGVyIGlkbWFwcGVyIGNh
bGwsIG92ZXJ3cml0aW5nDQo+ICAgICAgaWRtYXBfa2V5X2NvbnMgYmVmb3JlIGlkbWFwX3BpcGVf
ZG93bmNhbGwoKSBnZXRzIHRoZSBjaGFuY2UgdG8gY2FsbA0KPiAgICAgIGNvbXBsZXRlX3JlcXVl
c3Rfa2V5KCkuDQo+IA0KPiAgICAgIEkgKnRoaW5rKiB0aGF0IHJlYWRpbmcgaWRtYXBfa2V5X2Nv
bnMgb25seSBvbmNlLCBiZWZvcmUNCj4gICAgICBrZXlfaW5zdGFudGlhdGVfYW5kX2xpbmsoKSBp
cyBjYWxsZWQsIGFuZCB0aGVuIGNhY2hpbmcgdGhlIHJlc3VsdCBpbiBhDQo+ICAgICAgdmFyaWFi
bGUgaXMgc3VmZmljaWVudC4NCj4gDQo+IEJ1ZyAoNCkgaXMgdGhlIGNhdXNlIG9mOg0KPiANCj4g
QlVHOiB1bmFibGUgdG8gaGFuZGxlIGtlcm5lbCBOVUxMIHBvaW50ZXIgZGVyZWZlcmVuY2UgYXQg
ICAgICAgICAgIChudWxsKQ0KPiBJUDogWzwgICAgICAgICAgKG51bGwpPl0gICAgICAgICAgIChu
dWxsKQ0KPiBQR0QgMA0KPiBPb3BzOiAwMDEwIFsjMV0gU01QDQo+IENQVSAxDQo+IE1vZHVsZXMg
bGlua2VkIGluOiBwcGRldiBwYXJwb3J0X3BjIGxwIHBhcnBvcnQgaXA2dGFibGVfZmlsdGVyIGlw
Nl90YWJsZXMgZWJ0YWJsZV9uYXQgZWJ0YWJsZXMgaXB0X01BU1FVRVJBREUgaXB0YWJsZV9uYXQg
bmZfbmF0IG5mX2Nvbm50cmFja19pcHY0IG5mX2RlZnJhZ19pcHY0IHh0X3N0YXRlIG5mX2Nvbm50
cmFjayBuZnMgZnNjYWNoZSB4dF9DSEVDS1NVTSBhdXRoX3JwY2dzcyBpcHRhYmxlX21hbmdsZSBu
ZnNfYWNsIGJyaWRnZSBzdHAgbGxjIGxvY2tkIGJlMmlzY3NpIGlzY3NpX2Jvb3Rfc3lzZnMgYm54
MmkgY25pYyB1aW8gY3hnYjRpIGN4Z2I0IGN4Z2IzaSBsaWJjeGdiaSBjeGdiMyBtZGlvIGliX2lz
ZXIgcmRtYV9jbSBpYl9jbSBpd19jbSBpYl9zYSBpYl9tYWQgaWJfY29yZSBpYl9hZGRyIGlzY3Np
X3RjcCBsaWJpc2NzaV90Y3AgbGliaXNjc2kgc2NzaV90cmFuc3BvcnRfaXNjc2kgc25kX2hkYV9j
b2RlY19yZWFsdGVrIHNuZF91c2JfYXVkaW8gc25kX2hkYV9pbnRlbCBzbmRfaGRhX2NvZGVjIHNu
ZF9zZXEgc25kX3BjbSBzbmRfaHdkZXAgc25kX3VzYm1pZGlfbGliIHNuZF9yYXdtaWRpIHNuZF90
aW1lciB1dmN2aWRlbyB2aWRlb2J1ZjJfY29yZSB2aWRlb2RldiBtZWRpYSB2aWRlb2J1ZjJfdm1h
bGxvYyBzbmRfc2VxX2RldmljZSB2aWRlb2J1ZjJfbWVtb3BzIGUxMDAwZSB2aG9zdF9uZXQgaVRD
T193ZHQgam95ZGV2IGNvcmV0ZW1wIHNuZCBzb3VuZGNvcmUgbWFjdnRhcCBtYWN2bGFuIGkyY19p
ODAxIHNuZF9wYWdlX2FsbG9jIHR1biBpVENPX3ZlbmRvcl9zdXBwb3J0IG1pY3JvY29kZSBrdm1f
aW50ZWwga3ZtIHN1bnJwYyBoaWRfbG9naXRlY2hfZGogdXNiX3N0b3JhZ2UgaTkxNSBkcm1fa21z
X2hlbHBlciBkcm0gaTJjX2FsZ29fYml0IGkyY19jb3JlIHZpZGVvIFtsYXN0IHVubG9hZGVkOiBz
Y3NpX3dhaXRfc2Nhbl0NCj4gUGlkOiAxMjI5LCBjb21tOiBycGMuaWRtYXBkIE5vdCB0YWludGVk
IDMuNC4yLTEuZmMxNi54ODZfNjQgIzEgR2F0ZXdheSBEWDQ3MTAtVUI4MDFBL0czM00wNUcxDQo+
IFJJUDogMDAxMDpbPDAwMDAwMDAwMDAwMDAwMDA+XSAgWzwgICAgICAgICAgKG51bGwpPl0gICAg
ICAgICAgIChudWxsKQ0KPiBSU1A6IDAwMTg6ZmZmZjg4MDFhMzY0NWQ0MCAgRUZMQUdTOiAwMDAx
MDI0Ng0KPiBSQVg6IGZmZmY4ODAwNzc3MDdlMzAgUkJYOiBmZmZmODgwMDc3NzA3ZjUwIFJDWDog
ZmZmZjg4MDFhMThjY2Q4MA0KPiBSRFg6IDAwMDAwMDAwMDAwMDAwMDYgUlNJOiBmZmZmODgwMWEz
NjQ1ZTc1IFJESTogZmZmZjg4MDA3NzcwN2Y1MA0KPiBSQlA6IGZmZmY4ODAxYTM2NDVkODggUjA4
OiBmZmZmODgwMWE0MzBmOWMwIFIwOTogZmZmZjg4MDFhMzY0NWRiMA0KPiBSMTA6IDAwMDAwMDAw
MDAwMDAwMGEgUjExOiAwMDAwMDAwMDAwMDAwMjQ2IFIxMjogZmZmZjg4MDFhMThjY2Q4MA0KPiBS
MTM6IGZmZmY4ODAxYTM2NDVlNzUgUjE0OiBmZmZmODgwMWE0MzBmOWMwIFIxNTogMDAwMDAwMDAw
MDAwMDAwNg0KPiBGUzogIDAwMDA3ZmI2ZmI1MWE3MDAoMDAwMCkgR1M6ZmZmZjg4MDFhZmM4MDAw
MCgwMDAwKSBrbmxHUzowMDAwMDAwMDAwMDAwMDAwDQo+IENTOiAgMDAxMCBEUzogMDAwMCBFUzog
MDAwMCBDUjA6IDAwMDAwMDAwODAwNTAwMzMNCj4gQ1IyOiAwMDAwMDAwMDAwMDAwMDAwIENSMzog
MDAwMDAwMDFhNDliMDAwMCBDUjQ6IDAwMDAwMDAwMDAwMDI3ZTANCj4gRFIwOiAwMDAwMDAwMDAw
MDAwMDAwIERSMTogMDAwMDAwMDAwMDAwMDAwMCBEUjI6IDAwMDAwMDAwMDAwMDAwMDANCj4gRFIz
OiAwMDAwMDAwMDAwMDAwMDAwIERSNjogMDAwMDAwMDBmZmZmMGZmMCBEUjc6IDAwMDAwMDAwMDAw
MDA0MDANCj4gUHJvY2VzcyBycGMuaWRtYXBkIChwaWQ6IDEyMjksIHRocmVhZGluZm8gZmZmZjg4
MDFhMzY0NDAwMCwgdGFzayBmZmZmODgwMWEzYmY5NzEwKQ0KPiBTdGFjazoNCj4gIGZmZmZmZmZm
ODEyNjA4NzggZmZmZjg4MDFhMzY0NWRiMCBmZmZmODgwMWEzNjQ1ZGIwIGZmZmY4ODAwNzc3MDdh
OTANCj4gIGZmZmY4ODAwNzc3MDdmNTAgZmZmZjg4MDFhMThjY2Q4MCAwMDAwMDAwMDAwMDAwMDA2
IGZmZmY4ODAxYTM2NDVlNzUNCj4gIGZmZmY4ODAxYTQzMGY5YzAgZmZmZjg4MDFhMzY0NWRkOCBm
ZmZmZmZmZjgxMjYwOTgzIGZmZmY4ODAxYTM2NDVkZTgNCj4gQ2FsbCBUcmFjZToNCj4gIFs8ZmZm
ZmZmZmY4MTI2MDg3OD5dID8gX19rZXlfaW5zdGFudGlhdGVfYW5kX2xpbmsrMHg1OC8weDEwMA0K
PiAgWzxmZmZmZmZmZjgxMjYwOTgzPl0ga2V5X2luc3RhbnRpYXRlX2FuZF9saW5rKzB4NjMvMHhh
MA0KPiAgWzxmZmZmZmZmZmEwNTcwNjJiPl0gaWRtYXBfcGlwZV9kb3duY2FsbCsweDFjYi8weDFl
MCBbbmZzXQ0KPiAgWzxmZmZmZmZmZmEwMTA3ZjU3Pl0gcnBjX3BpcGVfd3JpdGUrMHg2Ny8weDkw
IFtzdW5ycGNdDQo+ICBbPGZmZmZmZmZmODExN2Y4MzM+XSB2ZnNfd3JpdGUrMHhiMy8weDE4MA0K
PiAgWzxmZmZmZmZmZjgxMTdmYjVhPl0gc3lzX3dyaXRlKzB4NGEvMHg5MA0KPiAgWzxmZmZmZmZm
ZjgxNjAwMzI5Pl0gc3lzdGVtX2NhbGxfZmFzdHBhdGgrMHgxNi8weDFiDQo+IENvZGU6ICBCYWQg
UklQIHZhbHVlLg0KPiBSSVAgIFs8ICAgICAgICAgIChudWxsKT5dICAgICAgICAgICAobnVsbCkN
Cj4gIFJTUCA8ZmZmZjg4MDFhMzY0NWQ0MD4NCj4gQ1IyOiAwMDAwMDAwMDAwMDAwMDAwDQo+IA0K
PiBTaWduZWQtb2ZmLWJ5OiBEYXZpZCBIb3dlbGxzIDxkaG93ZWxsc0ByZWRoYXQuY29tPg0KPiBS
ZXZpZXdlZC1ieTogU3RldmUgRGlja3NvbiA8c3RldmVkQHJlZGhhdC5jb20+DQo+IC0tLQ0KDQpU
aGlzIHBhdGNoIHNob3VsZCBhbHJlYWR5IGJlIHByZXNlbnQgaW4gbmZzLWZvci1uZXh0IGFzIG9m
IHllc3RlcmRheS4NCihJJ3ZlIGJlZW4gb24gYSAyIHdlZWsgdmFjYXRpb24pLg0KDQotLSANClRy
b25kIE15a2xlYnVzdA0KTGludXggTkZTIGNsaWVudCBtYWludGFpbmVyDQoNCk5ldEFwcA0KVHJv
bmQuTXlrbGVidXN0QG5ldGFwcC5jb20NCnd3dy5uZXRhcHAuY29tDQoNCg==
next prev parent reply other threads:[~2012-07-31 14:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-31 14:42 [PATCH 1/2] NFS: Fix a number of bugs in the idmapper David Howells
2012-07-31 14:42 ` [PATCH 2/2] NFS: Combine the idmapper key types David Howells
2012-07-31 14:54 ` Myklebust, Trond
2012-07-31 15:05 ` David Howells
2012-07-31 14:48 ` Myklebust, Trond [this message]
2012-07-31 14:56 ` [PATCH 1/2] NFS: Fix a number of bugs in the idmapper David Howells
2012-07-31 15:07 ` Myklebust, Trond
2012-07-31 16:37 ` J. Bruce Fields
2012-07-31 17:11 ` J. Bruce Fields
2012-08-01 12:39 ` Steve Dickson
-- strict thread matches above, loose matches on Subject: below --
2012-07-25 15:53 David Howells
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=1343746117.5528.3.camel@lade.trondhjem.org \
--to=trond.myklebust@netapp.com \
--cc=Bryan.Schumaker@netapp.com \
--cc=dhowells@redhat.com \
--cc=jlayton@redhat.com \
--cc=linux-nfs@vger.kernel.org \
--cc=sct@redhat.com \
--cc=steved@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).