Re: [PATCH] freezer, sched: report the frozen task stat as 'D'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Chen Ridong <chenridong@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
From: Michal Koutný <mkoutny@xxxxxxxx>
Date: Wed, 11 Dec 2024 17:02:29 +0100
Cc: Tejun Heo <tj@xxxxxxxxxx>, peterz@xxxxxxxxxxxxx, Valentin Schneider <vschneid@xxxxxxxxxx>, mingo@xxxxxxxxxx, juri.lelli@xxxxxxxxxx, vincent.guittot@xxxxxxxxxx, dietmar.eggemann@xxxxxxx, rostedt@xxxxxxxxxxx, bsegall@xxxxxxxxxx, mgorman@xxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, wangweiyang2@xxxxxxxxxx, cgroups@xxxxxxxxxxxxxxx
In-reply-to: <c56b2347-7475-4190-85a5-a38954ae9c08@huaweicloud.com>
References: <20241111135431.1813729-1-chenridong@huaweicloud.com> <xhsmhv7wrb3sc.mognet@vschneid-thinkpadt14sgen2i.remote.csb> <4f78d752-52ab-493d-8bf5-f12dc4f554c8@huaweicloud.com> <ZzYo19k9ZvkC7V-1@slm.duckdns.org> <2f755161-ec7e-4785-b0ca-ea68c01785a2@huaweicloud.com> <ZzajsLHrXXtYk04l@slm.duckdns.org> <3b03520e-775d-416a-91b1-1d78f3e91b1d@huaweicloud.com> <c56b2347-7475-4190-85a5-a38954ae9c08@huaweicloud.com>

Hi.

On Wed, Dec 04, 2024 at 09:52:15AM GMT, Chen Ridong <chenridong@xxxxxxxxxxxxxxx> wrote:
> Does anyone have any opinions?

When an 'R' task is frozen (v1) and it's shown as (unchanged) 'R' it's
acceptable.
When an 'S' task is equally frozen and its apparent state switches to
'R', I agree it's confusing.

Showing 'D' state in any case of a frozen task (like before that rework)
makes sense too. Please add Fixes: if you need to restore this behavior.

HTH,
Michal

Attachment: signature.asc
Description: PGP signature

Follow-Ups:
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Chen Ridong

References:
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Chen Ridong
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Tejun Heo
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Chen Ridong
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Tejun Heo
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Chen Ridong
- Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
  - From: Chen Ridong

Prev by Date: [PATCH] memcg: allow exiting tasks to write back data to swap
Next by Date: Re: [PATCH 2/2] vmalloc: Account memcg per vmalloc
Previous by thread: Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
Next by thread: Re: [PATCH] freezer, sched: report the frozen task stat as 'D'
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API] [Monitors]