On Fri, May 05, 2023 at 05:45:58AM -1000, Tejun Heo <tj@xxxxxxxxxx> wrote: > > There are three relevant nodes for each cgroupfs entry: > > > > R ... cgroup hierarchy root > > M ... mount root > > C ... reader's cgroup NS root > > > > mountinfo is supposed to show path from C to M. > > At least for cgroup2, the path from C to M isn't gonna change once NS is > established, right? Right. Although, the argument about M (when C above M or when C and M in different subtrees) implicitly relies on the namespace_sem. > Nothing can be moved or renamed while the NS root is there. If so, > can't we just cache the formatted path and return the same thing > without any locking? Here I find the caching complexity not worth it for v2 only (also C is in the eye of the beholder) -- because then css_set_lock can be dropped from cgroup_show_path with simpler reasoning. (Sadly, the bigger benefit would be on hybrid setups multiplied by the number of v1 hierarchies listed.) (OTOH, the caching argument and weights for it may be different for /proc/$pid/cgroup.) > The proposed changes seem a bit too brittle to me. OK, I'll look into separate cgroup_show_path and cgroup1_show_path approach. Thanks, Michal
Attachment:
signature.asc
Description: PGP signature
