From mboxrd@z Thu Jan 1 00:00:00 1970 From: Suresh Jayaraman Subject: Re: [PATCH] cifs-utils: mention the required kernel version to make cifs.idmap work Date: Mon, 17 Oct 2011 17:46:40 +0530 Message-ID: <4E9C1CA8.7040105@suse.de> References: <4E9C19A3.8010009@suse.de> <20111017081139.564089db@tlielax.poochiereds.net> Reply-To: sjayaraman-l3A5Bk7waGM@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-cifs , Shirish Pargaonkar To: Jeff Layton Return-path: In-Reply-To: <20111017081139.564089db-9yPaYZwiELC+kQycOl6kW4xkIHaj4LzF@public.gmane.org> Sender: linux-cifs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On 10/17/2011 05:41 PM, Jeff Layton wrote: > On Mon, 17 Oct 2011 17:33:47 +0530 > Suresh Jayaraman wrote: > >> .. properly in the "NOTES" section. >> >> Cc: Shirish Pargaonkar >> Signed-off-by: Suresh Jayaraman >> --- >> cifs.idmap.8.in | 3 +++ >> 1 files changed, 3 insertions(+), 0 deletions(-) >> >> diff --git a/cifs.idmap.8.in b/cifs.idmap.8.in >> index f2fa3b2..7adfdc6 100644 >> --- a/cifs.idmap.8.in >> +++ b/cifs.idmap.8.in >> @@ -76,6 +76,9 @@ create cifs\&.idmap * * @sbindir@/cifs\&.idmap %k >> See >> \fBrequest-key.conf\fR(5) >> for more info on each field\&. >> +.SH "NOTES" >> +.PP >> +For cifs.idmap to work properly you would need a kernel version 3.0 or above. >> .SH "SEE ALSO" >> .PP >> > > This looks reasonable, but I'm always a bit leery of calling out > specific versions like this. Some distros (e.g. Red Hat's and Novell's) > will backport features from later kernels, so saying you need a 3.0 > kernel might be confusing. > > We might want to rephrase this with something like "Support for upcalls > to cifs.idmap was initially introduced in the 3.0 kernel." It's a > little more weaselly but it isn't false if someone is working with a > kernel that has backported this code. > > Sound reasonable? Yes. Also, I'd like to get Shirish Ack (just to be sure) before this one gets merged. I'll repost it. Suresh