Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-07 Thread Yun Hsiang
On Mon, Oct 05, 2020 at 06:15:00PM +0100, Qais Yousef wrote: > On 10/05/20 18:58, Patrick Bellasi wrote: > > [...] > > > >> it can not go back to the initial state to let the module(group) control. > > > > > > In case A changes its values e.g. from 3a to 3b it will go back to be > > > controlled

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-05 Thread Qais Yousef
On 10/05/20 18:58, Patrick Bellasi wrote: [...] > >> it can not go back to the initial state to let the module(group) control. > > > > In case A changes its values e.g. from 3a to 3b it will go back to be > > controlled by /TG again (like it was when it had no user defined > > values). > > True,

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-05 Thread Patrick Bellasi
Hi Yun, Dietmar, On Mon, Oct 05, 2020 at 14:38:18 +0200, Dietmar Eggemann wrote... > + Patrick Bellasi > + Qais Yousef > > On 02.10.20 07:38, Yun Hsiang wrote: >> On Wed, Sep 30, 2020 at 03:12:51PM +0200, Dietmar Eggemann wrote: > > [...] > >>> On 28/09/2020 10:26, Yun Hsiang wrote: If

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-05 Thread Pavan Kondeti
On Fri, Oct 02, 2020 at 01:38:12PM +0800, Yun Hsiang wrote: > On Wed, Sep 30, 2020 at 03:12:51PM +0200, Dietmar Eggemann wrote: > Hi Dietmar, > > > Hi Yun, > > > > On 28/09/2020 10:26, Yun Hsiang wrote: > > > If the user wants to release the util clamp and let cgroup to control it, > > > we need

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-05 Thread Dietmar Eggemann
+ Patrick Bellasi + Qais Yousef On 02.10.20 07:38, Yun Hsiang wrote: > On Wed, Sep 30, 2020 at 03:12:51PM +0200, Dietmar Eggemann wrote: [...] >> On 28/09/2020 10:26, Yun Hsiang wrote: >>> If the user wants to release the util clamp and let cgroup to control it, >>> we need a method to reset.

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-10-01 Thread Yun Hsiang
On Wed, Sep 30, 2020 at 03:12:51PM +0200, Dietmar Eggemann wrote: Hi Dietmar, > Hi Yun, > > On 28/09/2020 10:26, Yun Hsiang wrote: > > If the user wants to release the util clamp and let cgroup to control it, > > we need a method to reset. > > > > So if the user set the task uclamp to the defaul

Re: [PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-09-30 Thread Dietmar Eggemann
Hi Yun, On 28/09/2020 10:26, Yun Hsiang wrote: > If the user wants to release the util clamp and let cgroup to control it, > we need a method to reset. > > So if the user set the task uclamp to the default value (0 for UCLAMP_MIN > and 1024 for UCLAMP_MAX), reset the user_defined flag to release

[PATCH 1/1] sched/uclamp: release per-task uclamp control if user set to default value

2020-09-28 Thread Yun Hsiang
If the user wants to release the util clamp and let cgroup to control it, we need a method to reset. So if the user set the task uclamp to the default value (0 for UCLAMP_MIN and 1024 for UCLAMP_MAX), reset the user_defined flag to release control. Signed-off-by: Yun Hsiang --- kernel/sched/cor