All of lore.kernel.org
 help / color / mirror / Atom feed
* [android-common:android15-6.6-2025-10 0/1] kernel/trace/ring_buffer.c:1821: warning: Function parameter or member 'writer' not described in '__ring_buffer_alloc'
@ 2026-07-08 14:15 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2026-07-08 14:15 UTC (permalink / raw)
  To: cros-kernel-buildreports; +Cc: oe-kbuild-all

Hi Vincent,

FYI, the error/warning still remains.

tree:   https://android.googlesource.com/kernel/common android15-6.6-2025-10
head:   a03fb833a7fa2cc2276b3c45eb0672ecb2ca3234
commit: bc115ff00f4cd4d21ddf7e6e200f389d81acc69e [0/1] ANDROID: ring-buffer: Introducing ring-buffer writer
config: i386-buildonly-randconfig-004-20260708 (https://download.01.org/0day-ci/archive/20260708/202607082235.y5EhJi3s-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.4.0-5) 12.4.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260708/202607082235.y5EhJi3s-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/202607082235.y5EhJi3s-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> kernel/trace/ring_buffer.c:1821: warning: Function parameter or member 'writer' not described in '__ring_buffer_alloc'


vim +1821 kernel/trace/ring_buffer.c

7a8e76a3829f10 Steven Rostedt            2008-09-29  1806  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1807  /**
d611851b421731 zhangwei(Jovi             2013-07-15  1808)  * __ring_buffer_alloc - allocate a new ring_buffer
68814b58c52077 Robert Richter            2008-11-24  1809   * @size: the size in bytes per cpu that is needed.
7a8e76a3829f10 Steven Rostedt            2008-09-29  1810   * @flags: attributes to set for the ring buffer.
59e7cffe5cca6f Fabian Frederick          2014-06-05  1811   * @key: ring buffer reader_lock_key.
7a8e76a3829f10 Steven Rostedt            2008-09-29  1812   *
7a8e76a3829f10 Steven Rostedt            2008-09-29  1813   * Currently the only flag that is available is the RB_FL_OVERWRITE
7a8e76a3829f10 Steven Rostedt            2008-09-29  1814   * flag. This flag means that the buffer will overwrite old data
7a8e76a3829f10 Steven Rostedt            2008-09-29  1815   * when the buffer wraps. If this flag is not set, the buffer will
7a8e76a3829f10 Steven Rostedt            2008-09-29  1816   * drop data when the tail hits the head.
7a8e76a3829f10 Steven Rostedt            2008-09-29  1817   */
13292494379f92 Steven Rostedt (VMware    2019-12-13  1818) struct trace_buffer *__ring_buffer_alloc(unsigned long size, unsigned flags,
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1819  					struct lock_class_key *key,
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1820  					struct ring_buffer_writer *writer)
7a8e76a3829f10 Steven Rostedt            2008-09-29 @1821  {
13292494379f92 Steven Rostedt (VMware    2019-12-13  1822) 	struct trace_buffer *buffer;
9b94a8fba501f3 Steven Rostedt (Red Hat   2016-05-12  1823) 	long nr_pages;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1824  	int bsize;
9b94a8fba501f3 Steven Rostedt (Red Hat   2016-05-12  1825) 	int cpu;
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1826  	int ret;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1827  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1828  	/* keep it in its own cache line */
7a8e76a3829f10 Steven Rostedt            2008-09-29  1829  	buffer = kzalloc(ALIGN(sizeof(*buffer), cache_line_size()),
7a8e76a3829f10 Steven Rostedt            2008-09-29  1830  			 GFP_KERNEL);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1831  	if (!buffer)
7a8e76a3829f10 Steven Rostedt            2008-09-29  1832  		return NULL;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1833  
b18cc3de00ec34 Sebastian Andrzej Siewior 2016-12-07  1834  	if (!zalloc_cpumask_var(&buffer->cpumask, GFP_KERNEL))
9e01c1b74c9531 Rusty Russell             2009-01-01  1835  		goto fail_free_buffer;
9e01c1b74c9531 Rusty Russell             2009-01-01  1836  
438ced1720b584 Vaibhav Nagarnaik         2012-02-02  1837  	nr_pages = DIV_ROUND_UP(size, BUF_PAGE_SIZE);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1838  	buffer->flags = flags;
37886f6a9f62d2 Steven Rostedt            2009-03-17  1839  	buffer->clock = trace_clock_local;
1f8a6a10fb9437 Peter Zijlstra            2009-06-08  1840  	buffer->reader_lock_key = key;
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1841  	if (writer) {
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1842  		buffer->writer = writer;
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1843  		atomic_inc(&buffer->record_disabled);
bc115ff00f4cd4 Vincent Donnefort         2023-06-09  1844  	}
7a8e76a3829f10 Steven Rostedt            2008-09-29  1845  
15693458c4bc06 Steven Rostedt (Red Hat   2013-02-28  1846) 	init_irq_work(&buffer->irq_work.work, rb_wake_up_waiters);
f1dc6725882b5c Steven Rostedt (Red Hat   2013-03-04  1847) 	init_waitqueue_head(&buffer->irq_work.waiters);
15693458c4bc06 Steven Rostedt (Red Hat   2013-02-28  1848) 
7a8e76a3829f10 Steven Rostedt            2008-09-29  1849  	/* need at least two pages */
438ced1720b584 Vaibhav Nagarnaik         2012-02-02  1850  	if (nr_pages < 2)
438ced1720b584 Vaibhav Nagarnaik         2012-02-02  1851  		nr_pages = 2;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1852  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1853  	buffer->cpus = nr_cpu_ids;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1854  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1855  	bsize = sizeof(void *) * nr_cpu_ids;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1856  	buffer->buffers = kzalloc(ALIGN(bsize, cache_line_size()),
7a8e76a3829f10 Steven Rostedt            2008-09-29  1857  				  GFP_KERNEL);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1858  	if (!buffer->buffers)
9e01c1b74c9531 Rusty Russell             2009-01-01  1859  		goto fail_free_cpumask;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1860  
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1861  	cpu = raw_smp_processor_id();
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1862  	cpumask_set_cpu(cpu, buffer->cpumask);
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1863  	buffer->buffers[cpu] = rb_allocate_cpu_buffer(buffer, nr_pages, cpu);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1864  	if (!buffer->buffers[cpu])
7a8e76a3829f10 Steven Rostedt            2008-09-29  1865  		goto fail_free_buffers;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1866  
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1867  	ret = cpuhp_state_add_instance(CPUHP_TRACE_RB_PREPARE, &buffer->node);
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1868  	if (ret < 0)
b32614c03413f8 Sebastian Andrzej Siewior 2016-11-27  1869  		goto fail_free_buffers;
554f786e284a6c Steven Rostedt            2009-03-11  1870  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1871  	mutex_init(&buffer->mutex);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1872  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1873  	return buffer;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1874  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1875   fail_free_buffers:
7a8e76a3829f10 Steven Rostedt            2008-09-29  1876  	for_each_buffer_cpu(buffer, cpu) {
7a8e76a3829f10 Steven Rostedt            2008-09-29  1877  		if (buffer->buffers[cpu])
7a8e76a3829f10 Steven Rostedt            2008-09-29  1878  			rb_free_cpu_buffer(buffer->buffers[cpu]);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1879  	}
7a8e76a3829f10 Steven Rostedt            2008-09-29  1880  	kfree(buffer->buffers);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1881  
9e01c1b74c9531 Rusty Russell             2009-01-01  1882   fail_free_cpumask:
9e01c1b74c9531 Rusty Russell             2009-01-01  1883  	free_cpumask_var(buffer->cpumask);
9e01c1b74c9531 Rusty Russell             2009-01-01  1884  
7a8e76a3829f10 Steven Rostedt            2008-09-29  1885   fail_free_buffer:
7a8e76a3829f10 Steven Rostedt            2008-09-29  1886  	kfree(buffer);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1887  	return NULL;
7a8e76a3829f10 Steven Rostedt            2008-09-29  1888  }
1f8a6a10fb9437 Peter Zijlstra            2009-06-08  1889  EXPORT_SYMBOL_GPL(__ring_buffer_alloc);
7a8e76a3829f10 Steven Rostedt            2008-09-29  1890  

:::::: The code at line 1821 was first introduced by commit
:::::: 7a8e76a3829f1067b70f715771ff88baf2fbf3c3 tracing: unified trace buffer

:::::: TO: Steven Rostedt <rostedt@goodmis.org>
:::::: CC: Ingo Molnar <mingo@elte.hu>

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2026-07-08 14:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-08 14:15 [android-common:android15-6.6-2025-10 0/1] kernel/trace/ring_buffer.c:1821: warning: Function parameter or member 'writer' not described in '__ring_buffer_alloc' kernel test robot

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.