* [PATCH] nfs(5): Document the minorversion= mount option
@ 2012-11-14 22:39 Chuck Lever
2012-11-14 23:01 ` Myklebust, Trond
0 siblings, 1 reply; 8+ messages in thread
From: Chuck Lever @ 2012-11-14 22:39 UTC (permalink / raw)
To: linux-nfs
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
Our pals on the Solaris team recently noticed that the minorversion=
option is not documented in nfs(5). This is a first take, just to
start the conversation. I'm pretty sure I got the kernel version
information wrong, for instance.
Any other comments?
utils/mount/nfs.man | 17 +++++++++++++++++
1 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/utils/mount/nfs.man b/utils/mount/nfs.man
index c15de98..66198cc 100644
--- a/utils/mount/nfs.man
+++ b/utils/mount/nfs.man
@@ -777,6 +777,23 @@ so if this mount option is not specified, the NFS version 4 client
uses the TCP protocol.
Refer to the TRANSPORT METHODS section for more details.
.TP 1.5i
+.BI minorversion= n
+Specifies the protocol minor version number.
+NFSv4 introduces "minor versioning," where NFS protocol enhancements can
+be introduced without bumping the NFS protocol version number.
+Before kernel 2.6.38, the minor version is always zero, and this
+option is not recognized.
+After this kernel, specifying "minorversion=1" enables a number of
+advanced features, such as NFSv4 sessions.
+.IP
+Recent kernels allow the minor version to be specified using the
+.B vers=
+option.
+For example, specifying
+.B vers=4.1
+is the same as specifying
+.BR vers=4,minorversion=1 .
+.TP 1.5i
.BI port= n
The numeric value of the server's NFS service port.
If the server's NFS service is not available on the specified port,
^ permalink raw reply related [flat|nested] 8+ messages in thread
* RE: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-14 22:39 [PATCH] nfs(5): Document the minorversion= mount option Chuck Lever
@ 2012-11-14 23:01 ` Myklebust, Trond
2012-11-14 23:03 ` Chuck Lever
0 siblings, 1 reply; 8+ messages in thread
From: Myklebust, Trond @ 2012-11-14 23:01 UTC (permalink / raw)
To: Chuck Lever, linux-nfs@vger.kernel.org
PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiBGcm9tOiBsaW51eC1uZnMtb3duZXJAdmdl
ci5rZXJuZWwub3JnIFttYWlsdG86bGludXgtbmZzLQ0KPiBvd25lckB2Z2VyLmtlcm5lbC5vcmdd
IE9uIEJlaGFsZiBPZiBDaHVjayBMZXZlcg0KPiBTZW50OiBXZWRuZXNkYXksIE5vdmVtYmVyIDE0
LCAyMDEyIDU6MzkgUE0NCj4gVG86IGxpbnV4LW5mc0B2Z2VyLmtlcm5lbC5vcmcNCj4gU3ViamVj
dDogW1BBVENIXSBuZnMoNSk6IERvY3VtZW50IHRoZSBtaW5vcnZlcnNpb249IG1vdW50IG9wdGlv
bg0KPiANCj4gU2lnbmVkLW9mZi1ieTogQ2h1Y2sgTGV2ZXIgPGNodWNrLmxldmVyQG9yYWNsZS5j
b20+DQo+IC0tLQ0KPiANCj4gT3VyIHBhbHMgb24gdGhlIFNvbGFyaXMgdGVhbSByZWNlbnRseSBu
b3RpY2VkIHRoYXQgdGhlIG1pbm9ydmVyc2lvbj0gb3B0aW9uDQo+IGlzIG5vdCBkb2N1bWVudGVk
IGluIG5mcyg1KS4gIFRoaXMgaXMgYSBmaXJzdCB0YWtlLCBqdXN0IHRvIHN0YXJ0IHRoZSBjb252
ZXJzYXRpb24uDQo+IEknbSBwcmV0dHkgc3VyZSBJIGdvdCB0aGUga2VybmVsIHZlcnNpb24gaW5m
b3JtYXRpb24gd3JvbmcsIGZvciBpbnN0YW5jZS4NCj4gDQo+IEFueSBvdGhlciBjb21tZW50cz8N
Cg0KWW91IG1pZ2h0IHdhbnQgdG8gbm90ZSB0aGF0IGl0IGlzIGRlcHJlY2F0ZWQgaW4gZmF2b3Vy
IG9mIHRoZSAidmVycz00LjEiIG5vdGF0aW9uLCBhbmQgZm9yIHRoYXQgcmVhc29uIHdlIG1pZ2h0
IHdhbnQgdG8gZm9yZ28gZG9jdW1lbnRpbmcgaXQgYWx0b2dldGhlci4NCg0KPiAgdXRpbHMvbW91
bnQvbmZzLm1hbiB8ICAgMTcgKysrKysrKysrKysrKysrKysNCj4gIDEgZmlsZXMgY2hhbmdlZCwg
MTcgaW5zZXJ0aW9ucygrKSwgMCBkZWxldGlvbnMoLSkNCj4gDQo+IGRpZmYgLS1naXQgYS91dGls
cy9tb3VudC9uZnMubWFuIGIvdXRpbHMvbW91bnQvbmZzLm1hbiBpbmRleA0KPiBjMTVkZTk4Li42
NjE5OGNjIDEwMDY0NA0KPiAtLS0gYS91dGlscy9tb3VudC9uZnMubWFuDQo+ICsrKyBiL3V0aWxz
L21vdW50L25mcy5tYW4NCj4gQEAgLTc3Nyw2ICs3NzcsMjMgQEAgc28gaWYgdGhpcyBtb3VudCBv
cHRpb24gaXMgbm90IHNwZWNpZmllZCwgdGhlIE5GUw0KPiB2ZXJzaW9uIDQgY2xpZW50ICB1c2Vz
IHRoZSBUQ1AgcHJvdG9jb2wuDQo+ICBSZWZlciB0byB0aGUgVFJBTlNQT1JUIE1FVEhPRFMgc2Vj
dGlvbiBmb3IgbW9yZSBkZXRhaWxzLg0KPiAgLlRQIDEuNWkNCj4gKy5CSSBtaW5vcnZlcnNpb249
IG4NCj4gK1NwZWNpZmllcyB0aGUgcHJvdG9jb2wgbWlub3IgdmVyc2lvbiBudW1iZXIuDQo+ICtO
RlN2NCBpbnRyb2R1Y2VzICJtaW5vciB2ZXJzaW9uaW5nLCIgd2hlcmUgTkZTIHByb3RvY29sIGVu
aGFuY2VtZW50cw0KPiArY2FuIGJlIGludHJvZHVjZWQgd2l0aG91dCBidW1waW5nIHRoZSBORlMg
cHJvdG9jb2wgdmVyc2lvbiBudW1iZXIuDQo+ICtCZWZvcmUga2VybmVsIDIuNi4zOCwgdGhlIG1p
bm9yIHZlcnNpb24gaXMgYWx3YXlzIHplcm8sIGFuZCB0aGlzIG9wdGlvbg0KPiAraXMgbm90IHJl
Y29nbml6ZWQuDQo+ICtBZnRlciB0aGlzIGtlcm5lbCwgc3BlY2lmeWluZyAibWlub3J2ZXJzaW9u
PTEiIGVuYWJsZXMgYSBudW1iZXIgb2YNCj4gK2FkdmFuY2VkIGZlYXR1cmVzLCBzdWNoIGFzIE5G
U3Y0IHNlc3Npb25zLg0KPiArLklQDQo+ICtSZWNlbnQga2VybmVscyBhbGxvdyB0aGUgbWlub3Ig
dmVyc2lvbiB0byBiZSBzcGVjaWZpZWQgdXNpbmcgdGhlIC5CDQo+ICt2ZXJzPSBvcHRpb24uDQo+
ICtGb3IgZXhhbXBsZSwgc3BlY2lmeWluZw0KPiArLkIgdmVycz00LjENCj4gK2lzIHRoZSBzYW1l
IGFzIHNwZWNpZnlpbmcNCj4gKy5CUiB2ZXJzPTQsbWlub3J2ZXJzaW9uPTEgLg0KPiArLlRQIDEu
NWkNCj4gIC5CSSBwb3J0PSBuDQo+ICBUaGUgbnVtZXJpYyB2YWx1ZSBvZiB0aGUgc2VydmVyJ3Mg
TkZTIHNlcnZpY2UgcG9ydC4NCj4gIElmIHRoZSBzZXJ2ZXIncyBORlMgc2VydmljZSBpcyBub3Qg
YXZhaWxhYmxlIG9uIHRoZSBzcGVjaWZpZWQgcG9ydCwNCj4gDQo+IC0tDQo+IFRvIHVuc3Vic2Ny
aWJlIGZyb20gdGhpcyBsaXN0OiBzZW5kIHRoZSBsaW5lICJ1bnN1YnNjcmliZSBsaW51eC1uZnMi
IGluIHRoZQ0KPiBib2R5IG9mIGEgbWVzc2FnZSB0byBtYWpvcmRvbW9Admdlci5rZXJuZWwub3Jn
IE1vcmUgbWFqb3Jkb21vIGluZm8gYXQNCj4gaHR0cDovL3ZnZXIua2VybmVsLm9yZy9tYWpvcmRv
bW8taW5mby5odG1sDQo=
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-14 23:01 ` Myklebust, Trond
@ 2012-11-14 23:03 ` Chuck Lever
2012-11-14 23:12 ` Myklebust, Trond
0 siblings, 1 reply; 8+ messages in thread
From: Chuck Lever @ 2012-11-14 23:03 UTC (permalink / raw)
To: Myklebust, Trond; +Cc: linux-nfs@vger.kernel.org
On Nov 14, 2012, at 6:01 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>> -----Original Message-----
>> From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-
>> owner@vger.kernel.org] On Behalf Of Chuck Lever
>> Sent: Wednesday, November 14, 2012 5:39 PM
>> To: linux-nfs@vger.kernel.org
>> Subject: [PATCH] nfs(5): Document the minorversion= mount option
>>
>> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
>> ---
>>
>> Our pals on the Solaris team recently noticed that the minorversion= option
>> is not documented in nfs(5). This is a first take, just to start the conversation.
>> I'm pretty sure I got the kernel version information wrong, for instance.
>>
>> Any other comments?
>
> You might want to note that it is deprecated in favour of the "vers=4.1" notation, and for that reason we might want to forgo documenting it altogether.
OK. minorversion= is the only way to get NFSv4.1 on EL6-based distributions, isn't it?
By "deprecated" do you mean that you intend to remove it?
>
>> utils/mount/nfs.man | 17 +++++++++++++++++
>> 1 files changed, 17 insertions(+), 0 deletions(-)
>>
>> diff --git a/utils/mount/nfs.man b/utils/mount/nfs.man index
>> c15de98..66198cc 100644
>> --- a/utils/mount/nfs.man
>> +++ b/utils/mount/nfs.man
>> @@ -777,6 +777,23 @@ so if this mount option is not specified, the NFS
>> version 4 client uses the TCP protocol.
>> Refer to the TRANSPORT METHODS section for more details.
>> .TP 1.5i
>> +.BI minorversion= n
>> +Specifies the protocol minor version number.
>> +NFSv4 introduces "minor versioning," where NFS protocol enhancements
>> +can be introduced without bumping the NFS protocol version number.
>> +Before kernel 2.6.38, the minor version is always zero, and this option
>> +is not recognized.
>> +After this kernel, specifying "minorversion=1" enables a number of
>> +advanced features, such as NFSv4 sessions.
>> +.IP
>> +Recent kernels allow the minor version to be specified using the .B
>> +vers= option.
>> +For example, specifying
>> +.B vers=4.1
>> +is the same as specifying
>> +.BR vers=4,minorversion=1 .
>> +.TP 1.5i
>> .BI port= n
>> The numeric value of the server's NFS service port.
>> If the server's NFS service is not available on the specified port,
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the
>> body of a message to majordomo@vger.kernel.org More majordomo info at
>> http://vger.kernel.org/majordomo-info.html--
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-14 23:03 ` Chuck Lever
@ 2012-11-14 23:12 ` Myklebust, Trond
2012-11-14 23:37 ` Chuck Lever
0 siblings, 1 reply; 8+ messages in thread
From: Myklebust, Trond @ 2012-11-14 23:12 UTC (permalink / raw)
To: Chuck Lever; +Cc: linux-nfs@vger.kernel.org
T24gV2VkLCAyMDEyLTExLTE0IGF0IDE4OjAzIC0wNTAwLCBDaHVjayBMZXZlciB3cm90ZToNCj4g
T24gTm92IDE0LCAyMDEyLCBhdCA2OjAxIFBNLCAiTXlrbGVidXN0LCBUcm9uZCIgPFRyb25kLk15
a2xlYnVzdEBuZXRhcHAuY29tPiB3cm90ZToNCj4gDQo+ID4+IC0tLS0tT3JpZ2luYWwgTWVzc2Fn
ZS0tLS0tDQo+ID4+IEZyb206IGxpbnV4LW5mcy1vd25lckB2Z2VyLmtlcm5lbC5vcmcgW21haWx0
bzpsaW51eC1uZnMtDQo+ID4+IG93bmVyQHZnZXIua2VybmVsLm9yZ10gT24gQmVoYWxmIE9mIENo
dWNrIExldmVyDQo+ID4+IFNlbnQ6IFdlZG5lc2RheSwgTm92ZW1iZXIgMTQsIDIwMTIgNTozOSBQ
TQ0KPiA+PiBUbzogbGludXgtbmZzQHZnZXIua2VybmVsLm9yZw0KPiA+PiBTdWJqZWN0OiBbUEFU
Q0hdIG5mcyg1KTogRG9jdW1lbnQgdGhlIG1pbm9ydmVyc2lvbj0gbW91bnQgb3B0aW9uDQo+ID4+
IA0KPiA+PiBTaWduZWQtb2ZmLWJ5OiBDaHVjayBMZXZlciA8Y2h1Y2subGV2ZXJAb3JhY2xlLmNv
bT4NCj4gPj4gLS0tDQo+ID4+IA0KPiA+PiBPdXIgcGFscyBvbiB0aGUgU29sYXJpcyB0ZWFtIHJl
Y2VudGx5IG5vdGljZWQgdGhhdCB0aGUgbWlub3J2ZXJzaW9uPSBvcHRpb24NCj4gPj4gaXMgbm90
IGRvY3VtZW50ZWQgaW4gbmZzKDUpLiAgVGhpcyBpcyBhIGZpcnN0IHRha2UsIGp1c3QgdG8gc3Rh
cnQgdGhlIGNvbnZlcnNhdGlvbi4NCj4gPj4gSSdtIHByZXR0eSBzdXJlIEkgZ290IHRoZSBrZXJu
ZWwgdmVyc2lvbiBpbmZvcm1hdGlvbiB3cm9uZywgZm9yIGluc3RhbmNlLg0KPiA+PiANCj4gPj4g
QW55IG90aGVyIGNvbW1lbnRzPw0KPiA+IA0KPiA+IFlvdSBtaWdodCB3YW50IHRvIG5vdGUgdGhh
dCBpdCBpcyBkZXByZWNhdGVkIGluIGZhdm91ciBvZiB0aGUgInZlcnM9NC4xIiBub3RhdGlvbiwg
YW5kIGZvciB0aGF0IHJlYXNvbiB3ZSBtaWdodCB3YW50IHRvIGZvcmdvIGRvY3VtZW50aW5nIGl0
IGFsdG9nZXRoZXIuDQo+IA0KPiBPSy4gIG1pbm9ydmVyc2lvbj0gaXMgdGhlIG9ubHkgd2F5IHRv
IGdldCBORlN2NC4xIG9uIEVMNi1iYXNlZCBkaXN0cmlidXRpb25zLCBpc24ndCBpdD8NCg0KWW91
IGRpZG4ndCBzZWVtIHRvIGJlIHdyaXRpbmcgYSBtYW5wYWdlIGZvciB0aGUgUkhFTC02IGRpc3Ry
b3M6ICJCZWZvcmUNCmtlcm5lbCAyLjYuMzgsIHRoZSBtaW5vciB2ZXJzaW9uIGlzIGFsd2F5cyB6
ZXJvLi4uIi4gQXMgZmFyIGFzIEkga25vdywNClJIRUwtNiBpcyBzdGlsbCBiYXNlZCBvbiAyLjYu
MzIuLi4NCg0KPiBCeSAiZGVwcmVjYXRlZCIgZG8geW91IG1lYW4gdGhhdCB5b3UgaW50ZW5kIHRv
IHJlbW92ZSBpdD8NCg0KSSBtZWFuIGZvciBpbnN0YW5jZSB0aGF0ICdjYXQgL3Byb2MvbW91bnRz
JyBhbmQgZnJpZW5kcyB3aWxsIHVzZSB0aGUNCid2ZXJzPTQuMScgbm90YXRpb24uIFRoZXkgd29u
J3QgZXZlciBkaXNwbGF5ICdtaW5vcnZlcnNpb249Jy4NCg0KV2UgbWF5IGVuZCB1cCByZW1vdmlu
ZyBzdXBwb3J0IGZvciBpdCBzb21lIGRheS4NCg0KLS0gDQpUcm9uZCBNeWtsZWJ1c3QNCkxpbnV4
IE5GUyBjbGllbnQgbWFpbnRhaW5lcg0KDQpOZXRBcHANClRyb25kLk15a2xlYnVzdEBuZXRhcHAu
Y29tDQp3d3cubmV0YXBwLmNvbQ0K
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-14 23:12 ` Myklebust, Trond
@ 2012-11-14 23:37 ` Chuck Lever
2012-11-16 13:39 ` Steve Dickson
0 siblings, 1 reply; 8+ messages in thread
From: Chuck Lever @ 2012-11-14 23:37 UTC (permalink / raw)
To: Myklebust, Trond; +Cc: linux-nfs@vger.kernel.org
On Nov 14, 2012, at 6:12 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
> On Wed, 2012-11-14 at 18:03 -0500, Chuck Lever wrote:
>> On Nov 14, 2012, at 6:01 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>>
>>>> -----Original Message-----
>>>> From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-
>>>> owner@vger.kernel.org] On Behalf Of Chuck Lever
>>>> Sent: Wednesday, November 14, 2012 5:39 PM
>>>> To: linux-nfs@vger.kernel.org
>>>> Subject: [PATCH] nfs(5): Document the minorversion= mount option
>>>>
>>>> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
>>>> ---
>>>>
>>>> Our pals on the Solaris team recently noticed that the minorversion= option
>>>> is not documented in nfs(5). This is a first take, just to start the conversation.
>>>> I'm pretty sure I got the kernel version information wrong, for instance.
>>>>
>>>> Any other comments?
>>>
>>> You might want to note that it is deprecated in favour of the "vers=4.1" notation, and for that reason we might want to forgo documenting it altogether.
>>
>> OK. minorversion= is the only way to get NFSv4.1 on EL6-based distributions, isn't it?
>
> You didn't seem to be writing a manpage for the RHEL-6 distros: "Before
> kernel 2.6.38, the minor version is always zero...". As far as I know,
> RHEL-6 is still based on 2.6.32...
...for some very loose definition of "based on". One wonders how to document kernel versions in this instance. One way to fix this is to have RH patch their copy of nfs(5) privately, while upstream nfs-utils can handle this in some other way.
However, search for "Parallel NFS" on this page:
https://access.redhat.com/knowledge/docs/en-US/Red_Hat_Enterprise_Linux/6/html-single/6.2_Release_Notes/index.html
This seems to me to count as public documentation of "minorversion=", and suggests that people are likely already using this formal API. I would hesitate to remove it in this instance.
--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-14 23:37 ` Chuck Lever
@ 2012-11-16 13:39 ` Steve Dickson
2012-11-16 14:05 ` Chuck Lever
0 siblings, 1 reply; 8+ messages in thread
From: Steve Dickson @ 2012-11-16 13:39 UTC (permalink / raw)
To: Chuck Lever; +Cc: Myklebust, Trond, linux-nfs@vger.kernel.org
Hey Chuck,
First of all, thanks for point this out...
On 14/11/12 18:37, Chuck Lever wrote:
>
> On Nov 14, 2012, at 6:12 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>
>> On Wed, 2012-11-14 at 18:03 -0500, Chuck Lever wrote:
>>> On Nov 14, 2012, at 6:01 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>>>
>>>>> -----Original Message-----
>>>>> From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-
>>>>> owner@vger.kernel.org] On Behalf Of Chuck Lever
>>>>> Sent: Wednesday, November 14, 2012 5:39 PM
>>>>> To: linux-nfs@vger.kernel.org
>>>>> Subject: [PATCH] nfs(5): Document the minorversion= mount option
>>>>>
>>>>> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
>>>>> ---
>>>>>
>>>>> Our pals on the Solaris team recently noticed that the minorversion= option
>>>>> is not documented in nfs(5). This is a first take, just to start the conversation.
>>>>> I'm pretty sure I got the kernel version information wrong, for instance.
>>>>>
>>>>> Any other comments?
>>>>
>>>> You might want to note that it is deprecated in favour of the "vers=4.1" notation, and for that reason we might want to forgo documenting it altogether.
>>>
>>> OK. minorversion= is the only way to get NFSv4.1 on EL6-based distributions, isn't it?
>>
>> You didn't seem to be writing a manpage for the RHEL-6 distros: "Before
>> kernel 2.6.38, the minor version is always zero...". As far as I know,
>> RHEL-6 is still based on 2.6.32...
2.6.32 is where we made the branch. So it accurate to say RHEL6 is based
on 2.6.36, but in reality it absolutely not a straight 2.6.32 upstream
kernel due to all the backporting we do.
>
> ...for some very loose definition of "based on". One wonders how to document kernel versions in this instance. One way to fix this is to have RH patch their copy of nfs(5) privately, while upstream nfs-utils can handle this in some other way.
Yeah, We'll have to carry an private patch that updates the man page.
>
> However, search for "Parallel NFS" on this page:
>
> https://access.redhat.com/knowledge/docs/en-US/Red_Hat_Enterprise_Linux/6/html-single/6.2_Release_Notes/index.html
>
> This seems to me to count as public documentation of "minorversion=", and suggests that people are likely already using this formal API. I would hesitate to remove it in this instance.
>
If upstream wants to deprecate minorversion= than so be it... That will have
little effect on a RHEL release since we can't change the API like that...
steved.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-16 13:39 ` Steve Dickson
@ 2012-11-16 14:05 ` Chuck Lever
2012-11-16 14:12 ` Steve Dickson
0 siblings, 1 reply; 8+ messages in thread
From: Chuck Lever @ 2012-11-16 14:05 UTC (permalink / raw)
To: Steve Dickson; +Cc: Myklebust, Trond, linux-nfs@vger.kernel.org
For the record I opened RH bugzilla 877052 yesterday to document this issue in RHEL 6.
Sent from my iPhone
On Nov 16, 2012, at 8:39 AM, Steve Dickson <SteveD@redhat.com> wrote:
> Hey Chuck,
>
> First of all, thanks for point this out...
>
> On 14/11/12 18:37, Chuck Lever wrote:
>>
>> On Nov 14, 2012, at 6:12 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>>
>>> On Wed, 2012-11-14 at 18:03 -0500, Chuck Lever wrote:
>>>> On Nov 14, 2012, at 6:01 PM, "Myklebust, Trond" <Trond.Myklebust@netapp.com> wrote:
>>>>
>>>>>> -----Original Message-----
>>>>>> From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-
>>>>>> owner@vger.kernel.org] On Behalf Of Chuck Lever
>>>>>> Sent: Wednesday, November 14, 2012 5:39 PM
>>>>>> To: linux-nfs@vger.kernel.org
>>>>>> Subject: [PATCH] nfs(5): Document the minorversion= mount option
>>>>>>
>>>>>> Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
>>>>>> ---
>>>>>>
>>>>>> Our pals on the Solaris team recently noticed that the minorversion= option
>>>>>> is not documented in nfs(5). This is a first take, just to start the conversation.
>>>>>> I'm pretty sure I got the kernel version information wrong, for instance.
>>>>>>
>>>>>> Any other comments?
>>>>>
>>>>> You might want to note that it is deprecated in favour of the "vers=4.1" notation, and for that reason we might want to forgo documenting it altogether.
>>>>
>>>> OK. minorversion= is the only way to get NFSv4.1 on EL6-based distributions, isn't it?
>>>
>>> You didn't seem to be writing a manpage for the RHEL-6 distros: "Before
>>> kernel 2.6.38, the minor version is always zero...". As far as I know,
>>> RHEL-6 is still based on 2.6.32...
> 2.6.32 is where we made the branch. So it accurate to say RHEL6 is based
> on 2.6.36, but in reality it absolutely not a straight 2.6.32 upstream
> kernel due to all the backporting we do.
>
>>
>> ...for some very loose definition of "based on". One wonders how to document kernel versions in this instance. One way to fix this is to have RH patch their copy of nfs(5) privately, while upstream nfs-utils can handle this in some other way.
> Yeah, We'll have to carry an private patch that updates the man page.
>
>>
>> However, search for "Parallel NFS" on this page:
>>
>> https://access.redhat.com/knowledge/docs/en-US/Red_Hat_Enterprise_Linux/6/html-single/6.2_Release_Notes/index.html
>>
>> This seems to me to count as public documentation of "minorversion=", and suggests that people are likely already using this formal API. I would hesitate to remove it in this instance.
> If upstream wants to deprecate minorversion= than so be it... That will have
> little effect on a RHEL release since we can't change the API like that...
>
> steved.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] nfs(5): Document the minorversion= mount option
2012-11-16 14:05 ` Chuck Lever
@ 2012-11-16 14:12 ` Steve Dickson
0 siblings, 0 replies; 8+ messages in thread
From: Steve Dickson @ 2012-11-16 14:12 UTC (permalink / raw)
To: Chuck Lever; +Cc: Myklebust, Trond, linux-nfs@vger.kernel.org
On 16/11/12 09:05, Chuck Lever wrote:
> For the record I opened RH bugzilla 877052 yesterday to document this issue in RHEL 6.
Thank you!
steved.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-11-16 14:12 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-14 22:39 [PATCH] nfs(5): Document the minorversion= mount option Chuck Lever
2012-11-14 23:01 ` Myklebust, Trond
2012-11-14 23:03 ` Chuck Lever
2012-11-14 23:12 ` Myklebust, Trond
2012-11-14 23:37 ` Chuck Lever
2012-11-16 13:39 ` Steve Dickson
2012-11-16 14:05 ` Chuck Lever
2012-11-16 14:12 ` Steve Dickson
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).