Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751096AbdIEGwt (ORCPT <rfc822;w@1wt.eu>);
        Tue, 5 Sep 2017 02:52:49 -0400
Received: from merlin.infradead.org ([205.233.59.134]:48926 "EHLO
        merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750729AbdIEGwq (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 5 Sep 2017 02:52:46 -0400
Date: Tue, 5 Sep 2017 08:52:30 +0200
From: Peter Zijlstra <peterz@infradead.org>
To: Joel Fernandes <joelaf@google.com>
Cc: Steven Rostedt <rostedt@goodmis.org>,
        LKML <linux-kernel@vger.kernel.org>, kernel-team@android.com,
        Ingo Molnar <mingo@redhat.com>,
        Byungchul Park <byungchul.park@lge.com>, Tejun Heo <tj@kernel.org>
Subject: Re: [PATCH 2/2] tracing: Add support for critical section events
Message-ID: <20170905065230.lq5fsxgiv6ynehxp@hirez.programming.kicks-ass.net>
References: <20170903085051.6348-1-joelaf@google.com>
 <20170903085051.6348-3-joelaf@google.com>
 <20170904075614.bjkkrgyv2dpz7x5v@hirez.programming.kicks-ass.net>
 <CAJWu+opjZaN4d2-+0jD3QwO+jEXx16WdY0Y_CrP4xihyoemgPQ@mail.gmail.com>
 <20170904194426.GD17526@worktop.programming.kicks-ass.net>
 <20170904193436.4a37fae4@gandalf.local.home>
 <CAJWu+or9O1P=bvGH8i+8v_WEyiFtuQACS4dQWhu19HcdeieETA@mail.gmail.com>
 <CAJWu+opzANvRKkoyVxCtpNgV8HbVARVLQWtfjfdbCV4rjZyW_g@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAJWu+opzANvRKkoyVxCtpNgV8HbVARVLQWtfjfdbCV4rjZyW_g@mail.gmail.com>
User-Agent: NeoMutt/20170609 (1.8.3)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 355
Lines: 17

On Mon, Sep 04, 2017 at 08:26:13PM -0700, Joel Fernandes wrote:

> Apologies, I meant (without the "off"):
> 
> subsystem: atomic_section
> events:
>   irqs_disable
>   irqs_enable
>   preempt_disable
>   preempt_enable
> 
> and additionally (similar to what my patch does):
>   preemptirq_enable
>   preemptirq_disable
> 

What do you need the last for?