<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Chen, Hanxiao <<a href="mailto:chenhx.fnst@cn.fujitsu.com" target="_blank">chenhx.fnst@cn.fujitsu.com</a>> wrote:<br></div><div dir="ltr" class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="ZH-CN">
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u><span class="gmail_default" style="font-size:small">...</span></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">>@Li, I think this patch is ok, Do you have some comment about it?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">>I'm ok to go with memory.use_hierarchy checking in the preconditioning phase.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">>But how can you assert the default memory cgroup is mount at path: "/sys/fs/cgroup/memory", is there any possibility the default path mount at other places(for different distribution)?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><a href="https://github.com/torvalds/linux/blob/b5fc7a89e58bcc059a3d5e4db79c481fb437de59/kernel/cgroup/cgroup.c#L5768" target="_blank"><span style="color:rgb(31,73,125);text-decoration:none">https://github.com/torvalds/linux/blob/b5fc7a89e58bcc059a3d5e4db79c481fb437de59/kernel/cgroup/cgroup.c#L5768</span></a><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">WARN_ON(sysfs_create_mount_point(fs_kobj, "cgroup"));<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">So the default mount place should be /sys/fs/cgroup if distributions<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">did not modify these lines.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">As the discussion above,
<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">mount -t cgroup -omemory memcg /dev/memcg<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">kernel will find a suitable cgroup root for us.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US" style="color:rgb(31,73,125)">How about reading /dev/memcg/memory.use_hierarchy as Yang recommended instead?</span></p></div></div></div></div></div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Sure.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">But also, shouldn't we check if the '/dev/memcg' has sub-directory</div><div class="gmail_default" style="font-size:small">before changing the memory.use_hiearchy in memcg_lib.sh?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">As you noted in the description, enabling/disabling will fail if either </div><div class="gmail_default" style="font-size:small">the cgroup already has other cgroups created below it.</div></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>