Received-SPF: pass (google.com: domain of linux-kernel+bounces-136127-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Huan Yang <link@vivo.com>
To: Tejun Heo <tj@kernel.org>,
	Zefan Li <lizefan.x@bytedance.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	cgroups@vger.kernel.org,
	linux-kernel@vger.kernel.org
Cc: opensource.kernel@vivo.com,
	Huan Yang <link@vivo.com>
Subject: [PATCH] cgroup: make cgroups info more readable
Date: Tue,  9 Apr 2024 10:18:26 +0800
Message-Id: <20240409021826.1171921-1-link@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?Fjtw0NL2jyv/BEyj+iqDC1gDxxxP8f80CpE0unTGgnf2IxeCtEy4obxvJWHG?=
 =?us-ascii?Q?wpzJ8cS5W6d6l8Smknow8uX2V5lGu6liQ6qAv/4dE/WTuo9m4BqnxDJRBiPB?=
 =?us-ascii?Q?El+OJuV5wmNWi2GlJ3lFHNSejYTwnf7L1deXRONCdmPglMeEzRFzmU7WxBGI?=
 =?us-ascii?Q?TY6YWPFHp9tJcvXhMHC0eKNzQd5B0ND7uPAoStFwx0gOqa/rOdBLWdjhwR12?=
 =?us-ascii?Q?8oJGYHgy1xNmgoknrpDba/YYOuZK0ttmMNd4sERnpI5B/kLGFzurcn0X7zQn?=
 =?us-ascii?Q?Ox+QrvuMHc0m7KLKQkcDWy53EoicgCQOuZtKy9cVIgwCmds9cdT8mmPBtXCF?=
 =?us-ascii?Q?ZlYV2qvNr+iPDyH/Qd7zrfagLzrcjZ7Vp7cyVTRbjDi4oK41sHoEf+R/4VsJ?=
 =?us-ascii?Q?FMtwExrr/hMR+X069OQFftUKNip5oQqM+9B/W12xlzd4mpsr+/a2dzBU+NX/?=
 =?us-ascii?Q?XFe+sDmF2exVSZtCEN0hB3TEJpa6xgg7Kutue9DdiDlOGNecCBaIjHcPNJfU?=
 =?us-ascii?Q?Vr9g8riFxIzHsaSNEI8mAoYfWas9LqJt7LbTgNgr68Lua7Szml+dYHSVrGSX?=
 =?us-ascii?Q?+VjZ8+tHfru5k1gPDtnYYhQ+gZ0PtNDLa8m/TToJ7o2IhTxMrn43WDDay6ue?=
 =?us-ascii?Q?OHedZ97YSICVwM1uMToTKdzDChruIFIUfCTgB89PaXPwUYKXGlXdoZBLLgTM?=
 =?us-ascii?Q?kLTP5Kt47NjuzDNUAPH9pS2RPig69xB5UV2J8uisGBEIF5MDsNbCtKZ4uam4?=
 =?us-ascii?Q?neByHJwdL8prKBTPzboU+f66aBC3rEMrHluOjAiJ36eljoOwcSHcCsC3e+Ew?=
 =?us-ascii?Q?VrsTTjxElyOSMsRoC8fE2fTZgSNagrKPIMr/J+3GcY+BGRCEbd5WBdUjJiis?=
 =?us-ascii?Q?Nu81K1mSjTnQiUTFWD5ZjPYwkwFHE1VXHL1iXtvcPzCps5e43mrm82zNKHBj?=
 =?us-ascii?Q?sFyyjhieA9KgSHv6y0s2K1GdjCyUvimFir/qPOVg+f18S9oti4Vu+R5uBY7V?=
 =?us-ascii?Q?zO5QD9wkshKbj71KvSkMzEICIjjC/E/O70LwyWo5dGb4Xg/R0Zs5jZUJwdSH?=
 =?us-ascii?Q?1rY6eJnWbfNP0iK/BZGbj2ZxHw8hDcFr6guj2Tcmcp845Gvtam32j2Ovc7qo?=
 =?us-ascii?Q?sQ811EdGP/wsto6p4gBxuLsiYZcvoO1qCy4SlB8MjKqoqOshB8nZIfmAKePJ?=
 =?us-ascii?Q?g8siySv+0Z8PuzLO/hGczoKMtQP+u4ueDEm1TwfWYRGhR4klqik2lWQO1art?=
 =?us-ascii?Q?/z+7RmPLvR7yesdPlLKdHSxbfwrYP0vlKySQmX1E1JpnvAoc6AxBVThsPLRT?=
 =?us-ascii?Q?bRN6ujKBfEhM4k0sydUp6RbuqnerMYo/KEEzQtOvJAatYnGrdFQL225HGBm0?=
 =?us-ascii?Q?d3BNvA2qAxL76WWOBQpxhcVUpS4f8DixlAayB4x2GH/OlGdQ6a8jckDj/lUl?=
 =?us-ascii?Q?lNpE6PXyL120HMYn1MB9bH54KrnOTfBESJKsxDzRw3jAul7re0NEj6MArH32?=
 =?us-ascii?Q?d0cANMyyMjDCymT8iKyf/ATdoiYi+jXfxqiB/Vs8poSMCIC5Hhhp1b1eX4tE?=
 =?us-ascii?Q?MNGGrqsBDvHyd/8HlUUPJiJwqiPeInhay7HJG9O4?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f3e7db00-7045-4dba-ace3-08dc583b5f9b
X-MS-Exchange-CrossTenant-AuthSource: PUZPR06MB5676.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2024 02:18:41.2040
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: r5pnZQo01KFrX8d25E2/XLNQxN0luL+C6G3wlxkWQmqQxtDnXRG0gKR/ttbo41EKw4lSdMxEP84xze6qlgllCA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: JH0PR06MB6677

The current cgroups output format is based on tabs, which
may cause misalignment of output information.

Using placeholder formatting can make the output information
more readable.

Signed-off-by: Huan Yang <link@vivo.com>
---
 kernel/cgroup/cgroup-v1.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/kernel/cgroup/cgroup-v1.c b/kernel/cgroup/cgroup-v1.c
index 520a11cb12f4..c082a78f4c22 100644
--- a/kernel/cgroup/cgroup-v1.c
+++ b/kernel/cgroup/cgroup-v1.c
@@ -669,15 +669,16 @@ int proc_cgroupstats_show(struct seq_file *m, void *v)
 	struct cgroup_subsys *ss;
 	int i;
 
-	seq_puts(m, "#subsys_name\thierarchy\tnum_cgroups\tenabled\n");
+	seq_printf(m, "%16s %16s %16s %16s\n", "#subsys_name", "hierarchy",
+		   "num_cgroups", "enabled");
 	/*
 	 * Grab the subsystems state racily. No need to add avenue to
 	 * cgroup_mutex contention.
 	 */
 
 	for_each_subsys(ss, i)
-		seq_printf(m, "%s\t%d\t%d\t%d\n",
-			   ss->legacy_name, ss->root->hierarchy_id,
+		seq_printf(m, "%16s %16d %16d %16d\n", ss->legacy_name,
+			   ss->root->hierarchy_id,
 			   atomic_read(&ss->root->nr_cgrps),
 			   cgroup_ssid_enabled(i));
 
-- 
2.34.1