Date: Tue, 4 Aug 2015 10:41:10 -0400
From: Steven Rostedt <rostedt@goodmis.org>
To: Borislav Petkov <bp@alien8.de>
Cc: Peter Zijlstra <peterz@infradead.org>, Vlastimil Babka <vbabka@suse.cz>,
        linux-kernel@vger.kernel.org, mingo@kernel.org, jasonbaron0@gmail.com,
        luto@amacapital.net, tglx@linutronix.de, 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, heiko.carstens@de.ibm.com, davem@davemloft.net
Subject: Re: [PATCH -v2 6/8] jump_label: Add a new static_key interface
Message-ID: <20150804104110.01676ca1@gandalf.local.home>
In-Reply-To: <20150804143344.GA5689@nazgul.tnic>
References: <20150728132313.164884020@infradead.org>
	<55B87E7A.2070509@suse.cz>
	<20150729084906.GH19282@twins.programming.kicks-ass.net>
	<20150803150359.0e76b576@gandalf.local.home>
	<20150803191816.GC25159@twins.programming.kicks-ass.net>
	<20150803152810.5a7bcf06@gandalf.local.home>
	<20150803200002.GE25159@twins.programming.kicks-ass.net>
	<20150803175757.2adf3275@gandalf.local.home>
	<20150804033733.GB31787@nazgul.tnic>
	<20150804080645.0f29617d@gandalf.local.home>
	<20150804143344.GA5689@nazgul.tnic>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1117
Lines: 31

On Tue, 4 Aug 2015 16:33:44 +0200
Borislav Petkov <bp@alien8.de> wrote:

> Hell, we can drop that "key" or "branch" from the whole API for all I
> know. "static_" is enough for me to say what the thing is. But that's
> just me - I like short names - no poems in code and sh*t.
> 
> Thoughts, comments?

I agree to get rid of the "key" and the "branch", I never liked them in
the first place.

I prefer static_set_true/false() but I'm also fine with
static_enable/disable() as long as the initializers are consistent.


> 
> So yeah, I absolutely see the problematic here and also the need for
> more bikeshedding. And this time, that bikeshedding is important.
> 

Right, because the broken part of this nuclear plant, was the bike
shed, and it's color was so ugly that the people in the nuclear plant
kept making mistakes by being distracted by that damn shed!

-- Steve
--
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/