lkml.org 
[lkml]   [2020]   [Oct]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v1 02/15] perf report: output trace file name in raw trace dump
From
Date

On 12.10.2020 19:01, Andi Kleen wrote:
> On Mon, Oct 12, 2020 at 11:54:24AM +0300, Alexey Budankov wrote:
>> @@ -180,7 +183,8 @@ static int ordered_events__deliver_event(struct ordered_events *oe,
>> ordered_events);
>>
>> return perf_session__deliver_event(session, event->event,
>> - session->tool, event->file_offset);
>> + session->tool, event->file_offset,
>> + event->file_path);
>
> Wouldn't it be better to pass "event" around now, which would contain at least
> four of the arguments.
>
> These functions are getting entirely too many arguments.

Well, either approach is possible, and even shrink of two arguments kept at session object.
However changing function signature more than posted can cause bigger adjustments all over
the code. So this needs more evaluation prior implementation.

Alexei

\
 
 \ /
  Last update: 2020-10-12 19:07    [W:2.305 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site