Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1193794

Re: [RFC][PATCH 5/7] jump_label: Rework update logic

Path csiph.com!aioe.org!eternal-september.org!feeder.eternal-september.org!feeds.phibee-telecom.net!news.panservice.it!diesel.cu.mi.it!bofh.it!news.nic.it!robomod
From Heiko Carstens <heiko.carstens@de.ibm.com>
Newsgroups linux.kernel
Subject Re: [RFC][PATCH 5/7] jump_label: Rework update logic
Date Tue, 28 Jul 2015 08:00:02 +0200
Message-ID <pR65I-d2-7@gated-at.bofh.it> (permalink)
References <pPPzX-4Yj-5@gated-at.bofh.it> <pPPzY-4Yj-25@gated-at.bofh.it> <pQO8O-7zV-7@gated-at.bofh.it> <pQTBw-7n7-19@gated-at.bofh.it>
X-Original-To Peter Zijlstra <peterz@infradead.org>
X-Helo d06dlp01.portsmouth.uk.ibm.com
X-Mailfrom heiko.carstens@de.ibm.com
X-Rcptto linux-kernel@vger.kernel.org
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
User-Agent Mutt/1.5.21 (2010-09-15)
X-Tm-As-Mml disable
X-Content-Scanned Fidelis XPS MAILER
X-Cbid 15072805-0041-0000-0000-0000051172EC
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 21
Organization linux.* mail to news gateway
X-Original-Cc linux-kernel@vger.kernel.org, mingo@kernel.org, jasonbaron0@gmail.com, bp@alien8.de, luto@amacapital.net, tglx@linutronix.de, rostedt@goodmis.org, will.deacon@arm.com, liuj97@gmail.com, rabin@rab.in, ralf@linux-mips.org, ddaney@caviumnetworks.com, benh@kernel.crashing.org, michael@ellerman.id.au, davem@davemloft.net
X-Original-Date Tue, 28 Jul 2015 07:54:34 +0200
X-Original-Message-ID <20150728055434.GA4137@osiris>
X-Original-References <20150724175209.814173117@infradead.org> <20150724175823.013936034@infradead.org> <20150727104714.GT19282@twins.programming.kicks-ass.net> <20150727163005.GP18673@twins.programming.kicks-ass.net>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref aioe.org linux.kernel:1193794

Show key headers only | View raw


On Mon, Jul 27, 2015 at 06:30:05PM +0200, Peter Zijlstra wrote:
> On Mon, Jul 27, 2015 at 12:47:14PM +0200, Peter Zijlstra wrote:
> > @@ -68,13 +63,8 @@ void static_key_slow_inc(struct static_k
> >  		return;
> >  
> >  	jump_label_lock();
> > +	if (atomic_inc_and_test(&key->enabled))
> 
> atomic_inc_return() == 1, works _much_ better.

Confirmed. With this change, the jump_label_init() patch, plus the
changes in s390 inline asm everything works for me.

There are a couple of places in s390 code where the DEFINE_STATIC_KEY_FALSE
and static_branch_likely() combination would make sense.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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


Thread

Re: [RFC][PATCH 5/7] jump_label: Rework update logic Peter Zijlstra <peterz@infradead.org> - 2015-07-27 12:50 +0200
  Re: [RFC][PATCH 5/7] jump_label: Rework update logic Heiko Carstens <heiko.carstens@de.ibm.com> - 2015-07-28 08:00 +0200

csiph-web