[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <54F87F34.80408@redhat.com>
Date: Thu, 05 Mar 2015 11:07:16 -0500
From: Rik van Riel <riel@...hat.com>
To: Frederic Weisbecker <fweisbec@...il.com>,
LKML <linux-kernel@...r.kernel.org>
CC: Paolo Bonzini <pbonzini@...hat.com>,
Alexander Graf <agraf@...e.de>,
Luiz Capitulino <lcapitulino@...hat.com>,
Christian Borntraeger <borntraeger@...ibm.com>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Marcelo Tosatti <mtosatti@...hat.com>,
"Paul E . McKenney" <paulmck@...ux.vnet.ibm.com>,
Andy Lutomirski <luto@...capital.net>,
Will deacon <will.deacon@....com>
Subject: Re: [PATCH 2/7] context_tracking: Rename context symbols to prepare
for transition state
On 03/04/2015 09:14 PM, Frederic Weisbecker wrote:
> Current context tracking symbols are designed to express living state.
> As such they are prefixed with "IN_": IN_USER, IN_KERNEL.
>
> Now we are going to use these symbols to also express state transitions
> such as context_tracking_enter(IN_USER) or context_tracking_exit(IN_USER).
> But while the "IN_" prefix works well to express entering a context, it's
> confusing to depict a context exit: context_tracking_exit(IN_USER)
> could mean two things:
> 1) We are exiting the current context to enter user context.
> 2) We are exiting the user context
> We want 2) but the reviewer may be confused and understand 1)
>
> So lets disambiguate these symbols and rename them to CONTEXT_USER and
> CONTEXT_KERNEL.
>
> Cc: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>
> Cc: Andy Lutomirski <luto@...capital.net>
> Cc: Will deacon <will.deacon@....com>
> Cc: Marcelo Tosatti <mtosatti@...hat.com>
> Cc: Christian Borntraeger <borntraeger@...ibm.com>
> Cc: Luiz Capitulino <lcapitulino@...hat.com>
> Cc: Paolo Bonzini <pbonzini@...hat.com>
> Signed-off-by: Frederic Weisbecker <fweisbec@...il.com>
Acked-by: Rik van Riel <riel@...hat.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists