[lkml]   [2002]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [ltt-dev] Re: [PATCH] LTT for 2.5.38 1/9: Core infrastructure
    Yes this is simple code - similar to the model we use in K42.  Still,
    couple of things about the below.

    1) the !event_wanted can be done outside the function, in a macro so that
    the only cost if tracing is disabled is a hot cache hit on a mask (not
    function call) - that helps with your comment:
    > The event_wanted() filter function should be made as fast as possible.

    2) If you use the lockless scheme you do not need to disable interrupts.
    In K42 we manage to do the entire log operation in 21 instructions and
    about as many cycles (couple more for getting time). We do this from user
    space as well, disabling interrupts precludes this model (may of may not be
    a problem). I was really leaning hard away from even the cost of making a
    system call and disabling interrupts. Do people on the kernel dev team
    feel this is an acceptable cost? Is migration prevented when interrupts
    are disabled? This is something for us to consider.

    3) All trace events should not have to have the same number of data words
    logged - though I think that's just a packaging/interface issue the code
    below would just be placed behind macros which correctly package up the
    right number of arguments.

    Ingo Molnar writes:
    > this is that a trace point should do, at most:
    > --------------------->
    > task_t *tracer_task;
    > int curr_idx[NR_CPUS];
    > int curr_pending[NR_CPUS];
    > struct trace_event **trace_ring;
    > void trace(event, data1, data2, data3)
    > {
    > int cpu = smp_processor_id();
    > int idx, pending, *curr = curr_idx + cpu;
    > struct trace_event *t;
    > unsigned long flags;
    > if (!event_wanted(current, event, data1, data2, data3))
    > return;
    > local_irq_save(flags);
    > idx = ++curr_idx[cpu] & (NR_TRACE_ENTRIES - 1);
    > pending = ++curr_pending[cpu];
    > t = trace_ring[cpu] + idx;
    > t->event = event;
    > rdtscll(t->timestamp);
    > t->data1 = data1;
    > t->data2 = data2;
    > t->data3 = data3;
    > if (curr_pending == TRACE_LOW_WATERMARK && tracer_task)
    > wake_up_process(tracer_task);
    > local_irq_restore(flags);
    > }
    > this should cover most of what's needed. The event_wanted() filter
    > function should be made as fast as possible. Note that the irq-disabled
    > section is not strictly needed but nice and also makes it work on the
    > preemptible kernel. (It's not a big issue at all to run these few
    > instructions with irqs disabled.)
    > [there are also other details like putting curr_index and curr_pending
    > into the per-cpu area and similar stuff.]
    > Ingo
    > _______________________________________________
    > ltt-dev mailing list
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:29    [W:0.022 / U:11.220 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site