lkml.org 
[lkml]   [2023]   [Sep]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATH v2] cgroup: add cgroup_favordynmods= command-line option
From

On 9/5/23 20:57, Luiz Capitulino wrote:
> We have a need of using favordynmods with cgroup v1, which doesn't support
> changing mount flags during remount. Enabling CONFIG_FAVOR_DYNMODS at
> build-time is not an option because we want to be able to selectively
> enable it for certain systems.
>
> This commit addresses this by introducing the cgroup_favordynmods=
> command-line option. This option works for both cgroup v1 and v2 and
> also allows for disabling favorynmods when the kernel built with
> CONFIG_FAVOR_DYNMODS=y.
>
> Signed-off-by: Luiz Capitulino <luizcap@amazon.com>
> ---
> Documentation/admin-guide/kernel-parameters.txt | 4 ++++
> kernel/cgroup/cgroup.c | 14 +++++++++++---
> 2 files changed, 15 insertions(+), 3 deletions(-)
>
> o v2
> - Use __ro_after_init [Waiman]
>
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index 0a1731a0f0ef..8b744d39d393 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -580,6 +580,10 @@
> named mounts. Specifying both "all" and "named" disables
> all v1 hierarchies.
>
> + cgroup_favordynmods= [KNL] Enable or Disable favordynmods.
> + Format: { "true" | "false" }
> + Defaults to the value of CONFIG_CGROUP_FAVOR_DYNMODS.
> +
> cgroup.memory= [KNL] Pass options to the cgroup memory controller.
> Format: <string>
> nosocket -- Disable socket memory accounting.
> diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c
> index 1fb7f562289d..2b7d74304606 100644
> --- a/kernel/cgroup/cgroup.c
> +++ b/kernel/cgroup/cgroup.c
> @@ -207,6 +207,8 @@ static u16 have_exit_callback __read_mostly;
> static u16 have_release_callback __read_mostly;
> static u16 have_canfork_callback __read_mostly;
>
> +static bool have_favordynmods __ro_after_init = IS_ENABLED(CONFIG_CGROUP_FAVOR_DYNMODS);
> +
> /* cgroup namespace for init task */
> struct cgroup_namespace init_cgroup_ns = {
> .ns.count = REFCOUNT_INIT(2),
> @@ -2243,9 +2245,9 @@ static int cgroup_init_fs_context(struct fs_context *fc)
> fc->user_ns = get_user_ns(ctx->ns->user_ns);
> fc->global = true;
>
> -#ifdef CONFIG_CGROUP_FAVOR_DYNMODS
> - ctx->flags |= CGRP_ROOT_FAVOR_DYNMODS;
> -#endif
> + if (have_favordynmods)
> + ctx->flags |= CGRP_ROOT_FAVOR_DYNMODS;
> +
> return 0;
> }
>
> @@ -6764,6 +6766,12 @@ static int __init enable_cgroup_debug(char *str)
> }
> __setup("cgroup_debug", enable_cgroup_debug);
>
> +static int __init cgroup_favordynmods_setup(char *str)
> +{
> + return (kstrtobool(str, &have_favordynmods) == 0);
> +}
> +__setup("cgroup_favordynmods=", cgroup_favordynmods_setup);
> +
> /**
> * css_tryget_online_from_dir - get corresponding css from a cgroup dentry
> * @dentry: directory dentry of interest
Reviewed-by: Waiman Long <longman@redhat.com>

\
 
 \ /
  Last update: 2023-09-06 03:55    [W:0.157 / U:0.492 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site