linux-kernel - Re: [PATCH v2 4/7] per-cgroup tcp buffers control

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CANaxB-wy8VDv0Wjni6UzcfBzSgNn=bZBey5f+fXHebNuek=O1A@mail.gmail.com>
Date:	Wed, 28 Sep 2011 15:58:13 +0400
From:	Andrew Wagin <avagin@...il.com>
To:	Glauber Costa <glommer@...allels.com>
Cc:	linux-kernel@...r.kernel.org, paul@...lmenage.org,
	lizf@...fujitsu.com, kamezawa.hiroyu@...fujitsu.com,
	ebiederm@...ssion.com, davem@...emloft.net, gthelen@...gle.com,
	netdev@...r.kernel.org, linux-mm@...ck.org
Subject: Re: [PATCH v2 4/7] per-cgroup tcp buffers control

* tcp_destroy_cgroup_fill() is executed for each cgroup and
initializes some proto methods. proto_list is global and we can
initialize each proto one time. Do we need this really?

* And when a cgroup is destroyed, it cleans proto methods
(tcp_destroy_cgroup_fill), how other cgroups will work after that?

* What about proto, which is registered when cgroup mounted?

My opinion that we may initialize proto by the following way:

+#ifdef CONFIG_CGROUP_MEM_RES_CTLR_KMEM+       .enter_memory_pressure
= tcp_enter_memory_pressure_nocg,
+       .sockets_allocated      = sockets_allocated_tcp_nocg,
+       .memory_allocated       = memory_allocated_tcp_nocg,
+       .memory_pressure        = memory_pressure_tcp_nocg,
+#else
        .enter_memory_pressure  = tcp_enter_memory_pressure,
        .sockets_allocated      = sockets_allocated_tcp,
        .memory_allocated       = memory_allocated_tcp,
        .memory_pressure        = memory_pressure_tcp,
+#endif

It should work, because the root memory cgroup always exists.

>+int tcp_init_cgroup_fill(struct proto *prot, struct cgroup *cgrp,
>+                        struct cgroup_subsys *ss)
>+{
>+       prot->enter_memory_pressure     = tcp_enter_memory_pressure;
>+       prot->memory_allocated          = memory_allocated_tcp;
>+       prot->prot_mem                  = tcp_sysctl_mem;
>+       prot->sockets_allocated         = sockets_allocated_tcp;
>+       prot->memory_pressure           = memory_pressure_tcp;
>+
>+       return 0;
>+}


> +void tcp_destroy_cgroup_fill(struct proto *prot, struct cgroup *cgrp,
> +                            struct cgroup_subsys *ss)
> +{
> +       prot->enter_memory_pressure     = tcp_enter_memory_pressure_nocg;
> +       prot->memory_allocated          = memory_allocated_tcp_nocg;
> +       prot->prot_mem                  = tcp_sysctl_mem_nocg;
> +       prot->sockets_allocated         = sockets_allocated_tcp_nocg;
> +       prot->memory_pressure           = memory_pressure_tcp_nocg;
>

>@@ -2220,12 +2220,16 @@ struct proto tcpv6_prot = {
>       .hash                   = tcp_v6_hash,
>       .unhash                 = inet_unhash,
>       .get_port               = inet_csk_get_port
> +       .enter_memory_pressure  = tcp_enter_memory_pressure_nocg,
> +       .sockets_allocated      = sockets_allocated_tcp_nocg,
> +       .memory_allocated       = memory_allocated_tcp_nocg,
> +       .memory_pressure        = memory_pressure_tcp_nocg,
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/