diff mbox series

rdma.8: Add basic description for users unfamiliar with rdma

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

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Alan Perry Dec. 23, 2020, 4:47 a.m. UTC
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

Comments

Leon Romanovsky Dec. 23, 2020, 8:19 a.m. UTC | #1
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>
Alan Perry Dec. 24, 2020, 6:23 a.m. UTC | #2
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
Alan Perry Jan. 21, 2021, 9:32 p.m. UTC | #3
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
Leon Romanovsky Jan. 24, 2021, 6:31 a.m. UTC | #4
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
Stephen Hemminger Feb. 2, 2021, 10:39 p.m. UTC | #5
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 mbox series

Patch

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