Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions

From	"Rafael J. Wysocki" <rafael@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions
Date	2016-03-01 22:40 +0100
Message-ID	<r80bp-6s3-41@gated-at.bofh.it> (permalink)
References	<r7PJ0-7P6-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, Mar 1, 2016 at 11:26 AM, Peter Zijlstra <peterz@infradead.org> wrote:
>
> $ make tags
>   GEN     tags
> ctags: Warning: drivers/acpi/processor_idle.c:64: null expansion of name pattern "\1"
> ctags: Warning: drivers/xen/events/events_2l.c:41: null expansion of name pattern "\1"
> ctags: Warning: kernel/locking/lockdep.c:151: null expansion of name pattern "\1"
> ctags: Warning: kernel/rcu/rcutorture.c:133: null expansion of name pattern "\1"
> ctags: Warning: kernel/rcu/rcutorture.c:135: null expansion of name pattern "\1"
> ctags: Warning: kernel/workqueue.c:323: null expansion of name pattern "\1"
> ctags: Warning: net/ipv4/syncookies.c:53: null expansion of name pattern "\1"
> ctags: Warning: net/ipv6/syncookies.c:44: null expansion of name pattern "\1"
> ctags: Warning: net/rds/page.c:45: null expansion of name pattern "\1"
>
> Which are all the result of the DEFINE_PER_CPU pattern:
>
> scripts/tags.sh:200:    '/\<DEFINE_PER_CPU([^,]*, *\([[:alnum:]_]*\)/\1/v/'
> scripts/tags.sh:201:    '/\<DEFINE_PER_CPU_SHARED_ALIGNED([^,]*, *\([[:alnum:]_]*\)/\1/v/'
>
> The below cures them. All except the workqueue one are within reasonable
> distance of the 80 char limit. TJ do you have any preference on how to
> fix the wq one, or shall we just not care its too long?
>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> ---
>  drivers/acpi/processor_idle.c  | 4 ++--
>  drivers/xen/events/events_2l.c | 5 +++--
>  kernel/locking/lockdep.c       | 3 +--
>  kernel/rcu/rcutorture.c        | 6 ++----
>  kernel/workqueue.c             | 3 +--
>  net/ipv4/syncookies.c          | 3 +--
>  net/ipv6/syncookies.c          | 3 +--
>  net/rds/page.c                 | 4 ++--
>  8 files changed, 13 insertions(+), 18 deletions(-)
>
> diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
> index 175c86bee3a9..9ca2b2fefd76 100644
> --- a/drivers/acpi/processor_idle.c
> +++ b/drivers/acpi/processor_idle.c
> @@ -61,8 +61,8 @@ module_param(latency_factor, uint, 0644);
>
>  static DEFINE_PER_CPU(struct cpuidle_device *, acpi_cpuidle_device);
>
> -static DEFINE_PER_CPU(struct acpi_processor_cx * [CPUIDLE_STATE_MAX],
> -                                                               acpi_cstate);
> +static
> +DEFINE_PER_CPU(struct acpi_processor_cx * [CPUIDLE_STATE_MAX], acpi_cstate);
>
>  static int disabled_by_idle_boot_param(void)
>  {

For the above:

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions Peter Zijlstra <peterz@infradead.org> - 2016-03-01 11:30 +0100
  Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions David Miller <davem@davemloft.net> - 2016-03-01 17:40 +0100
  Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions Tejun Heo <tj@kernel.org> - 2016-03-01 17:40 +0100
    Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions Peter Zijlstra <peterz@infradead.org> - 2016-03-01 19:20 +0100
      Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions Tejun Heo <tj@kernel.org> - 2016-03-01 20:00 +0100
  Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-03-01 20:30 +0100
    Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions Peter Zijlstra <peterz@infradead.org> - 2016-03-01 22:40 +0100
      Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2016-03-01 23:10 +0100
  Re: [RFC][PATCH] tags: Fix DEFINE_PER_CPU expansions "Rafael J. Wysocki" <rafael@kernel.org> - 2016-03-01 22:40 +0100

csiph-web