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: J0Fwl6I6MACF0ypZmmOx5t/58kyBJ9IdSOp/0BSbDWuDqV6SQuHbYWHBBd0roN+Utf/jz0oNTY
 PatkTTkAgyuA==
IronPort-SDR: GkWkWCIONHSCdtAuLY+8mHQ14PCFEDSwCkg+Em9obOvLwfZf62nLWYl73hz9G0nPVphp9GGSVJ
 3OpXTyUQo7yQ==
Subject: Re: [PATCH] perf stat: Align CSV output for summary mode
To:     Andi Kleen <ak@linux.intel.com>
Cc:     Jiri Olsa <jolsa@redhat.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        jolsa@kernel.org, peterz@infradead.org, mingo@redhat.com,
        alexander.shishkin@linux.intel.com, Linux-kernel@vger.kernel.org,
        kan.liang@intel.com, yao.jin@intel.com
References: <20210316072900.1739-1-yao.jin@linux.intel.com>
 <YFCtfXlaKbIeEMwk@krava> <20210316163421.GQ203350@tassilo.jf.intel.com>
 <YFEBaVNDokSVW/34@kernel.org> <20210316200220.GS203350@tassilo.jf.intel.com>
 <YFEpQs4TnnxPPznF@krava>
 <ea1afc3b-0617-5b21-d0b1-9fd77bbcf0d1@linux.intel.com>
 <20210317013028.GT203350@tassilo.jf.intel.com>
From:   "Jin, Yao" <yao.jin@linux.intel.com>
Message-ID: <6117aa2a-293a-e7e9-a821-2d759697c551@linux.intel.com>
Date:   Wed, 17 Mar 2021 09:39:58 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.8.1
MIME-Version: 1.0
In-Reply-To: <20210317013028.GT203350@tassilo.jf.intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 3/17/2021 9:30 AM, Andi Kleen wrote:
>> Is it serious or just a joke? :)
> 
> I would prefer to not be compatible (at least not until someone complains),
> but if compatibility is required then yes opting in to the broken
> format would be better. Perhaps not with that name.
> 
> And the option could be hidden in the perf config file instead
> of being on the command line.
> 
> -Andi
> 

That makes sense, thanks Andi!

Thanks
Jin Yao