From: Jonathan Corbet <corbet@lwn.net> To: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> Cc: mchehab+samsung@kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, skhan@linuxfoundation.org, linux-kernel-mentees@lists.linuxfoundation.org Subject: Re: [PATCH v2 3/8] Documentation: nfs-rdma: convert to ReST Date: Mon, 30 Dec 2019 12:28:41 -0700 [thread overview] Message-ID: <20191230122841.3a10db96@lwn.net> (raw) In-Reply-To: <dd45de2519a3cc1fc07b6b29db77d6be113b0983.1577681164.git.dwlsalmeida@gmail.com> On Mon, 30 Dec 2019 01:55:57 -0300 "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> wrote: > From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> > > Convert nfs-rdma to ReST and move it to admin-guide. Content > remais mostly untouched. > > Signed-off-by: Daniel W. S. Almeida <dwlsalmeida@gmail.com> With this one, my main concern is that this document looks *way* out of date, to the point that I wonder whether it is still useful or not. It would be good to find somebody who knows about this stuff to figure that out. Consider: > +The NFS/RDMA client was first included in Linux 2.6.24. The NFS/RDMA server > +was first included in the following release, Linux 2.6.25. That was a while ago at this point. > +Getting Help > +============ > + > +If you get stuck, you can ask questions on the > +nfs-rdma-devel@lists.sourceforge.net mailing list. What are the chances that this list still works and has relevant people to it? It might be worth sending a copy of this patch there and seeing what results... > +- Install a Linux distribution and tools > + > + The first kernel release to contain both the NFS/RDMA client and server was > + Linux 2.6.25 Therefore, a distribution compatible with this and subsequent > + Linux kernel release should be installed. Hmmm..where might I find such a distribution...? :) > + The procedures described in this document have been tested with > + distributions from Red Hat's Fedora Project (http://fedora.redhat.com/). > + > +- Install nfs-utils-1.1.2 or greater on the client I have nfs-utils 2.4.2 here. So probably nobody needs to do this installation at this point. > + Download the latest package from: http://www.kernel.org/pub/linux/utils/nfs This directory, amusingly, has nothing after 1.0.7, so this advice is actively wrong. I could go on, but I think you get the point. At a bare minimum we should put a big warning at the top saying that this document is obsolete. I should create a standard warning, I guess; for now anything that gets the point across should do. Thanks, jon
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Corbet <corbet@lwn.net> To: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> Cc: mchehab+samsung@kernel.org, linux-kernel-mentees@lists.linuxfoundation.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org Subject: Re: [Linux-kernel-mentees] [PATCH v2 3/8] Documentation: nfs-rdma: convert to ReST Date: Mon, 30 Dec 2019 12:28:41 -0700 [thread overview] Message-ID: <20191230122841.3a10db96@lwn.net> (raw) In-Reply-To: <dd45de2519a3cc1fc07b6b29db77d6be113b0983.1577681164.git.dwlsalmeida@gmail.com> On Mon, 30 Dec 2019 01:55:57 -0300 "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> wrote: > From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com> > > Convert nfs-rdma to ReST and move it to admin-guide. Content > remais mostly untouched. > > Signed-off-by: Daniel W. S. Almeida <dwlsalmeida@gmail.com> With this one, my main concern is that this document looks *way* out of date, to the point that I wonder whether it is still useful or not. It would be good to find somebody who knows about this stuff to figure that out. Consider: > +The NFS/RDMA client was first included in Linux 2.6.24. The NFS/RDMA server > +was first included in the following release, Linux 2.6.25. That was a while ago at this point. > +Getting Help > +============ > + > +If you get stuck, you can ask questions on the > +nfs-rdma-devel@lists.sourceforge.net mailing list. What are the chances that this list still works and has relevant people to it? It might be worth sending a copy of this patch there and seeing what results... > +- Install a Linux distribution and tools > + > + The first kernel release to contain both the NFS/RDMA client and server was > + Linux 2.6.25 Therefore, a distribution compatible with this and subsequent > + Linux kernel release should be installed. Hmmm..where might I find such a distribution...? :) > + The procedures described in this document have been tested with > + distributions from Red Hat's Fedora Project (http://fedora.redhat.com/). > + > +- Install nfs-utils-1.1.2 or greater on the client I have nfs-utils 2.4.2 here. So probably nobody needs to do this installation at this point. > + Download the latest package from: http://www.kernel.org/pub/linux/utils/nfs This directory, amusingly, has nothing after 1.0.7, so this advice is actively wrong. I could go on, but I think you get the point. At a bare minimum we should put a big warning at the top saying that this document is obsolete. I should create a standard warning, I guess; for now anything that gets the point across should do. Thanks, jon _______________________________________________ Linux-kernel-mentees mailing list Linux-kernel-mentees@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
next prev parent reply other threads:[~2019-12-30 19:28 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-12-30 4:55 [PATCH v2 0/8] Documentation: nfs: Convert a few documents to RST and move them to admin-guide Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 4:55 ` [PATCH v2 1/8] Documentation: convert nfs.txt to ReST Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 19:12 ` Jonathan Corbet 2019-12-30 19:12 ` [Linux-kernel-mentees] " Jonathan Corbet 2019-12-30 4:55 ` [PATCH v2 2/8] Documentation: nfsroot.txt: convert " Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 19:18 ` Jonathan Corbet 2019-12-30 19:18 ` [Linux-kernel-mentees] " Jonathan Corbet 2019-12-31 4:08 ` Daniel W. S. Almeida 2019-12-31 4:08 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-31 15:32 ` Jonathan Corbet 2019-12-31 15:32 ` [Linux-kernel-mentees] " Jonathan Corbet 2019-12-31 19:15 ` Daniel W. S. Almeida 2019-12-31 19:15 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 4:55 ` [PATCH v2 3/8] Documentation: nfs-rdma: " Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 19:28 ` Jonathan Corbet [this message] 2019-12-30 19:28 ` Jonathan Corbet 2019-12-30 4:55 ` [PATCH v2 4/8] Documentation: convert nfsd-admin-interfaces " Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 19:30 ` Jonathan Corbet 2019-12-30 19:30 ` [Linux-kernel-mentees] " Jonathan Corbet 2019-12-30 4:55 ` [PATCH v2 5/8] Documentation: nfs: idmapper: convert " Daniel W. S. Almeida 2019-12-30 4:55 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 4:56 ` [PATCH v2 6/8] Documentation: nfs: convert pnfs-block-server " Daniel W. S. Almeida 2019-12-30 4:56 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 4:56 ` [PATCH v2 7/8] Documentation: nfs: pnfs-scsi-server: convert " Daniel W. S. Almeida 2019-12-30 4:56 ` [Linux-kernel-mentees] " Daniel W. S. Almeida 2019-12-30 4:56 ` [PATCH v2 8/8] Documentation: nfs: fault_injection: " Daniel W. S. Almeida 2019-12-30 4:56 ` [Linux-kernel-mentees] " Daniel W. S. Almeida
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=20191230122841.3a10db96@lwn.net \ --to=corbet@lwn.net \ --cc=dwlsalmeida@gmail.com \ --cc=linux-doc@vger.kernel.org \ --cc=linux-kernel-mentees@lists.linuxfoundation.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mchehab+samsung@kernel.org \ --cc=skhan@linuxfoundation.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: linkBe 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.