Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Heiko Carstens
On Tue, Apr 05, 2016 at 01:57:42PM +0200, Sebastian Andrzej Siewior wrote: > On 04/05/2016 01:51 PM, rcoch...@linutronix.de wrote: > > On Tue, Apr 05, 2016 at 01:36:38PM +0200, Heiko Carstens wrote: > >> On Tue, Apr 05, 2016 at 01:23:36PM +0200, Heiko Carstens wrote: > >>> Subsequently, in this cas

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Sebastian Andrzej Siewior
On 04/05/2016 01:51 PM, rcoch...@linutronix.de wrote: > On Tue, Apr 05, 2016 at 01:36:38PM +0200, Heiko Carstens wrote: >> On Tue, Apr 05, 2016 at 01:23:36PM +0200, Heiko Carstens wrote: >>> Subsequently, in this case, the setup_pmc_cpu() call will be executed on >>> the wrong cpu. >> >> .. or to i

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Heiko Carstens
On Tue, Apr 05, 2016 at 01:51:29PM +0200, rcoch...@linutronix.de wrote: > On Tue, Apr 05, 2016 at 01:36:38PM +0200, Heiko Carstens wrote: > > On Tue, Apr 05, 2016 at 01:23:36PM +0200, Heiko Carstens wrote: > > > Subsequently, in this case, the setup_pmc_cpu() call will be executed on > > > the wron

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread rcochran
On Tue, Apr 05, 2016 at 01:36:38PM +0200, Heiko Carstens wrote: > On Tue, Apr 05, 2016 at 01:23:36PM +0200, Heiko Carstens wrote: > > Subsequently, in this case, the setup_pmc_cpu() call will be executed on > > the wrong cpu. > > .. or to illustrate this behaviour: the following patch (white space

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Heiko Carstens
On Tue, Apr 05, 2016 at 01:23:36PM +0200, Heiko Carstens wrote: > On Tue, Apr 05, 2016 at 01:13:06PM +0200, Sebastian Andrzej Siewior wrote: > > On 04/05/2016 12:49 PM, Heiko Carstens wrote: > > >> --- a/arch/s390/kernel/perf_cpum_sf.c > > >> +++ b/arch/s390/kernel/perf_cpum_sf.c > > >> @@ -1510,7

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Heiko Carstens
On Tue, Apr 05, 2016 at 01:13:06PM +0200, Sebastian Andrzej Siewior wrote: > On 04/05/2016 12:49 PM, Heiko Carstens wrote: > >> --- a/arch/s390/kernel/perf_cpum_sf.c > >> +++ b/arch/s390/kernel/perf_cpum_sf.c > >> @@ -1510,7 +1510,6 @@ static void cpumf_measurement_alert(stru > >> static int cpumf

Re: [PREEMPT-RT] [PATCH] s390/cpum_sf: Remove superfluous SMP function call

2016-04-05 Thread Sebastian Andrzej Siewior
On 04/05/2016 12:49 PM, Heiko Carstens wrote: >> --- a/arch/s390/kernel/perf_cpum_sf.c >> +++ b/arch/s390/kernel/perf_cpum_sf.c >> @@ -1510,7 +1510,6 @@ static void cpumf_measurement_alert(stru >> static int cpumf_pmu_notifier(struct notifier_block *self, >>unsigned lon