Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: fEUDugv1nyYhPtYS6tusqzkj92W8+9ImKPJb1yhwCDUQE1S8jNenRLd9jIGd47SNgp1SXziBWF
 ivPCVYx9N6CA==
IronPort-SDR: 8UlM6QoB4Cha/TQLxBcu+GBUwJpDUcClXtoLykjmGNpdtIBzSvsS1eU2RYPL6M7vH76YujtI7q
 c+loA2epoNqg==
Date:   Tue, 20 Oct 2020 13:29:13 -0700
From:   Andi Kleen <ak@linux.intel.com>
To:     Alexey Budankov <alexey.budankov@linux.intel.com>
Cc:     Arnaldo Carvalho de Melo <acme@kernel.org>,
        Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Adrian Hunter <adrian.hunter@intel.com>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v1 02/15] perf report: output trace file name in raw
 trace dump
Message-ID: <20201020202913.GN466880@tassilo.jf.intel.com>
References: <810f3a69-0004-9dff-a911-b7ff97220ae0@linux.intel.com>
 <87e2050b-37e6-8ed8-e1e0-cfa074b030fa@linux.intel.com>
 <20201012160144.GA466880@tassilo.jf.intel.com>
 <42d5395a-71f5-63e1-f7c9-fd76e2b8bee4@linux.intel.com>
 <a1c21077-0383-aa29-cc79-a8921916c63f@linux.intel.com>
 <19dd0fa0-c236-6042-8ba6-48b4696e66bb@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <19dd0fa0-c236-6042-8ba6-48b4696e66bb@linux.intel.com>
Precedence: bulk

> > After brief evaluation it still doesn't look easy. The simplest thing
> > I could imagine is to probably combine file_path and file_offset at a
> > struct object on stack and then pass the object by the reference along
> > function calls. I expect it will roughly cause the same amount of changes
> > in the code and saves one argument (GP register). It is not that much
> > but still. However I don't see issues with passing even 6 args on stack.
> 
> Sorry - "passing 6 args to a function call"

Ah it wasn't about code efficiency, just maintainability. Function calls
with too many arguments are generally hard to maintain, and typically
at some point have to be refactored anyways because they tend to grow
even more over time.

But if it's too difficult, ok.

-Andi