lkml.org 
[lkml]   [2014]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: perf: Add support for full Intel event lists v6
    On Sun, Jun 15, 2014 at 10:03:09PM +0900, Namhyung Kim wrote:
    > Hi Hagen,
    >
    > 2014-06-14 (토), 22:49 +0200, Hagen Paul Pfeifer:
    > > Probably too late, but IMHO the subcommand "download" is unhappy
    > > named. What is "downloaded"? traces? Python helper libs for
    > > perf-python support, I don't know it. What about "events-download",
    > > "events-database", ...
    >
    > Hmm.. maybe we can add support something like above later - "perf
    > download event" or "perf download blabla"

    Sorry I think that's all overdesigning. Designing for unknown
    possibilities is usually a bad idea.

    In the unlikely case that something else was ever downloaded a new command
    can be added.

    > But I have a different idea - leaving perf download as a plumbing tool,
    > and add --download option into the perf list to call it internally.

    That would be syntactical vinegar.

    > Maybe it can automatically download event file if it didn't find the
    > file in the local cache.

    That wasn't done intentionally, so that people who don't want
    network accesses don't get them.

    -Andi
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2014-06-15 18:41    [W:5.055 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site