lkml.org 
[lkml]   [2021]   [Dec]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v4 2/2] smp: Fix kernel-doc related mistakes
From


On 12/16/21 06:40, Zhen Lei wrote:
> 1. Change "function-name:" to "function-name -".
> 2. The kernel-doc comments should start with "/**".
> 3. Add descriptions for the missing parameters.
> 4. Fix a mismatched function name in the comment.
>
> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>

Acked-by: Randy Dunlap <rdunlap@infradead.org>

thanks.


> ---
> kernel/smp.c | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/kernel/smp.c b/kernel/smp.c
> index 1e5221fe200296a..d00cc8e0e5d8b16 100644
> --- a/kernel/smp.c
> +++ b/kernel/smp.c
> @@ -698,8 +698,9 @@ void flush_smp_call_function_from_idle(void)
> local_irq_restore(flags);
> }
>
> -/*
> +/**
> * smp_call_function_single - Run a function on a specific CPU
> + * @cpu: The CPU to run on.
> * @func: The function to run. This must be fast and non-blocking.
> * @info: An arbitrary pointer to pass to the function.
> * @wait: If true, wait until function has completed on other CPUs.
> @@ -809,7 +810,7 @@ int smp_call_function_single_async(int cpu, struct __call_single_data *csd)
> }
> EXPORT_SYMBOL_GPL(smp_call_function_single_async);
>
> -/*
> +/**
> * smp_call_function_any - Run a function on any of the given cpus
> * @mask: The mask of cpus it can run on.
> * @func: The function to run. This must be fast and non-blocking.
> @@ -1005,7 +1006,7 @@ void smp_call_function_many(const struct cpumask *mask,
> EXPORT_SYMBOL(smp_call_function_many);
>
> /**
> - * smp_call_function(): Run a function on all other CPUs.
> + * smp_call_function() - Run a function on all other CPUs.
> * @func: The function to run. This must be fast and non-blocking.
> * @info: An arbitrary pointer to pass to the function.
> * @wait: If true, wait (atomically) until function has completed
> @@ -1111,8 +1112,8 @@ void __init smp_init(void)
> smp_cpus_done(setup_max_cpus);
> }
>
> -/*
> - * on_each_cpu_cond(): Call a function on each processor for which
> +/**
> + * on_each_cpu_cond_mask() - Call a function on each processor for which
> * the supplied function cond_func returns true, optionally waiting
> * for all the required CPUs to finish. This may include the local
> * processor.
> @@ -1126,6 +1127,7 @@ void __init smp_init(void)
> * @info: An arbitrary pointer to pass to both functions.
> * @wait: If true, wait (atomically) until function has
> * completed on other CPUs.
> + * @mask: The set of cpus to run on (only runs on online subset).
> *
> * Preemption is disabled to protect against CPUs going offline but not online.
> * CPUs going online during the call will not be seen or sent an IPI.

--
~Randy

\
 
 \ /
  Last update: 2021-12-18 06:53    [W:0.134 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site