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
