public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one
@ 2015-06-22  7:08 Valentin Rothberg
  2015-07-06  7:36 ` Valentin Rothberg
  0 siblings, 1 reply; 4+ messages in thread
From: Valentin Rothberg @ 2015-06-22  7:08 UTC (permalink / raw)
  To: chuck.lever, bfields, Paul Bolle, Andreas Ruprecht,
	hengelein Stefan, trond.myklebust, anna.schumaker, davem,
	linux-kernel

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
Please read the FAQ at  http://www.tux.org/lkml/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one
  2015-06-22  7:08 Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one Valentin Rothberg
@ 2015-07-06  7:36 ` Valentin Rothberg
  2015-07-06 15:30   ` Chuck Lever
  0 siblings, 1 reply; 4+ messages in thread
From: Valentin Rothberg @ 2015-07-06  7:36 UTC (permalink / raw)
  To: chuck.lever, bfields, Paul Bolle, Andreas Ruprecht,
	hengelein Stefan, trond.myklebust, anna.schumaker, davem,
	linux-kernel

On Mon, Jun 22, 2015 at 9:08 AM, Valentin Rothberg
<valentinrothberg@gmail.com> 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.

Kind regards,
 Valentin

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one
  2015-07-06  7:36 ` Valentin Rothberg
@ 2015-07-06 15:30   ` Chuck Lever
  2015-07-08 20:41     ` J. Bruce Fields
  0 siblings, 1 reply; 4+ messages in thread
From: Chuck Lever @ 2015-07-06 15:30 UTC (permalink / raw)
  To: Valentin Rothberg
  Cc: J. Bruce Fields, Paul Bolle, Andreas Ruprecht, hengelein Stefan,
	Trond Myklebust, anna.schumaker, davem, linux-kernel

Hi Valentin-


On Jul 6, 2015, at 3:36 AM, Valentin Rothberg <valentinrothberg@gmail.com> wrote:

> On Mon, Jun 22, 2015 at 9:08 AM, Valentin Rothberg
> <valentinrothberg@gmail.com> 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.

--
Chuck Lever




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one
  2015-07-06 15:30   ` Chuck Lever
@ 2015-07-08 20:41     ` J. Bruce Fields
  0 siblings, 0 replies; 4+ messages in thread
From: J. Bruce Fields @ 2015-07-08 20:41 UTC (permalink / raw)
  To: Chuck Lever
  Cc: Valentin Rothberg, Paul Bolle, Andreas Ruprecht, hengelein Stefan,
	Trond Myklebust, anna.schumaker, davem, linux-kernel

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@gmail.com> wrote:
> 
> > On Mon, Jun 22, 2015 at 9:08 AM, Valentin Rothberg
> > <valentinrothberg@gmail.com> 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.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-07-08 20:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-22  7:08 Documentation: rpcrdma: Merge svcrdma and xprtrdma modules into one Valentin Rothberg
2015-07-06  7:36 ` Valentin Rothberg
2015-07-06 15:30   ` Chuck Lever
2015-07-08 20:41     ` J. Bruce Fields

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox