Received-SPF: pass (google.com: domain of linux-kernel+bounces-175279-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <218f9d7c-135a-16a2-3585-4b13d2d39526@amd.com>
Date: Fri, 10 May 2024 10:16:13 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.0
Subject: Re: [RFC 0/4] perf sched: Introduce schedstat tool
To: Namhyung Kim <namhyung@kernel.org>
Cc: peterz@infradead.org, mingo@redhat.com, acme@kernel.org,
 irogers@google.com, swapnil.sapkal@amd.com, mark.rutland@arm.com,
 alexander.shishkin@linux.intel.com, jolsa@kernel.org, rostedt@goodmis.org,
 vincent.guittot@linaro.org, bristot@redhat.com, adrian.hunter@intel.com,
 james.clark@arm.com, kan.liang@linux.intel.com, gautham.shenoy@amd.com,
 kprateek.nayak@amd.com, juri.lelli@redhat.com, yangjihong@bytedance.com,
 linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org,
 santosh.shukla@amd.com, ananth.narayan@amd.com, sandipan.das@amd.com,
 Ravi Bangoria <ravi.bangoria@amd.com>
References: <20240508060427.417-1-ravi.bangoria@amd.com>
 <CAM9d7cgFKcHXeTXxax7GRCK__0U3HUnG3Ls09GpnD8FipyJk1A@mail.gmail.com>
 <0689aa59-426e-2f37-dcc4-d79b1e89403c@amd.com>
 <CAM9d7chvDczhMoOJFLz=MW0ojGm=Ex90CgHhfOsZuLre3vSE3g@mail.gmail.com>
Content-Language: en-US
From: Ravi Bangoria <ravi.bangoria@amd.com>
In-Reply-To: <CAM9d7chvDczhMoOJFLz=MW0ojGm=Ex90CgHhfOsZuLre3vSE3g@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?dWl2VmJyL010TE9NSFRyN241bFRXVHNDOFJWbWMrSlgwNXlxbHM5Qk5sd29F?=
 =?utf-8?B?eGc3UThxMURWelpMQTZFTncraEpuOE1aYVFxVFJXT2MyeHJlQkowT1MrSm8y?=
 =?utf-8?B?cGF0NnVOM3dWWkhDNGFPbzRidzl4MzRPWkZRZDlNN3JzODBKNnFBVXFrajJp?=
 =?utf-8?B?SXFjTlplUVcxRXZtcGhVSFVYMWNxSWwxWTY0aXVpMHBVamtaL2NINU50QVRk?=
 =?utf-8?B?MUJGaTZBSXFkK1pMeUtKUDdTWFlKM05LS08vU0lid1FsSnBBUWM3Y3B3cVFj?=
 =?utf-8?B?bVRCd0tqRmFyVnExZGN1SG5pUkV0T1VtT3JWZ0l3UWVQRnpSY1c5ejU3UXpW?=
 =?utf-8?B?dUxpczNBTE80SDh5K291RGxEeFpIaUx6TStJeC85ZW1qUFVsUklyNkg5bWlh?=
 =?utf-8?B?RXpaOUZyeDYyS1UrNjRHUTl3R2RaM3JDaUFPckN3ck0vY2tGU09oU3orWmJW?=
 =?utf-8?B?M0ZLa01TcVVVam9TWVVwa0NWSWtxVmcyUitoRXJ6dyt2MS9qaWRibUdkYXkz?=
 =?utf-8?B?RHFkWHd1alJBZCttSWVTS1ZIbkpEdW9qS1dINVB1QWV0Q2szUS9uNysxWjBp?=
 =?utf-8?B?QzBlakxBdzE5K3NpUUk0V3FUdW4rc1IwZXVyRGJ2TmZGVU16amhpeDdQTVZz?=
 =?utf-8?B?MG95NVVocXh5ZEFVSyt3SWhOaS92dXhEOG40TkZmaWsxYnhKVWpZNVpOaVk5?=
 =?utf-8?B?cVp6c1ZrM3Vlb1VuSzh6TVl5c1k2Ry9ZeGltWGF0TkZTVnBzbzJseHBXSDNJ?=
 =?utf-8?B?RjBXTkJqM2dHUDhFZkF6R1NKTUx0NDRySzlZaHVWUnlrZkRQUFVRSHJYRHFm?=
 =?utf-8?B?SHRaQXlCQjNkbklncVNxekZDZ0ZCOVdRdVRWUFVJanZqNGU5REt5TVBtS1ZJ?=
 =?utf-8?B?bWxXd1ZWaDlaKzVQVFlMZkhLSlFBRmFYa1dZaTkvRmpKVDZQQ0J0T002OXI4?=
 =?utf-8?B?UWdEREVtbkU5R2pXUUJJeVdPTXBmRDFZMTdPOVRKcW1hU090WjgwUENRc3pu?=
 =?utf-8?B?am9adU5MMTZCQkgrdTNwTFVHM0M1cTVORVZVRkNRVTZESWc5bkROeEhLM2Vh?=
 =?utf-8?B?OEEzRE0xNXdNWk5oQVhXWk1nckhPbTFOVG91MmRjN295azhFS3pTazd0cy9U?=
 =?utf-8?B?Nng2ai9CWHBWR3cwbWc4UnJVcXVPbjFTSWs5ZWRGMWM3Z2t4ZGVzZ3UwOE9h?=
 =?utf-8?B?WTBUbUpWS3U3bmRJb3JuZ21JWjBldlNmZUlEVGNkakg5OTdIZUtSQWdXejFx?=
 =?utf-8?B?cUZRZk1lS2IwdEVwUmYvTGNFbEh0NnV6MlA3QWhNaGZ2VzRqV1hQOGFjM04z?=
 =?utf-8?B?RFlTV2JBTFZzRTFuNlVZcnB2NVhnTUxiSFB5QVBBa050NHhtZzcvdXREY3pJ?=
 =?utf-8?B?UFhFQVNLV0trckNpVVBZVGg3UjJ2eDNSTXYrQ2VqaUx1NUxlVEhmM05yMmJl?=
 =?utf-8?B?V3FNLzdQWUlYbWVqWUJvWDhxTkpJbGw4Nk5KeDRqUVFyMGhEMTFZa0pyK3Zn?=
 =?utf-8?B?dnA5NEVhU1dPb1JjakhqNTlzaGVjSEN6MjBCYUI3LythUERYUm9DSjVRbnlm?=
 =?utf-8?B?SkI5b2xwQWg0cm94cTVKQ3VUSGFjK1pVSi96S3JJVnYveU8xVDNhUytKajRS?=
 =?utf-8?B?MGtrcExqSnBQYk1XUXVsWVhxRzJ5eFlpNG40dHgwbzd1RjdzTE8xNDRoSjdP?=
 =?utf-8?B?NmJnYTNDUHVoQzg5Z2pzcDBaSFM1L05yK0lkVlhUb0IzeFJuNWV4dVd2UnpI?=
 =?utf-8?B?R3dSSHJKWEh2N3V5dm5qSHAvYXNabUttalIyUmpYeE5aYjlpNUlNUWF2ZlBW?=
 =?utf-8?B?cXFGTDFPZ0MwVm5UWEwwZmpjWDNKVG1KVGIrZC9rOENWeldYZ2FMUktZU3pS?=
 =?utf-8?B?aGt1VEJsV2g2QXlmTmFOYTB4ZG5GMUw5alkzMXZ4cEtnUFk3K09ISkpxYzdZ?=
 =?utf-8?B?ckxBWFNiMDg5eEtDcWNLNndNejRPbEwzSURQbDhEajBzS2dEbVp5a1NseHoy?=
 =?utf-8?B?cDRyQ1dEK0hXOXE4b2w0OU1mWW9YcGJZNXNJS2dqdEhzUi81MDQwWFgzQnhZ?=
 =?utf-8?B?U00yR2JDUGxwSzZ1b0oxSFpVQmJXSGFWNEtrS2NUbDR5aHdkaE9mNGROV1c5?=
 =?utf-8?Q?Q1vX9tDdT0j0PigxIr3hwS3S+?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a2d194d2-9a15-400b-4ae1-08dc70ac2786
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB6588.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2024 04:46:28.0770
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: aPsjYLIIyBNG+eQAQZZtR4PdbAPO9NZCJvzRoyD5+8qOPsnzx2HlqCffEAu2sT3qPTDDyACYlTey3tLz8080yQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA0PR12MB8327

>>>> USAGE
>>>> -----
>>>>
>>>>   # perf sched schedstat record
>>>>   # perf sched schedstat report
>>>
>>> Hmm. I think we can remove the duplication in 'sched'. :)
>>
>> You mean `perf sched stat record/report` ?
>>
>>> Given you are thinking of taskstat, how about making it
>>> 'cpustat' instead?
>>
>> Sure. How about:
>>
>>   # perf sched stat --cpu --task record
> 
> If you plan to support both cpu and task at the same time,
> then I'm ok with this.  But if they're mutually exclusive, then
> probably you want to have them as sub-commands.

Sure, will think about it while preparing next version.

Thanks,
Ravi