Re: Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one

From: J. Bruce Fields
Date: Wed Jul 08 2015 - 16:41:37 EST


On Mon, Jul 06, 2015 at 11:30:26AM -0400, Chuck Lever wrote:
> Hi Valentin-
>
>
> On Jul 6, 2015, at 3:36 AM, Valentin Rothberg <valentinrothberg@xxxxxxxxx> wrote:
>
> > On Mon, Jun 22, 2015 at 9:08 AM, Valentin Rothberg
> > <valentinrothberg@xxxxxxxxx> wrote:
> >> Hi Chuck,
> >>
> >> your commit ffe1f0df5862 ("rpcrdma: Merge svcrdma and xprtrdma modules
> >> into one") has shown up in linux next-20150620. The commit merges
> >> SUNRPC_XPRT_RDMA_CLIENT and SUNRPC_XPRT_RDMA_SERVER into
> >> SUNRPC_XPRT_RDMA, but misses to update references in Documentation,
> >> see:
> >>
> >> Documentation/filesystems/nfs/nfs-rdma.txt:142:
> >> SUNRPC_XPRT_RDMA_CLIENT and SUNRPC_XPRT_RDMA_SERVER config options
> >> that both
> >> Documentation/filesystems/nfs/nfs-rdma.txt:242:
> >> (CONFIG_SUNRPC_XPRT_RDMA_SERVER=m in kernel config), load the RDMA
> >> Documentation/filesystems/nfs/nfs-rdma.txt:263:
> >> (CONFIG_SUNRPC_XPRT_RDMA_CLIENT=m in kernel config), load the RDMA
> >> client
> >>
> >> I guess that commit 731d5cca8272 ("Documentation: NFS/RDMA: Document
> >> separate Kconfig symbols") needs to be reverted as well.
> >>
> >> I detected this issue with scripts/checkkconfigsymbols.py.
> >>
> >> Kind regards,
> >> Valentin
> >
> > Hi,
> >
> > this is just a kind reminder that the issue above is now present in v4.2-rc1.
>
> I somehow missed your previous note. You can start by reverting
> commit 731d5cca8272.

OK, queueing up that revert for 4.3.

--b.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/