Hello, On Fri, Oct 21, 2022 at 05:32:58PM -0700, Yonghong Song wrote: > > LGTM, but I do think that BPF_MAP_TYPE_CG_STORAGE and "cg_storage" is > > easier to read and talk about. But that's minor. > > I searched kernel/cgroup/* and kernel/bpf/cgroup.c and > include/linux/cgroup*.h. The 'cgrp' for abbreviation of 'cgroup' is much > more than > 'cg' for 'cgroup' unless 'cg' appears in like 'memcg' or 'rdmacg'. So I > would just use 'cgrp' for now. Yeah, cgrp is more consistent for prefixes and variable names. cg is usually used as a part of an abbreviated word - memcg, blkcg, cgid and so on. Thanks. -- tejun
