Return-Path: linux-nfs-owner@vger.kernel.org Received: from cmexedge2.ext.emulex.com ([138.239.224.100]:47435 "EHLO CMEXEDGE2.ext.emulex.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755191AbaEPHIR (ORCPT ); Fri, 16 May 2014 03:08:17 -0400 From: Devesh Sharma To: Chuck Lever , "linux-nfs@vger.kernel.org" , "linux-rdma@vger.kernel.org" CC: "Anna.Schumaker@netapp.com" Subject: RE: [PATCH V3 01/17] xprtrdma: mind the device's max fast register page list depth Date: Fri, 16 May 2014 07:08:15 +0000 Message-ID: References: <20140430191433.5663.16217.stgit@manet.1015granger.net> <20140430192936.5663.66537.stgit@manet.1015granger.net> In-Reply-To: <20140430192936.5663.66537.stgit@manet.1015granger.net> Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-nfs-owner@vger.kernel.org List-ID: Q2h1Y2sNCg0KVGhpcyBwYXRjaCBpcyBjYXVzaW5nIGEgQ1BVIHNvZnQtbG9ja3VwIGlmIHVuZGVy bHlpbmcgdmVuZG9yIHJlcG9ydHMgZGV2YXR0ci5tYXhfZmFzdF9yZWdfcGFncl9saXN0X2xlbiA9 IDAgYW5kIGlhLT5yaV9tZW1yZWdfc3RyYXRlZ3kgPSBGUk1SIChEZWZhdWx0IG9wdGlvbikuDQpJ IHRoaW5rIHRoZXJlIGlzIG5lZWQgdG8gcmVmZXIgdG8gZGV2aWNlIGNhcGFiaWxpdHkgZmxhZ3Mu IElmIHN0cmF0ZWd5ID0gRlJNUiBpcyBmb3JjZWQgYW5kIGRldmF0dHIubWF4X2Zhc3RfcmVnX3Bh Z3JfbGlzdF9sZW49MCB0aGVuIGZsYXNoIGFuIGVycm9yIGFuZCBmYWlsIFJQQyB3aXRoIC1FSU8u DQoNClNlZSBpbmxpbmU6DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTog bGludXgtcmRtYS1vd25lckB2Z2VyLmtlcm5lbC5vcmcgW21haWx0bzpsaW51eC1yZG1hLQ0KPiBv d25lckB2Z2VyLmtlcm5lbC5vcmddIE9uIEJlaGFsZiBPZiBDaHVjayBMZXZlcg0KPiBTZW50OiBU aHVyc2RheSwgTWF5IDAxLCAyMDE0IDE6MDAgQU0NCj4gVG86IGxpbnV4LW5mc0B2Z2VyLmtlcm5l bC5vcmc7IGxpbnV4LXJkbWFAdmdlci5rZXJuZWwub3JnDQo+IENjOiBBbm5hLlNjaHVtYWtlckBu ZXRhcHAuY29tDQo+IFN1YmplY3Q6IFtQQVRDSCBWMyAwMS8xN10geHBydHJkbWE6IG1pbmQgdGhl IGRldmljZSdzIG1heCBmYXN0IHJlZ2lzdGVyDQo+IHBhZ2UgbGlzdCBkZXB0aA0KPiANCj4gRnJv bTogU3RldmUgV2lzZSA8c3dpc2VAb3BlbmdyaWRjb21wdXRpbmcuY29tPg0KPiANCj4gU29tZSBy ZG1hIGRldmljZXMgZG9uJ3Qgc3VwcG9ydCBhIGZhc3QgcmVnaXN0ZXIgcGFnZSBsaXN0IGRlcHRo IG9mIGF0IGxlYXN0DQo+IFJQQ1JETUFfTUFYX0RBVEFfU0VHUy4gIFNvIHhwcnRyZG1hIG5lZWRz IHRvIGNodW5rIGl0cyBmYXN0IHJlZ2lzdGVyDQo+IHJlZ2lvbnMgYWNjb3JkaW5nIHRvIHRoZSBt aW5pbXVtIG9mIHRoZSBkZXZpY2UgbWF4IHN1cHBvcnRlZCBkZXB0aCBvcg0KPiBSUENSRE1BX01B WF9EQVRBX1NFR1MuDQo+IA0KPiBTaWduZWQtb2ZmLWJ5OiBTdGV2ZSBXaXNlIDxzd2lzZUBvcGVu Z3JpZGNvbXB1dGluZy5jb20+DQo+IFJldmlld2VkLWJ5OiBDaHVjayBMZXZlciA8Y2h1Y2subGV2 ZXJAb3JhY2xlLmNvbT4NCj4gLS0tDQo+IA0KPiAgbmV0L3N1bnJwYy94cHJ0cmRtYS9ycGNfcmRt YS5jICB8ICAgIDQgLS0tDQo+ICBuZXQvc3VucnBjL3hwcnRyZG1hL3ZlcmJzLmMgICAgIHwgICA0 NyArKysrKysrKysrKysrKysrKysrKysrKysrKysrKy0NCj4gLS0tLS0tLS0tDQo+ICBuZXQvc3Vu cnBjL3hwcnRyZG1hL3hwcnRfcmRtYS5oIHwgICAgMSArDQo+ICAzIGZpbGVzIGNoYW5nZWQsIDM2 IGluc2VydGlvbnMoKyksIDE2IGRlbGV0aW9ucygtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL25ldC9z dW5ycGMveHBydHJkbWEvcnBjX3JkbWEuYw0KPiBiL25ldC9zdW5ycGMveHBydHJkbWEvcnBjX3Jk bWEuYyBpbmRleCA5NmVhZDUyLi40MDBhYTFiIDEwMDY0NA0KPiAtLS0gYS9uZXQvc3VucnBjL3hw cnRyZG1hL3JwY19yZG1hLmMNCj4gKysrIGIvbmV0L3N1bnJwYy94cHJ0cmRtYS9ycGNfcmRtYS5j DQo+IEBAIC0yNDgsMTAgKzI0OCw2IEBAIHJwY3JkbWFfY3JlYXRlX2NodW5rcyhzdHJ1Y3QgcnBj X3Jxc3QgKnJxc3QsIHN0cnVjdA0KPiB4ZHJfYnVmICp0YXJnZXQsDQo+ICAJLyogc3VjY2Vzcy4g YWxsIGZhaWx1cmVzIHJldHVybiBhYm92ZSAqLw0KPiAgCXJlcS0+cmxfbmNodW5rcyA9IG5jaHVu a3M7DQo+IA0KPiAtCUJVR19PTihuY2h1bmtzID09IDApOw0KPiAtCUJVR19PTigocl94cHJ0LT5y eF9pYS5yaV9tZW1yZWdfc3RyYXRlZ3kgPT0gUlBDUkRNQV9GUk1SKQ0KPiAtCSAgICAgICAmJiAo bmNodW5rcyA+IDMpKTsNCj4gLQ0KPiAgCS8qDQo+ICAJICogZmluaXNoIG9mZiBoZWFkZXIuIElm IHdyaXRlLCBtYXJzaGFsIGRpc2NyaW0gYW5kIG5jaHVua3MuDQo+ICAJICovDQo+IGRpZmYgLS1n aXQgYS9uZXQvc3VucnBjL3hwcnRyZG1hL3ZlcmJzLmMgYi9uZXQvc3VucnBjL3hwcnRyZG1hL3Zl cmJzLmMNCj4gaW5kZXggOTM3MjY1Ni4uNTVmYjA5YSAxMDA2NDQNCj4gLS0tIGEvbmV0L3N1bnJw Yy94cHJ0cmRtYS92ZXJicy5jDQo+ICsrKyBiL25ldC9zdW5ycGMveHBydHJkbWEvdmVyYnMuYw0K PiBAQCAtNTM5LDYgKzUzOSwxMSBAQCBycGNyZG1hX2lhX29wZW4oc3RydWN0IHJwY3JkbWFfeHBy dCAqeHBydCwgc3RydWN0DQo+IHNvY2thZGRyICphZGRyLCBpbnQgbWVtcmVnKQ0KPiAgCQkJCV9f ZnVuY19fKTsNCj4gIAkJCW1lbXJlZyA9IFJQQ1JETUFfUkVHSVNURVI7DQo+ICAjZW5kaWYNCj4g KwkJfSBlbHNlIHsNCj4gKwkJCS8qIE1pbmQgdGhlIGlhIGxpbWl0IG9uIEZSTVIgcGFnZSBsaXN0 IGRlcHRoICovDQo+ICsJCQlpYS0+cmlfbWF4X2ZybXJfZGVwdGggPSBtaW5fdCh1bnNpZ25lZCBp bnQsDQo+ICsJCQkJUlBDUkRNQV9NQVhfREFUQV9TRUdTLA0KPiArCQkJCWRldmF0dHIubWF4X2Zh c3RfcmVnX3BhZ2VfbGlzdF9sZW4pOw0KPiAgCQl9DQo+ICAJCWJyZWFrOw0KPiAgCX0NCj4gQEAg LTY1OSwyNCArNjY0LDQyIEBAIHJwY3JkbWFfZXBfY3JlYXRlKHN0cnVjdCBycGNyZG1hX2VwICpl cCwgc3RydWN0DQo+IHJwY3JkbWFfaWEgKmlhLA0KPiAgCWVwLT5yZXBfYXR0ci5zcnEgPSBOVUxM Ow0KPiAgCWVwLT5yZXBfYXR0ci5jYXAubWF4X3NlbmRfd3IgPSBjZGF0YS0+bWF4X3JlcXVlc3Rz Ow0KPiAgCXN3aXRjaCAoaWEtPnJpX21lbXJlZ19zdHJhdGVneSkgew0KPiAtCWNhc2UgUlBDUkRN QV9GUk1SOg0KPiArCWNhc2UgUlBDUkRNQV9GUk1SOiB7DQo+ICsJCWludCBkZXB0aCA9IDc7DQo+ ICsNCj4gIAkJLyogQWRkIHJvb20gZm9yIGZybXIgcmVnaXN0ZXIgYW5kIGludmFsaWRhdGUgV1Jz Lg0KPiAgCQkgKiAxLiBGUk1SIHJlZyBXUiBmb3IgaGVhZA0KPiAgCQkgKiAyLiBGUk1SIGludmFs aWRhdGUgV1IgZm9yIGhlYWQNCj4gLQkJICogMy4gRlJNUiByZWcgV1IgZm9yIHBhZ2VsaXN0DQo+ IC0JCSAqIDQuIEZSTVIgaW52YWxpZGF0ZSBXUiBmb3IgcGFnZWxpc3QNCj4gKwkJICogMy4gTiBG Uk1SIHJlZyBXUnMgZm9yIHBhZ2VsaXN0DQo+ICsJCSAqIDQuIE4gRlJNUiBpbnZhbGlkYXRlIFdS cyBmb3IgcGFnZWxpc3QNCj4gIAkJICogNS4gRlJNUiByZWcgV1IgZm9yIHRhaWwNCj4gIAkJICog Ni4gRlJNUiBpbnZhbGlkYXRlIFdSIGZvciB0YWlsDQo+ICAJCSAqIDcuIFRoZSBSRE1BX1NFTkQg V1INCj4gIAkJICovDQo+IC0JCWVwLT5yZXBfYXR0ci5jYXAubWF4X3NlbmRfd3IgKj0gNzsNCj4g Kw0KPiArCQkvKiBDYWxjdWxhdGUgTiBpZiB0aGUgZGV2aWNlIG1heCBGUk1SIGRlcHRoIGlzIHNt YWxsZXIgdGhhbg0KPiArCQkgKiBSUENSRE1BX01BWF9EQVRBX1NFR1MuDQo+ICsJCSAqLw0KPiAr CQlpZiAoaWEtPnJpX21heF9mcm1yX2RlcHRoIDwgUlBDUkRNQV9NQVhfREFUQV9TRUdTKSB7DQo+ ICsJCQlpbnQgZGVsdGEgPSBSUENSRE1BX01BWF9EQVRBX1NFR1MgLQ0KPiArCQkJCSAgICBpYS0+ cmlfbWF4X2ZybXJfZGVwdGg7DQo+ICsNCj4gKwkJCWRvIHsNCj4gKwkJCQlkZXB0aCArPSAyOyAv KiBGUk1SIHJlZyArIGludmFsaWRhdGUgKi8NCj4gKwkJCQlkZWx0YSAtPSBpYS0+cmlfbWF4X2Zy bXJfZGVwdGg7DQoNCklmIGlhLT5yaV9tYXhfZnJtcl9kZXB0aCBpcyA9IDAuIFRoaXMgbG9vcCBi ZWNvbWVzIGluZmluaXRlIGxvb3AuDQoNCj4gKwkJCX0gd2hpbGUgKGRlbHRhID4gMCk7DQo+ICsN Cj4gKwkJfQ0KPiArCQllcC0+cmVwX2F0dHIuY2FwLm1heF9zZW5kX3dyICo9IGRlcHRoOw0KPiAg CQlpZiAoZXAtPnJlcF9hdHRyLmNhcC5tYXhfc2VuZF93ciA+IGRldmF0dHIubWF4X3FwX3dyKSB7 DQo+IC0JCQljZGF0YS0+bWF4X3JlcXVlc3RzID0gZGV2YXR0ci5tYXhfcXBfd3IgLyA3Ow0KPiAr CQkJY2RhdGEtPm1heF9yZXF1ZXN0cyA9IGRldmF0dHIubWF4X3FwX3dyIC8gZGVwdGg7DQo+ICAJ CQlpZiAoIWNkYXRhLT5tYXhfcmVxdWVzdHMpDQo+ICAJCQkJcmV0dXJuIC1FSU5WQUw7DQo+IC0J CQllcC0+cmVwX2F0dHIuY2FwLm1heF9zZW5kX3dyID0gY2RhdGEtDQo+ID5tYXhfcmVxdWVzdHMg KiA3Ow0KPiArCQkJZXAtPnJlcF9hdHRyLmNhcC5tYXhfc2VuZF93ciA9IGNkYXRhLQ0KPiA+bWF4 X3JlcXVlc3RzICoNCj4gKwkJCQkJCSAgICAgICBkZXB0aDsNCj4gIAkJfQ0KPiAgCQlicmVhazsN Cj4gKwl9DQo+ICAJY2FzZSBSUENSRE1BX01FTVdJTkRPV1NfQVNZTkM6DQo+ICAJY2FzZSBSUENS RE1BX01FTVdJTkRPV1M6DQo+ICAJCS8qIEFkZCByb29tIGZvciBtd19iaW5kcyt1bmJpbmRzIC0g b3ZlcmtpbGwhICovIEBAIC0NCj4gMTA0MywxNiArMTA2NiwxNiBAQCBycGNyZG1hX2J1ZmZlcl9j cmVhdGUoc3RydWN0IHJwY3JkbWFfYnVmZmVyICpidWYsDQo+IHN0cnVjdCBycGNyZG1hX2VwICpl cCwNCj4gIAljYXNlIFJQQ1JETUFfRlJNUjoNCj4gIAkJZm9yIChpID0gYnVmLT5yYl9tYXhfcmVx dWVzdHMgKiBSUENSRE1BX01BWF9TRUdTOyBpOyBpLS0NCj4gKSB7DQo+ICAJCQlyLT5yLmZybXIu ZnJfbXIgPSBpYl9hbGxvY19mYXN0X3JlZ19tcihpYS0+cmlfcGQsDQo+IC0NCj4gUlBDUkRNQV9N QVhfU0VHUyk7DQo+ICsJCQkJCQlpYS0+cmlfbWF4X2ZybXJfZGVwdGgpOw0KPiAgCQkJaWYgKElT X0VSUihyLT5yLmZybXIuZnJfbXIpKSB7DQo+ICAJCQkJcmMgPSBQVFJfRVJSKHItPnIuZnJtci5m cl9tcik7DQo+ICAJCQkJZHByaW50aygiUlBDOiAgICAgICAlczogaWJfYWxsb2NfZmFzdF9yZWdf bXIiDQo+ICAJCQkJCSIgZmFpbGVkICVpXG4iLCBfX2Z1bmNfXywgcmMpOw0KPiAgCQkJCWdvdG8g b3V0Ow0KPiAgCQkJfQ0KPiAtCQkJci0+ci5mcm1yLmZyX3BnbCA9DQo+IC0JCQkJaWJfYWxsb2Nf ZmFzdF9yZWdfcGFnZV9saXN0KGlhLT5yaV9pZC0NCj4gPmRldmljZSwNCj4gLQ0KPiBSUENSRE1B X01BWF9TRUdTKTsNCj4gKwkJCXItPnIuZnJtci5mcl9wZ2wgPSBpYl9hbGxvY19mYXN0X3JlZ19w YWdlX2xpc3QoDQo+ICsJCQkJCQlpYS0+cmlfaWQtPmRldmljZSwNCj4gKwkJCQkJCWlhLT5yaV9t YXhfZnJtcl9kZXB0aCk7DQo+ICAJCQlpZiAoSVNfRVJSKHItPnIuZnJtci5mcl9wZ2wpKSB7DQo+ ICAJCQkJcmMgPSBQVFJfRVJSKHItPnIuZnJtci5mcl9wZ2wpOw0KPiAgCQkJCWRwcmludGsoIlJQ QzogICAgICAgJXM6ICINCj4gQEAgLTE0OTgsOCArMTUyMSw4IEBAIHJwY3JkbWFfcmVnaXN0ZXJf ZnJtcl9leHRlcm5hbChzdHJ1Y3QNCj4gcnBjcmRtYV9tcl9zZWcgKnNlZywNCj4gIAlzZWcxLT5t cl9vZmZzZXQgLT0gcGFnZW9mZjsJLyogc3RhcnQgb2YgcGFnZSAqLw0KPiAgCXNlZzEtPm1yX2xl biArPSBwYWdlb2ZmOw0KPiAgCWxlbiA9IC1wYWdlb2ZmOw0KPiAtCWlmICgqbnNlZ3MgPiBSUENS RE1BX01BWF9EQVRBX1NFR1MpDQo+IC0JCSpuc2VncyA9IFJQQ1JETUFfTUFYX0RBVEFfU0VHUzsN Cj4gKwlpZiAoKm5zZWdzID4gaWEtPnJpX21heF9mcm1yX2RlcHRoKQ0KPiArCQkqbnNlZ3MgPSBp YS0+cmlfbWF4X2ZybXJfZGVwdGg7DQo+ICAJZm9yIChwYWdlX25vID0gaSA9IDA7IGkgPCAqbnNl Z3M7KSB7DQo+ICAJCXJwY3JkbWFfbWFwX29uZShpYSwgc2VnLCB3cml0aW5nKTsNCj4gIAkJcGEg PSBzZWctPm1yX2RtYTsNCj4gZGlmZiAtLWdpdCBhL25ldC9zdW5ycGMveHBydHJkbWEveHBydF9y ZG1hLmgNCj4gYi9uZXQvc3VucnBjL3hwcnRyZG1hL3hwcnRfcmRtYS5oIGluZGV4IGNjMTQ0NWQu Ljk4MzQwYTMgMTAwNjQ0DQo+IC0tLSBhL25ldC9zdW5ycGMveHBydHJkbWEveHBydF9yZG1hLmgN Cj4gKysrIGIvbmV0L3N1bnJwYy94cHJ0cmRtYS94cHJ0X3JkbWEuaA0KPiBAQCAtNjYsNiArNjYs NyBAQCBzdHJ1Y3QgcnBjcmRtYV9pYSB7DQo+ICAJc3RydWN0IGNvbXBsZXRpb24JcmlfZG9uZTsN Cj4gIAlpbnQJCQlyaV9hc3luY19yYzsNCj4gIAllbnVtIHJwY3JkbWFfbWVtcmVnCXJpX21lbXJl Z19zdHJhdGVneTsNCj4gKwl1bnNpZ25lZCBpbnQJCXJpX21heF9mcm1yX2RlcHRoOw0KPiAgfTsN Cj4gDQo+ICAvKg0KPiANCj4gLS0NCj4gVG8gdW5zdWJzY3JpYmUgZnJvbSB0aGlzIGxpc3Q6IHNl bmQgdGhlIGxpbmUgInVuc3Vic2NyaWJlIGxpbnV4LXJkbWEiIGluIHRoZQ0KPiBib2R5IG9mIGEg bWVzc2FnZSB0byBtYWpvcmRvbW9Admdlci5rZXJuZWwub3JnIE1vcmUgbWFqb3Jkb21vIGluZm8g YXQNCj4gaHR0cDovL3ZnZXIua2VybmVsLm9yZy9tYWpvcmRvbW8taW5mby5odG1sDQo=