[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V2 1/3] vl: Allow multiple -overcommit commands
From: |
Zhao Liu |
Subject: |
Re: [PATCH V2 1/3] vl: Allow multiple -overcommit commands |
Date: |
Thu, 30 May 2024 21:39:18 +0800 |
On Fri, May 24, 2024 at 01:00:15PM -0700, Zide Chen wrote:
> Date: Fri, 24 May 2024 13:00:15 -0700
> From: Zide Chen <zide.chen@intel.com>
> Subject: [PATCH V2 1/3] vl: Allow multiple -overcommit commands
> X-Mailer: git-send-email 2.34.1
>
> Both cpu-pm and mem-lock are related to system resource overcommit, but
> they are separate from each other, in terms of how they are realized,
> and of course, they are applied to different system resources.
>
> It's tempting to use separate command lines to specify their behavior.
> e.g., in the following example, the cpu-pm command is quietly
> overwritten, and it's not easy to notice it without careful inspection.
>
> --overcommit mem-lock=on
> --overcommit cpu-pm=on
>
> Fixes: c8c9dc42b7ca ("Remove the deprecated -realtime option")
> Suggested-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Zide Chen <zide.chen@intel.com>
> ---
>
> v2:
>
> Thanks to Thomas' suggestion, changed to this better approach, which
> is more generic and can handle situations like: "enabled the option in
> the config file, and now you'd like to disable it on the command line
> again".
>
> system/vl.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
- [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Zide Chen, 2024/05/24
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Igor Mammedov, 2024/05/28
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Chen, Zide, 2024/05/28
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Igor Mammedov, 2024/05/29
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Chen, Zide, 2024/05/29
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Zhao Liu, 2024/05/30
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Igor Mammedov, 2024/05/30
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Sean Christopherson, 2024/05/30
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Igor Mammedov, 2024/05/30
- Re: [PATCH V2 0/3] improve -overcommit cpu-pm=on|off, Chen, Zide, 2024/05/30