Re: [PATCH v2 1/2] docs-rst: Convert user verbs doc to rst
From: Joel Nider
Date: Tue Jan 15 2019 - 15:41:39 EST
Jonathan Corbet <corbet@xxxxxxx> wrote on 01/15/2019 08:02:18 PM:
> From: Jonathan Corbet <corbet@xxxxxxx>
> To: Jason Gunthorpe <jgg@xxxxxxxx>
> Cc: Joel Nider <joeln@xxxxxxxxxx>, Leon Romanovsky <leon@xxxxxxxxxx>,
Doug
> Ledford <dledford@xxxxxxxxxx>, Mike Rapoport <rppt@xxxxxxxxxxxxx>,
linux-
> doc@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
> Date: 01/15/2019 08:14 PM
> Subject: Re: [PATCH v2 1/2] docs-rst: Convert user verbs doc to rst
>
> On Tue, 15 Jan 2019 08:14:02 -0700
> Jason Gunthorpe <jgg@xxxxxxxx> wrote:
>
> > > Move user_verbs from infiniband to userspace while changing the
> > > format. Replace the existing Documentation/infiniband/user_verbs.txt
> > > with Documentation/userspace-api/user_verbs.rst. No substantial
changes
> > > to the content - just some minor reformatting to have the rendering
> > > come out nicely.
> > >
> > > Since this documents a userspace API, its home should be with the
> > > other userspace API docs.
> > >
> > > This is in preparation for updating the content in a subsequent
> > > patch.
> > >
> > > Signed-off-by: Joel Nider <joeln@xxxxxxxxxx>
> > > ---
> > > Documentation/infiniband/user_verbs.txt | 69
-----------------------------
> > > Documentation/userspace-api/index.rst | 1 +
> > > Documentation/userspace-api/user_verbs.rst | 70
++++++++++++++++++++++++++++++
> > > 3 files changed, 71 insertions(+), 69 deletions(-)
> > > delete mode 100644 Documentation/infiniband/user_verbs.txt
> > > create mode 100644 Documentation/userspace-api/user_verbs.rst
> >
> > Need to update MAINTAINERS for the new file.. Maybe call this
> > rdma_user_verbs.rst as we could also have rdma_umad, rdma_cm, etc
>
> Both of those suggestions make sense. Joel, can you do a quick respin
> along those lines? Then I think this one is ready.
Yes, if I understood your other message correctly (I'll wait to see your
response on the other thread), I can make these two changes.
> Thanks,
>
> jon
>