Message ID | eb358848-49a8-1a8e-3919-c07b6aa3d21d@snowmoose.com (mailing list archive) |
---|---|
State | New, archived |
Delegated to: | Stephen Hemminger |
Headers | show |
Series | rdma.8: Add basic description for users unfamiliar with rdma | expand |
Context | Check | Description |
---|---|---|
netdev/tree_selection | success | Not a local patch |
On Tue, Dec 22, 2020 at 08:47:51PM -0800, Alan Perry wrote: > Add a description section with basic info about the rdma command for users > unfamiliar with it. > > Signed-off-by: Alan Perry <alanp@snowmoose.com> > --- > man/man8/rdma.8 | 6 +++++- > 1 file changed, 5 insertion(+), 1 deletion(-) > > diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 > index c9e5d50d..d68d0cf6 100644 > --- a/man/man8/rdma.8 > +++ b/man/man8/rdma.8 > @@ -1,4 +1,4 @@ > -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" > +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" > .SH NAME > rdma \- RDMA tool > .SH SYNOPSIS > @@ -29,6 +29,10 @@ rdma \- RDMA tool > \fB\-j\fR[\fIson\fR] } > \fB\-p\fR[\fIretty\fR] } > > +.SH DESCRIPTION > +.B rdma > +is a tool for querying and setting the configuration for RDMA, direct > memory access between the memory of two computers without use of the > operating system on either computer. > + Thanks, it is too close to the Wikipedia description that can be written slightly differently (without "two computers"), what about the following description from Mellanox site? "is a tool for querying and setting the configuration for RDMA-capable devices. Remote direct memory access (RDMA) is the ability of accessing (read, write) memory on a remote machine without interrupting the processing of the CPU(s) on that system." Thanks, Acked-by: Leon Romanovsky <leonro@nvidia.com>
On 12/23/20 12:19 AM, Leon Romanovsky wrote: > On Tue, Dec 22, 2020 at 08:47:51PM -0800, Alan Perry wrote: >> Add a description section with basic info about the rdma command for users >> unfamiliar with it. >> >> Signed-off-by: Alan Perry <alanp@snowmoose.com> >> --- >> man/man8/rdma.8 | 6 +++++- >> 1 file changed, 5 insertion(+), 1 deletion(-) >> >> diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 >> index c9e5d50d..d68d0cf6 100644 >> --- a/man/man8/rdma.8 >> +++ b/man/man8/rdma.8 >> @@ -1,4 +1,4 @@ >> -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" >> +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" >> .SH NAME >> rdma \- RDMA tool >> .SH SYNOPSIS >> @@ -29,6 +29,10 @@ rdma \- RDMA tool >> \fB\-j\fR[\fIson\fR] } >> \fB\-p\fR[\fIretty\fR] } >> >> +.SH DESCRIPTION >> +.B rdma >> +is a tool for querying and setting the configuration for RDMA, direct >> memory access between the memory of two computers without use of the >> operating system on either computer. >> + > > Thanks, it is too close to the Wikipedia description that can be written > slightly differently (without "two computers"), what about the following > description from Mellanox site? > > "is a tool for querying and setting the configuration for RDMA-capable > devices. Remote direct memory access (RDMA) is the ability of accessing > (read, write) memory on a remote machine without interrupting the processing > of the CPU(s) on that system." > > Thanks, > Acked-by: Leon Romanovsky <leonro@nvidia.com> > That change looks good to me. One other change that I wanted to make, but couldn't figure out how to do it without it in a manner wasn't awkward is to make it more clear in the rdma.8 man page that the other rdma man pages describe the rdma "object" variants. That wasn't immediately clear to me the first couple times that I read the rdma.8 page. alan
On 12/23/20 12:19 AM, Leon Romanovsky wrote: > On Tue, Dec 22, 2020 at 08:47:51PM -0800, Alan Perry wrote: >> Add a description section with basic info about the rdma command for users >> unfamiliar with it. >> >> Signed-off-by: Alan Perry <alanp@snowmoose.com> >> --- >> man/man8/rdma.8 | 6 +++++- >> 1 file changed, 5 insertion(+), 1 deletion(-) >> >> diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 >> index c9e5d50d..d68d0cf6 100644 >> --- a/man/man8/rdma.8 >> +++ b/man/man8/rdma.8 >> @@ -1,4 +1,4 @@ >> -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" >> +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" >> .SH NAME >> rdma \- RDMA tool >> .SH SYNOPSIS >> @@ -29,6 +29,10 @@ rdma \- RDMA tool >> \fB\-j\fR[\fIson\fR] } >> \fB\-p\fR[\fIretty\fR] } >> >> +.SH DESCRIPTION >> +.B rdma >> +is a tool for querying and setting the configuration for RDMA, direct >> memory access between the memory of two computers without use of the >> operating system on either computer. >> + > > Thanks, it is too close to the Wikipedia description that can be written > slightly differently (without "two computers"), what about the following > description from Mellanox site? > > "is a tool for querying and setting the configuration for RDMA-capable > devices. Remote direct memory access (RDMA) is the ability of accessing > (read, write) memory on a remote machine without interrupting the processing > of the CPU(s) on that system." > > Thanks, > Acked-by: Leon Romanovsky <leonro@nvidia.com> > I noticed that the rdma man page has not been changed. I am unfamiliar with the process. Should I have submitted an updated patch with the alternate wording after this exchange? alan
On Thu, Jan 21, 2021 at 01:32:42PM -0800, Alan Perry wrote: > > > On 12/23/20 12:19 AM, Leon Romanovsky wrote: > > On Tue, Dec 22, 2020 at 08:47:51PM -0800, Alan Perry wrote: > > > Add a description section with basic info about the rdma command for users > > > unfamiliar with it. > > > > > > Signed-off-by: Alan Perry <alanp@snowmoose.com> > > > --- > > > man/man8/rdma.8 | 6 +++++- > > > 1 file changed, 5 insertion(+), 1 deletion(-) > > > > > > diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 > > > index c9e5d50d..d68d0cf6 100644 > > > --- a/man/man8/rdma.8 > > > +++ b/man/man8/rdma.8 > > > @@ -1,4 +1,4 @@ > > > -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" > > > +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" > > > .SH NAME > > > rdma \- RDMA tool > > > .SH SYNOPSIS > > > @@ -29,6 +29,10 @@ rdma \- RDMA tool > > > \fB\-j\fR[\fIson\fR] } > > > \fB\-p\fR[\fIretty\fR] } > > > > > > +.SH DESCRIPTION > > > +.B rdma > > > +is a tool for querying and setting the configuration for RDMA, direct > > > memory access between the memory of two computers without use of the > > > operating system on either computer. > > > + > > > > Thanks, it is too close to the Wikipedia description that can be written > > slightly differently (without "two computers"), what about the following > > description from Mellanox site? > > > > "is a tool for querying and setting the configuration for RDMA-capable > > devices. Remote direct memory access (RDMA) is the ability of accessing > > (read, write) memory on a remote machine without interrupting the processing > > of the CPU(s) on that system." > > > > Thanks, > > Acked-by: Leon Romanovsky <leonro@nvidia.com> > > > > I noticed that the rdma man page has not been changed. I am unfamiliar with > the process. Should I have submitted an updated patch with the alternate > wording after this exchange? Yes, please. Thanks > > alan
On Tue, 22 Dec 2020 20:47:51 -0800 Alan Perry <alanp@snowmoose.com> wrote: > Add a description section with basic info about the rdma command for > users unfamiliar with it. > > Signed-off-by: Alan Perry <alanp@snowmoose.com> > --- > man/man8/rdma.8 | 6 +++++- > 1 file changed, 5 insertion(+), 1 deletion(-) > > diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 > index c9e5d50d..d68d0cf6 100644 > --- a/man/man8/rdma.8 > +++ b/man/man8/rdma.8 > @@ -1,4 +1,4 @@ > -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" > +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" Please leave the man page date alone, we don't update it anymore > .SH SYNOPSIS > @@ -29,6 +29,10 @@ rdma \- RDMA tool > \fB\-j\fR[\fIson\fR] } > \fB\-p\fR[\fIretty\fR] } > > +.SH DESCRIPTION > +.B rdma > +is a tool for querying and setting the configuration for RDMA, direct > memory access between the memory of two computers without use of the > operating system on either computer. > + > .SH OPTIONS > > .TP This version of the patch does not apply cleanly.
diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 index c9e5d50d..d68d0cf6 100644 --- a/man/man8/rdma.8 +++ b/man/man8/rdma.8 @@ -1,4 +1,4 @@ -.TH RDMA 8 "28 Mar 2017" "iproute2" "Linux" +.TH RDMA 8 "22 Dec 2020" "iproute2" "Linux" .SH NAME rdma \- RDMA tool .SH SYNOPSIS @@ -29,6 +29,10 @@ rdma \- RDMA tool \fB\-j\fR[\fIson\fR] } \fB\-p\fR[\fIretty\fR] } +.SH DESCRIPTION +.B rdma +is a tool for querying and setting the configuration for RDMA, direct memory access between the memory of two computers without use of the operating system on either computer. + .SH OPTIONS
Add a description section with basic info about the rdma command for users unfamiliar with it. Signed-off-by: Alan Perry <alanp@snowmoose.com> --- man/man8/rdma.8 | 6 +++++- 1 file changed, 5 insertion(+), 1 deletion(-) .TP