All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Schwidefsky <schwidefsky@de.ibm.com>
To: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>,
	linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org,
	Hendrik Brueckner <brueckner@linux.vnet.ibm.com>
Subject: Re: [PATCH] s390: delete new instances of __cpuinit usage
Date: Mon, 13 Jan 2014 16:52:05 +0100	[thread overview]
Message-ID: <20140113165205.3d81070b@mschwide> (raw)
In-Reply-To: <1389366050-25210-1-git-send-email-paul.gortmaker@windriver.com>

On Fri, 10 Jan 2014 10:00:50 -0500
Paul Gortmaker <paul.gortmaker@windriver.com> wrote:

> Commit 8c069ff4bd6063a3f15e606c882e03f75c7e7711 ("s390/perf: add
> support for the CPU-Measurement Sampling Facility") added a new
> instance of the __cpuinit macro usage.
> 
> We removed this a couple versions ago; we now want to remove
> the compat no-op stubs.  Introducing new users is not what
> we want to see at this point in time, as it will break once
> the stubs are gone.
> 
> Cc: Hendrik Brueckner <brueckner@linux.vnet.ibm.com>
> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
> ---
> 
> [Patch based on today's linux-next tree, so if s390 next content
>  gets rebased, then the above commit ID will be invalidated; if it
>  _does_ get rebased, then feel free to squash this fix into the orig.]
> 
>  arch/s390/kernel/perf_cpum_sf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/s390/kernel/perf_cpum_sf.c b/arch/s390/kernel/perf_cpum_sf.c
> index 3c3bc8d7b220..ce3a1d9c521b 100644
> --- a/arch/s390/kernel/perf_cpum_sf.c
> +++ b/arch/s390/kernel/perf_cpum_sf.c
> @@ -1512,7 +1512,7 @@ static void cpumf_measurement_alert(struct ext_code ext_code,
>  	}
>  }
> 
> -static int __cpuinit cpumf_pmu_notifier(struct notifier_block *self,
> +static int cpumf_pmu_notifier(struct notifier_block *self,
>  					unsigned long action, void *hcpu)
>  {
>  	unsigned int cpu = (long) hcpu;

Applied to linux-s390 for the next merge window. Thanks.

-- 
blue skies,
   Martin.

"Reality continues to ruin my life." - Calvin.

WARNING: multiple messages have this Message-ID (diff)
From: Martin Schwidefsky <schwidefsky@de.ibm.com>
To: Paul Gortmaker <paul.gortmaker@windriver.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>,
	<linux-s390@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	Hendrik Brueckner <brueckner@linux.vnet.ibm.com>
Subject: Re: [PATCH] s390: delete new instances of __cpuinit usage
Date: Mon, 13 Jan 2014 16:52:05 +0100	[thread overview]
Message-ID: <20140113165205.3d81070b@mschwide> (raw)
In-Reply-To: <1389366050-25210-1-git-send-email-paul.gortmaker@windriver.com>

On Fri, 10 Jan 2014 10:00:50 -0500
Paul Gortmaker <paul.gortmaker@windriver.com> wrote:

> Commit 8c069ff4bd6063a3f15e606c882e03f75c7e7711 ("s390/perf: add
> support for the CPU-Measurement Sampling Facility") added a new
> instance of the __cpuinit macro usage.
> 
> We removed this a couple versions ago; we now want to remove
> the compat no-op stubs.  Introducing new users is not what
> we want to see at this point in time, as it will break once
> the stubs are gone.
> 
> Cc: Hendrik Brueckner <brueckner@linux.vnet.ibm.com>
> Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
> ---
> 
> [Patch based on today's linux-next tree, so if s390 next content
>  gets rebased, then the above commit ID will be invalidated; if it
>  _does_ get rebased, then feel free to squash this fix into the orig.]
> 
>  arch/s390/kernel/perf_cpum_sf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/s390/kernel/perf_cpum_sf.c b/arch/s390/kernel/perf_cpum_sf.c
> index 3c3bc8d7b220..ce3a1d9c521b 100644
> --- a/arch/s390/kernel/perf_cpum_sf.c
> +++ b/arch/s390/kernel/perf_cpum_sf.c
> @@ -1512,7 +1512,7 @@ static void cpumf_measurement_alert(struct ext_code ext_code,
>  	}
>  }
> 
> -static int __cpuinit cpumf_pmu_notifier(struct notifier_block *self,
> +static int cpumf_pmu_notifier(struct notifier_block *self,
>  					unsigned long action, void *hcpu)
>  {
>  	unsigned int cpu = (long) hcpu;

Applied to linux-s390 for the next merge window. Thanks.

-- 
blue skies,
   Martin.

"Reality continues to ruin my life." - Calvin.


  reply	other threads:[~2014-01-13 15:52 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-10 15:00 [PATCH] s390: delete new instances of __cpuinit usage Paul Gortmaker
2014-01-10 15:00 ` Paul Gortmaker
2014-01-13 15:52 ` Martin Schwidefsky [this message]
2014-01-13 15:52   ` Martin Schwidefsky

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=20140113165205.3d81070b@mschwide \
    --to=schwidefsky@de.ibm.com \
    --cc=brueckner@linux.vnet.ibm.com \
    --cc=heiko.carstens@de.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=paul.gortmaker@windriver.com \
    /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.