[lkml]   [2010]   [Dec]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH/RFC 1/2] jump label: make enable/disable o(1)
    * Peter Zijlstra ( wrote:
    > On Thu, 2010-12-16 at 15:36 -0500, Mathieu Desnoyers wrote:
    > > Tracepoints keep their own reference counts for enable/disable, so a
    > > simple "enable/disable" is fine as far as tracepoints are concerned. Why
    > > does perf need that refcounting done by the static jumps ?
    > Because the refcount is all we have... Why not replace that tracepoint
    > refcount with the jumplabel thing?

    The reason why tracepoints need to keep their own refcount is because
    they support dynamically loadable modules, and hence the refcount must
    be kept outside of the modules, in a table internal to tracepoints,
    so we can attach a probe to a yet unloaded module. Therefore, relying on
    this lower level jump label to keep the refcount is not appropriate for
    tracepoints, because the refcount only exists when the module is live.

    I know that your point of view is "let users of modules suffer", but
    this represents a very large portion of Linux users I am not willing to
    let suffer knowingly.



    Mathieu Desnoyers
    Operating System Efficiency R&D Consultant
    EfficiOS Inc.

     \ /
      Last update: 2010-12-16 22:43    [from the cache]
    ©2003-2014 Jasper Spaans. hosted at Digital Ocean