lkml.org 
[lkml]   [2010]   [Jun]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [rfc] Describe events in a structured way via sysfs

    * Lin Ming <ming.m.lin@intel.com> wrote:

    > > Also, we can (optionally) consider 'generic', subsystem level events to
    > > also show up under:
    > >
    > > /sys/bus/pci/drivers/i915/events/
    > >
    > > This would give a model to non-device-specific events to be listed one
    > > level higher in the sysfs hierarchy.
    > >
    > > This too would be done in the driver, not by generic code. It's generally
    > > the driver which knows how the events should be categorized.
    >
    > This is a bit difficult. I'd like not to touch TRACE_EVENT(). [...]

    We can certainly start with the simpler variant - it's also the more common
    case.

    > [...] How does the driver know if an event is 'generic' if TRACE_EVENT is
    > not touched?

    Well, it's per driver code which creates the 'events' directory anyway, so
    that code decides where to link things. It can link it to the per driver kobj
    - or to the per subsys kobj.

    > > I'd imagine something similar for wireless drivers as well - most
    > > currently defined events would show up on a per device basis there.
    > >
    > > Can you see practical problems with this scheme?
    >
    > Not now. I may find some problems when write more detail code.

    Ok. Feel free to post RFC patches (even if they are not fully complete yet),
    so that we can see how things are progressing.

    I suspect the best approach would be to try to figure out the right sysfs
    placement for one or two existing driver tracepoints, so that we can see it
    all in practice. (Obviously any changes to drivers will have to go via the
    relevant driver maintainer tree(s).)

    Thanks,

    Ingo


    \
     
     \ /
      Last update: 2010-06-29 12:29    [W:0.022 / U:0.660 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site