linux-next: build failure after merge of the rcu tree

Subsystems: networking [general], shared memory communications (smc) sockets, the rest

8 messages, 3 authors, 2017-03-08 · open the first message on its own page

linux-next: build failure after merge of the rcu tree

From: Stephen Rothwell <hidden>
Date: 2017-01-19 03:58:48

Hi Paul,

After merging the rcu tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

net/smc/af_smc.c:102:16: error: 'SLAB_DESTROY_BY_RCU' undeclared here (not in a function)
  .slab_flags = SLAB_DESTROY_BY_RCU,
                ^

Caused by commit

  c7a545924ca1 ("mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU")

interacting with commit

  ac7138746e14 ("smc: establish new socket family")

from the net-next tree.

I have applied the following merge fix patch (someone will need to
remember to mention this to Linus):

From: Stephen Rothwell <redacted>
Date: Thu, 19 Jan 2017 14:29:12 +1100
Subject: [PATCH] smc: merge fix for "mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU"

Signed-off-by: Stephen Rothwell <redacted>
---
 net/smc/af_smc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/smc/af_smc.c b/net/smc/af_smc.c
index 4875e65f0c4a..a48260f9ebb7 100644
--- a/net/smc/af_smc.c
+++ b/net/smc/af_smc.c
@@ -99,7 +99,7 @@ struct proto smc_proto = {
 	.unhash		= smc_unhash_sk,
 	.obj_size	= sizeof(struct smc_sock),
 	.h.smc_hash	= &smc_v4_hashinfo,
-	.slab_flags	= SLAB_DESTROY_BY_RCU,
+	.slab_flags	= SLAB_TYPESAFE_BY_RCU,
 };
 EXPORT_SYMBOL_GPL(smc_proto);
 
-- 
2.10.2

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the rcu tree

From: Paul McKenney <hidden>
Date: 2017-01-19 22:17:36

On Wed, Jan 18, 2017 at 7:34 PM, Stephen Rothwell [off-list ref] wrote:
Hi Paul,

After merging the rcu tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

net/smc/af_smc.c:102:16: error: 'SLAB_DESTROY_BY_RCU' undeclared here (not in a function)
  .slab_flags = SLAB_DESTROY_BY_RCU,
                ^

Caused by commit

  c7a545924ca1 ("mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU")

interacting with commit

  ac7138746e14 ("smc: establish new socket family")

from the net-next tree.

I have applied the following merge fix patch (someone will need to
remember to mention this to Linus):
Thank you, Stephen!  I expect that there might be a bit more
bikeshedding on the name, but here is hoping...  :-/

                                                      Thanx, Paul
quoted hunk
From: Stephen Rothwell <redacted>
Date: Thu, 19 Jan 2017 14:29:12 +1100
Subject: [PATCH] smc: merge fix for "mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU"

Signed-off-by: Stephen Rothwell <redacted>
---
 net/smc/af_smc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/smc/af_smc.c b/net/smc/af_smc.c
index 4875e65f0c4a..a48260f9ebb7 100644
--- a/net/smc/af_smc.c
+++ b/net/smc/af_smc.c
@@ -99,7 +99,7 @@ struct proto smc_proto = {
        .unhash         = smc_unhash_sk,
        .obj_size       = sizeof(struct smc_sock),
        .h.smc_hash     = &smc_v4_hashinfo,
-       .slab_flags     = SLAB_DESTROY_BY_RCU,
+       .slab_flags     = SLAB_TYPESAFE_BY_RCU,
 };
 EXPORT_SYMBOL_GPL(smc_proto);

--
2.10.2

--
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the rcu tree

From: Stephen Rothwell <hidden>
Date: 2017-02-13 02:21:40

Hi Paul,

On Thu, 19 Jan 2017 13:54:37 -0800 Paul McKenney [off-list ref] wrote:
On Wed, Jan 18, 2017 at 7:34 PM, Stephen Rothwell [off-list ref] wrote:
quoted
Hi Paul,

After merging the rcu tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

net/smc/af_smc.c:102:16: error: 'SLAB_DESTROY_BY_RCU' undeclared here (not in a function)
  .slab_flags = SLAB_DESTROY_BY_RCU,
                ^

Caused by commit

  c7a545924ca1 ("mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU")

interacting with commit

  ac7138746e14 ("smc: establish new socket family")

from the net-next tree.

I have applied the following merge fix patch (someone will need to
remember to mention this to Linus):  
Thank you, Stephen!  I expect that there might be a bit more
bikeshedding on the name, but here is hoping...  :-/
The need for this merge fix patch has gone away today.  Is that a
permanent situation, or will it come back?
-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the rcu tree

From: Paul E. McKenney <hidden>
Date: 2017-02-13 04:37:54

On Mon, Feb 13, 2017 at 01:21:33PM +1100, Stephen Rothwell wrote:
Hi Paul,

On Thu, 19 Jan 2017 13:54:37 -0800 Paul McKenney [off-list ref] wrote:
quoted
On Wed, Jan 18, 2017 at 7:34 PM, Stephen Rothwell [off-list ref] wrote:
quoted
Hi Paul,

After merging the rcu tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

net/smc/af_smc.c:102:16: error: 'SLAB_DESTROY_BY_RCU' undeclared here (not in a function)
  .slab_flags = SLAB_DESTROY_BY_RCU,
                ^

Caused by commit

  c7a545924ca1 ("mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU")

interacting with commit

  ac7138746e14 ("smc: establish new socket family")

from the net-next tree.

I have applied the following merge fix patch (someone will need to
remember to mention this to Linus):  
Thank you, Stephen!  I expect that there might be a bit more
bikeshedding on the name, but here is hoping...  :-/
The need for this merge fix patch has gone away today.  Is that a
permanent situation, or will it come back?
I chickened out on that commit for this merge window, so it will come
back at -rc1.  But I will cover that when I rebase to -rc1.

							Thanx, Paul

Re: linux-next: build failure after merge of the rcu tree

From: Stephen Rothwell <hidden>
Date: 2017-02-13 06:43:29

Hi Paul,

On Sun, 12 Feb 2017 20:37:48 -0800 "Paul E. McKenney" [off-list ref] wrote:
I chickened out on that commit for this merge window, so it will come
back at -rc1.  But I will cover that when I rebase to -rc1.
OK, thanks.

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the rcu tree

From: Stephen Rothwell <hidden>
Date: 2017-03-08 01:34:53

Hi Paul,

On Mon, 13 Feb 2017 17:43:24 +1100 Stephen Rothwell [off-list ref] wrote:
On Sun, 12 Feb 2017 20:37:48 -0800 "Paul E. McKenney" [off-list ref] wrote:
quoted
I chickened out on that commit for this merge window, so it will come
back at -rc1.  But I will cover that when I rebase to -rc1.  
OK, thanks.
[PATCH] smc: merge fix for "mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU"
is needed again ... maybe time to rebase?

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build failure after merge of the rcu tree

From: Paul E. McKenney <hidden>
Date: 2017-03-08 05:23:31

On Wed, Mar 08, 2017 at 12:16:05PM +1100, Stephen Rothwell wrote:
Hi Paul,

On Mon, 13 Feb 2017 17:43:24 +1100 Stephen Rothwell [off-list ref] wrote:
quoted
On Sun, 12 Feb 2017 20:37:48 -0800 "Paul E. McKenney" [off-list ref] wrote:
quoted
I chickened out on that commit for this merge window, so it will come
back at -rc1.  But I will cover that when I rebase to -rc1.  
OK, thanks.
[PATCH] smc: merge fix for "mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU"
is needed again ... maybe time to rebase?
On my list for tomorrow morning.  ;-)

							Thanx, Paul

Re: linux-next: build failure after merge of the rcu tree

From: Paul E. McKenney <hidden>
Date: 2017-03-08 18:06:14

On Wed, Mar 08, 2017 at 12:16:05PM +1100, Stephen Rothwell wrote:
Hi Paul,

On Mon, 13 Feb 2017 17:43:24 +1100 Stephen Rothwell [off-list ref] wrote:
quoted
On Sun, 12 Feb 2017 20:37:48 -0800 "Paul E. McKenney" [off-list ref] wrote:
quoted
I chickened out on that commit for this merge window, so it will come
back at -rc1.  But I will cover that when I rebase to -rc1.  
OK, thanks.
[PATCH] smc: merge fix for "mm: Rename SLAB_DESTROY_BY_RCU to SLAB_TYPESAFE_BY_RCU"
is needed again ... maybe time to rebase?
And I did rebase and push.  Passes light build testing, will be testing
more vigorously later today.

							Thanx, Paul
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help