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

Groups > linux.kernel > #1726327

Re: [PATCH 2/2] tracing: Add support for critical section events

Show all headers | View raw

On Mon, Sep 4, 2017 at 5:54 PM, Joel Fernandes <joelaf@google.com> wrote:
> On Mon, Sep 4, 2017 at 4:34 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
>> On Mon, 4 Sep 2017 21:44:26 +0200
>> Peter Zijlstra <peterz@infradead.org> wrote:
>>
>>> > I can change the name to something else, but at the moment I can't
>>> > think of anything better. Could you suggest a better name? Also btw,
>>> > 'critical timings' is the terminology used within the irqsoff tracer
>>> > so this is in line with that.
>>>
>>> So 'critical section' is what some mis-guided people call the locked
>>> region of a lock :-) Using it for something else is prone to cause more
>>> confusion...
>>>
>>> I would simply call them what they are: irq_disable,irq_enable
>>> preempt_disable,preempt_enable.
>>
>> Yes please. The "critical section" naming came from the code that was
>> from the latency tracer days of the real time patch (pre-ftrace). The
>> irqsoff tracer has the least modification from the original code, and
>> probably should be rewritten one of these days.
>
> Sounds good to me. For the subsystem, could you guys suggest a name? I
> was thinking "atomic_section"?
>
> Something like:
>
> subsystem: atomic_section
> events:
>   irqsoff_disable
>   irqsoff_enable
>   preemptoff_disable
>   preemptoff_enable
>
> and additionally (to do what my patch does):
>   preemptirqsoff_enable
>   preemptirqsoff_disable

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


Could you let me know if we are in agreement about this naming?

thanks,

-Joel

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

Thread

[PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-03 11:00 +0200
  Re: [PATCH 2/2] tracing: Add support for critical section events Peter Zijlstra <peterz@infradead.org> - 2017-09-04 10:00 +0200
    Re: [PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-04 21:10 +0200
      Re: [PATCH 2/2] tracing: Add support for critical section events Peter Zijlstra <peterz@infradead.org> - 2017-09-04 21:50 +0200
        Re: [PATCH 2/2] tracing: Add support for critical section events Steven Rostedt <rostedt@goodmis.org> - 2017-09-05 01:40 +0200
          Re: [PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-05 03:00 +0200
            Re: [PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-05 05:30 +0200
              Re: [PATCH 2/2] tracing: Add support for critical section events Peter Zijlstra <peterz@infradead.org> - 2017-09-05 09:00 +0200
                Re: [PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-05 18:40 +0200
                Re: [PATCH 2/2] tracing: Add support for critical section events Peter Zijlstra <peterz@infradead.org> - 2017-09-06 10:50 +0200
                Re: [PATCH 2/2] tracing: Add support for critical section events Joel Fernandes <joelaf@google.com> - 2017-09-07 02:30 +0200

csiph-web