* [PATCH] rdma.8: Add basic description for users unfamiliar with rdma @ 2020-12-23 4:47 Alan Perry 2020-12-23 8:19 ` Leon Romanovsky 2021-02-02 22:39 ` [PATCH] " Stephen Hemminger 0 siblings, 2 replies; 8+ messages in thread From: Alan Perry @ 2020-12-23 4:47 UTC (permalink / raw) To: leonro, netdev 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. + .SH OPTIONS .TP ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] rdma.8: Add basic description for users unfamiliar with rdma 2020-12-23 4:47 [PATCH] rdma.8: Add basic description for users unfamiliar with rdma Alan Perry @ 2020-12-23 8:19 ` Leon Romanovsky 2020-12-24 6:23 ` Alan Perry 2021-01-21 21:32 ` Alan Perry 2021-02-02 22:39 ` [PATCH] " Stephen Hemminger 1 sibling, 2 replies; 8+ messages in thread From: Leon Romanovsky @ 2020-12-23 8:19 UTC (permalink / raw) To: Alan Perry; +Cc: netdev 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> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] rdma.8: Add basic description for users unfamiliar with rdma 2020-12-23 8:19 ` Leon Romanovsky @ 2020-12-24 6:23 ` Alan Perry 2021-01-21 21:32 ` Alan Perry 1 sibling, 0 replies; 8+ messages in thread From: Alan Perry @ 2020-12-24 6:23 UTC (permalink / raw) To: Leon Romanovsky; +Cc: netdev 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] rdma.8: Add basic description for users unfamiliar with rdma 2020-12-23 8:19 ` Leon Romanovsky 2020-12-24 6:23 ` Alan Perry @ 2021-01-21 21:32 ` Alan Perry 2021-01-24 6:31 ` Leon Romanovsky 1 sibling, 1 reply; 8+ messages in thread From: Alan Perry @ 2021-01-21 21:32 UTC (permalink / raw) To: Leon Romanovsky; +Cc: netdev 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] rdma.8: Add basic description for users unfamiliar with rdma 2021-01-21 21:32 ` Alan Perry @ 2021-01-24 6:31 ` Leon Romanovsky 2021-01-24 18:09 ` [PATCH v2] " Alan Perry 0 siblings, 1 reply; 8+ messages in thread From: Leon Romanovsky @ 2021-01-24 6:31 UTC (permalink / raw) To: Alan Perry; +Cc: netdev 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2] rdma.8: Add basic description for users unfamiliar with rdma 2021-01-24 6:31 ` Leon Romanovsky @ 2021-01-24 18:09 ` Alan Perry 2021-01-24 19:03 ` Leon Romanovsky 0 siblings, 1 reply; 8+ messages in thread From: Alan Perry @ 2021-01-24 18:09 UTC (permalink / raw) To: Leon Romanovsky; +Cc: netdev 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 | 9 ++++++++- 1 file changed, 8 insertion(+), 1 deletion(-) diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 index c9e5d50d..66ef9902 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 "24 Jan 2021" "iproute2" "Linux" .SH NAME rdma \- RDMA tool .SH SYNOPSIS @@ -29,6 +29,13 @@ 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-capable +devices. Remote direct memory access (RDMA) is the ability of accessing +(reading, writing) memory on a remote machine without interrupting the +processing of the CPU(s) on that system. + .SH OPTIONS .TP On 1/23/21 10:31 PM, Leon Romanovsky wrote: > 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 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2] rdma.8: Add basic description for users unfamiliar with rdma 2021-01-24 18:09 ` [PATCH v2] " Alan Perry @ 2021-01-24 19:03 ` Leon Romanovsky 0 siblings, 0 replies; 8+ messages in thread From: Leon Romanovsky @ 2021-01-24 19:03 UTC (permalink / raw) To: Alan Perry; +Cc: netdev On Sun, Jan 24, 2021 at 10:09:55AM -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 | 9 ++++++++- > 1 file changed, 8 insertion(+), 1 deletion(-) > diff --git a/man/man8/rdma.8 b/man/man8/rdma.8 > index c9e5d50d..66ef9902 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 "24 Jan 2021" "iproute2" "Linux" > .SH NAME > rdma \- RDMA tool > .SH SYNOPSIS > @@ -29,6 +29,13 @@ 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-capable > +devices. Remote direct memory access (RDMA) is the ability of accessing > +(reading, writing) memory on a remote machine without interrupting the > +processing of the CPU(s) on that system. > + > .SH OPTIONS > > .TP Please send patch with git send-email and not as reply. Also don't forget to add my Acked-by, use [PATCH iproute2-next] in the title and add Changelog under "---". Thanks > > On 1/23/21 10:31 PM, Leon Romanovsky wrote: > > 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] rdma.8: Add basic description for users unfamiliar with rdma 2020-12-23 4:47 [PATCH] rdma.8: Add basic description for users unfamiliar with rdma Alan Perry 2020-12-23 8:19 ` Leon Romanovsky @ 2021-02-02 22:39 ` Stephen Hemminger 1 sibling, 0 replies; 8+ messages in thread From: Stephen Hemminger @ 2021-02-02 22:39 UTC (permalink / raw) To: Alan Perry; +Cc: leonro, netdev 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. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-02-02 22:40 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-12-23 4:47 [PATCH] rdma.8: Add basic description for users unfamiliar with rdma Alan Perry 2020-12-23 8:19 ` Leon Romanovsky 2020-12-24 6:23 ` Alan Perry 2021-01-21 21:32 ` Alan Perry 2021-01-24 6:31 ` Leon Romanovsky 2021-01-24 18:09 ` [PATCH v2] " Alan Perry 2021-01-24 19:03 ` Leon Romanovsky 2021-02-02 22:39 ` [PATCH] " Stephen Hemminger
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).