On Mon, Feb 25, 2008 at 12:20:13PM -0500, James Lentini wrote:
>
>
> On Mon, 25 Feb 2008, Roland Dreier wrote:
>
> > > + - Install an OpenFabrics-compatible RDMA card
> > > +
> > > + Any device that is compatible with the OpenFabrics.org software stack is
> > > + acceptable.
> >
> > This seems a little odd in the context of instructions on how to use
> > the code in the kernel. Maybe something like:
> >
> > - Install an RDMA device
> >
> > Any device supported by the drivers in drivers/infiniband/hw is
> > acceptable.
> >
> > - R.
>
> I agree. Your wording is clearer.
>
> Bruce,
>
> Here's an update incorporating Roland's suggestion.
>
> Signed-off-by: James Lentini <[email protected]>
>
> --- /dev/null 2008-01-17 11:34:08.491004516 -0500
> +++ Documentation/nfs-rdma.txt 2008-02-25 12:18:17.937932000 -0500
That actually puts the file in the top-level directory. I moved it to
./Documentation/filesystems/, and stripped trailing whitespace.
Assuming that's OK, it's queued up for 2.6.26. Thanks!
--b.
On Mon, 25 Feb 2008, J. Bruce Fields wrote:
> On Mon, Feb 25, 2008 at 12:20:13PM -0500, James Lentini wrote:
> >
> >
> > On Mon, 25 Feb 2008, Roland Dreier wrote:
> >
> > > > + - Install an OpenFabrics-compatible RDMA card
> > > > +
> > > > + Any device that is compatible with the OpenFabrics.org software stack is
> > > > + acceptable.
> > >
> > > This seems a little odd in the context of instructions on how to use
> > > the code in the kernel. Maybe something like:
> > >
> > > - Install an RDMA device
> > >
> > > Any device supported by the drivers in drivers/infiniband/hw is
> > > acceptable.
> > >
> > > - R.
> >
> > I agree. Your wording is clearer.
> >
> > Bruce,
> >
> > Here's an update incorporating Roland's suggestion.
> >
> > Signed-off-by: James Lentini <[email protected]>
> >
> > --- /dev/null 2008-01-17 11:34:08.491004516 -0500
> > +++ Documentation/nfs-rdma.txt 2008-02-25 12:18:17.937932000 -0500
>
> That actually puts the file in the top-level directory. I moved it to
> ./Documentation/filesystems/, and stripped trailing whitespace.
> Assuming that's OK,
Sounds good.
> it's queued up for 2.6.26. Thanks!
Great.
> --b.
>