linux-next: manual merge of the slab tree with the rcu tree

Stephen Rothwell posted 1 patch 10 months, 1 week ago
linux-next: manual merge of the slab tree with the rcu tree
Posted by Stephen Rothwell 10 months, 1 week ago
Hi all,

Today's linux-next merge of the slab tree got a conflict in:

  kernel/rcu/tiny.c

between commits:

  84ae91018af5 ("rcutorture: Include grace-period sequence numbers in failure/close-call")
  2db7ab8c1086 ("rcutorture: Expand failure/close-call grace-period output")
  7acc2d90151f ("rcutorture: Make cur_ops->format_gp_seqs take buffer length")

from the rcu tree and commit:

  b14ff274e8aa ("slab, rcu: move TINY_RCU variant of kvfree_rcu() to SLAB")

from the slab tree.

I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging.  You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.

-- 
Cheers,
Stephen Rothwell

diff --cc kernel/rcu/tiny.c
index 8a52aca686a5,7a34a99d4664..000000000000
--- a/kernel/rcu/tiny.c
+++ b/kernel/rcu/tiny.c
@@@ -246,31 -232,6 +232,20 @@@ bool poll_state_synchronize_rcu(unsigne
  }
  EXPORT_SYMBOL_GPL(poll_state_synchronize_rcu);
  
- #ifdef CONFIG_KASAN_GENERIC
- void kvfree_call_rcu(struct rcu_head *head, void *ptr)
- {
- 	if (head)
- 		kasan_record_aux_stack(ptr);
- 
- 	__kvfree_call_rcu(head, ptr);
- }
- EXPORT_SYMBOL_GPL(kvfree_call_rcu);
- #endif
- 
 +#if IS_ENABLED(CONFIG_RCU_TORTURE_TEST)
 +unsigned long long rcutorture_gather_gp_seqs(void)
 +{
 +	return READ_ONCE(rcu_ctrlblk.gp_seq) & 0xffffULL;
 +}
 +EXPORT_SYMBOL_GPL(rcutorture_gather_gp_seqs);
 +
 +void rcutorture_format_gp_seqs(unsigned long long seqs, char *cp, size_t len)
 +{
 +	snprintf(cp, len, "g%04llx", seqs & 0xffffULL);
 +}
 +EXPORT_SYMBOL_GPL(rcutorture_format_gp_seqs);
 +#endif
 +
  void __init rcu_init(void)
  {
  	open_softirq(RCU_SOFTIRQ, rcu_process_callbacks);
Re: linux-next: manual merge of the slab tree with the rcu tree
Posted by Boqun Feng 10 months, 1 week ago
Hi Stephen,

On Wed, Feb 12, 2025 at 03:09:41PM +1100, Stephen Rothwell wrote:
> Hi all,
> 
> Today's linux-next merge of the slab tree got a conflict in:
> 
>   kernel/rcu/tiny.c
> 
> between commits:
> 
>   84ae91018af5 ("rcutorture: Include grace-period sequence numbers in failure/close-call")
>   2db7ab8c1086 ("rcutorture: Expand failure/close-call grace-period output")
>   7acc2d90151f ("rcutorture: Make cur_ops->format_gp_seqs take buffer length")
> 
> from the rcu tree and commit:
> 
>   b14ff274e8aa ("slab, rcu: move TINY_RCU variant of kvfree_rcu() to SLAB")
> 
> from the slab tree.
> 
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your tree
> is submitted for merging.  You may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any particularly
> complex conflicts.
> 

Thanks for reporting this, the below resolution looks good to me, I also
created a branch recording this:

	git://git.kernel.org/pub/scm/linux/kernel/git/rcu/linux.git merge/rcu-vs-slab.2025.02.11a

Regards,
Boqun

> -- 
> Cheers,
> Stephen Rothwell
> 
> diff --cc kernel/rcu/tiny.c
> index 8a52aca686a5,7a34a99d4664..000000000000
> --- a/kernel/rcu/tiny.c
> +++ b/kernel/rcu/tiny.c
> @@@ -246,31 -232,6 +232,20 @@@ bool poll_state_synchronize_rcu(unsigne
>   }
>   EXPORT_SYMBOL_GPL(poll_state_synchronize_rcu);
>   
> - #ifdef CONFIG_KASAN_GENERIC
> - void kvfree_call_rcu(struct rcu_head *head, void *ptr)
> - {
> - 	if (head)
> - 		kasan_record_aux_stack(ptr);
> - 
> - 	__kvfree_call_rcu(head, ptr);
> - }
> - EXPORT_SYMBOL_GPL(kvfree_call_rcu);
> - #endif
> - 
>  +#if IS_ENABLED(CONFIG_RCU_TORTURE_TEST)
>  +unsigned long long rcutorture_gather_gp_seqs(void)
>  +{
>  +	return READ_ONCE(rcu_ctrlblk.gp_seq) & 0xffffULL;
>  +}
>  +EXPORT_SYMBOL_GPL(rcutorture_gather_gp_seqs);
>  +
>  +void rcutorture_format_gp_seqs(unsigned long long seqs, char *cp, size_t len)
>  +{
>  +	snprintf(cp, len, "g%04llx", seqs & 0xffffULL);
>  +}
>  +EXPORT_SYMBOL_GPL(rcutorture_format_gp_seqs);
>  +#endif
>  +
>   void __init rcu_init(void)
>   {
>   	open_softirq(RCU_SOFTIRQ, rcu_process_callbacks);