All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Alexei Starovoitov <ast@kernel.org>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [ast-bpf:trylock 11/11] mm/slub.c:4115:16: error: too few arguments to function 'get_partial'
Date: Sat, 5 Apr 2025 16:36:51 +0800	[thread overview]
Message-ID: <202504051839.ExpJDPAS-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/ast/bpf.git trylock
head:   a8b51ec33155e2a633ff561d09e3ac44faaa3117
commit: a8b51ec33155e2a633ff561d09e3ac44faaa3117 [11/11] kmalloc
config: arc-randconfig-002-20250405 (https://download.01.org/0day-ci/archive/20250405/202504051839.ExpJDPAS-lkp@intel.com/config)
compiler: arc-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250405/202504051839.ExpJDPAS-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202504051839.ExpJDPAS-lkp@intel.com/

All errors (new ones prefixed by >>):

   mm/slub.c: In function '__slab_alloc_node':
>> mm/slub.c:4115:16: error: too few arguments to function 'get_partial'
    4115 |         slab = get_partial(s, node, &pc);
         |                ^~~~~~~~~~~
   mm/slub.c:3000:21: note: declared here
    3000 | static struct slab *get_partial(struct kmem_cache *s, int node,
         |                     ^~~~~~~~~~~


vim +/get_partial +4115 mm/slub.c

f7c80fad6c2b64 Christoph Lameter   2024-10-01  4074  
1f04b07d976da0 Thomas Gleixner     2022-08-25  4075  	if (!USE_LOCKLESS_FAST_PATH() ||
bb192ed9aa7191 Vlastimil Babka     2021-11-03  4076  	    unlikely(!object || !slab || !node_match(slab, node))) {
6edf2576a6cc46 Feng Tang           2022-09-13  4077  		object = __slab_alloc(s, gfpflags, node, addr, c, orig_size);
8eae1492675d0f Dave Hansen         2014-06-04  4078  	} else {
0ad9500e16fe24 Eric Dumazet        2011-12-16  4079  		void *next_object = get_freepointer_safe(s, object);
0ad9500e16fe24 Eric Dumazet        2011-12-16  4080  
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4081  		/*
25985edcedea63 Lucas De Marchi     2011-03-30  4082  		 * The cmpxchg will only match if there was no additional
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4083  		 * operation and if we are on the right processor.
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4084  		 *
d0e0ac9772f8ec Chen Gang           2013-07-15  4085  		 * The cmpxchg does the following atomically (without lock
d0e0ac9772f8ec Chen Gang           2013-07-15  4086  		 * semantics!)
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4087  		 * 1. Relocate first pointer to the current per cpu area.
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4088  		 * 2. Verify that tid and freelist have not been changed
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4089  		 * 3. If they were not changed replace tid and freelist
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4090  		 *
d0e0ac9772f8ec Chen Gang           2013-07-15  4091  		 * Since this is without lock semantics the protection is only
d0e0ac9772f8ec Chen Gang           2013-07-15  4092  		 * against code executing on this cpu *not* from access by
d0e0ac9772f8ec Chen Gang           2013-07-15  4093  		 * other cpus.
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4094  		 */
6801be4f2653e5 Peter Zijlstra      2023-05-31  4095  		if (unlikely(!__update_cpu_freelist_fast(s, object, next_object, tid))) {
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4096  			note_cmpxchg_failure("slab_alloc", s, tid);
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4097  			goto redo;
8a5ec0ba42c491 Christoph Lameter   2011-02-25  4098  		}
0ad9500e16fe24 Eric Dumazet        2011-12-16  4099  		prefetch_freepointer(s, next_object);
84e554e6865c4f Christoph Lameter   2009-12-18  4100  		stat(s, ALLOC_FASTPATH);
894b8788d7f265 Christoph Lameter   2007-05-10  4101  	}
0f181f9fbea8bc Alexander Potapenko 2019-10-14  4102  
56d5a2b9ba85a3 Vlastimil Babka     2022-11-21  4103  	return object;
56d5a2b9ba85a3 Vlastimil Babka     2022-11-21  4104  }
0af8489b0216fa Vlastimil Babka     2022-11-15  4105  #else /* CONFIG_SLUB_TINY */
0af8489b0216fa Vlastimil Babka     2022-11-15  4106  static void *__slab_alloc_node(struct kmem_cache *s,
0af8489b0216fa Vlastimil Babka     2022-11-15  4107  		gfp_t gfpflags, int node, unsigned long addr, size_t orig_size)
0af8489b0216fa Vlastimil Babka     2022-11-15  4108  {
0af8489b0216fa Vlastimil Babka     2022-11-15  4109  	struct partial_context pc;
0af8489b0216fa Vlastimil Babka     2022-11-15  4110  	struct slab *slab;
0af8489b0216fa Vlastimil Babka     2022-11-15  4111  	void *object;
0af8489b0216fa Vlastimil Babka     2022-11-15  4112  
0af8489b0216fa Vlastimil Babka     2022-11-15  4113  	pc.flags = gfpflags;
0af8489b0216fa Vlastimil Babka     2022-11-15  4114  	pc.orig_size = orig_size;
43c4c349149c77 Chengming Zhou      2023-11-02 @4115  	slab = get_partial(s, node, &pc);
0af8489b0216fa Vlastimil Babka     2022-11-15  4116  
43c4c349149c77 Chengming Zhou      2023-11-02  4117  	if (slab)
43c4c349149c77 Chengming Zhou      2023-11-02  4118  		return pc.object;
0af8489b0216fa Vlastimil Babka     2022-11-15  4119  
0af8489b0216fa Vlastimil Babka     2022-11-15  4120  	slab = new_slab(s, gfpflags, node);
0af8489b0216fa Vlastimil Babka     2022-11-15  4121  	if (unlikely(!slab)) {
0af8489b0216fa Vlastimil Babka     2022-11-15  4122  		slab_out_of_memory(s, gfpflags, node);
0af8489b0216fa Vlastimil Babka     2022-11-15  4123  		return NULL;
0af8489b0216fa Vlastimil Babka     2022-11-15  4124  	}
0af8489b0216fa Vlastimil Babka     2022-11-15  4125  
0af8489b0216fa Vlastimil Babka     2022-11-15  4126  	object = alloc_single_from_new_slab(s, slab, orig_size);
0af8489b0216fa Vlastimil Babka     2022-11-15  4127  
0af8489b0216fa Vlastimil Babka     2022-11-15  4128  	return object;
0af8489b0216fa Vlastimil Babka     2022-11-15  4129  }
0af8489b0216fa Vlastimil Babka     2022-11-15  4130  #endif /* CONFIG_SLUB_TINY */
56d5a2b9ba85a3 Vlastimil Babka     2022-11-21  4131  

:::::: The code at line 4115 was first introduced by commit
:::::: 43c4c349149c77f27c8e5801755a7b8883a70ebe slub: Change get_partial() interfaces to return slab

:::::: TO: Chengming Zhou <zhouchengming@bytedance.com>
:::::: CC: Vlastimil Babka <vbabka@suse.cz>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

                 reply	other threads:[~2025-04-05  8:37 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202504051839.ExpJDPAS-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=ast@kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.