Messages in this thread | | | Subject | Re: [PATCH 4/6] perf_counter: Add PERF_COUNTER_IOC_SET_FILTER ioctl | From | Peter Zijlstra <> | Date | Mon, 07 Sep 2009 18:55:45 +0200 |
| |
On Mon, 2009-09-07 at 18:48 +0200, Ingo Molnar wrote: > * Peter Zijlstra <peterz@infradead.org> wrote: > > > On Mon, 2009-09-07 at 16:13 +0800, Li Zefan wrote: > > > Allow to set profile filter via ioctl. > > > > Hrm,.. not at all sure about this.. what are the ABI implications? > > I think the ABI should be fine if it's always a sub-set of C syntax. > That would be C expressions initially. Hm?
Right, so I've no clue what filter expressions look like, and the changelog doesn't help us at all. It doesn't mention its a well considered decision to henceforth freeze the expression syntax.
Of course, since filters so far only work with tracepoint things, and since you can only come by tracepoint things through debugfs, and since anything debugfs is basically a free-for-all ABI-less world, we might be good, but then this is a very ill-defined ioctl() indeed.
So please, consider this well -- there might not be a second chance.
| |