Hello, KAME. On Sat, Apr 28, 2012 at 06:31:38PM +0900, Hiroyuki Kamezawa wrote: > > So, IMHO, just making pre_destroy() clean up its own charges and > > always returning 0 is enough. There's no need to fix up old > > non-critical race condition at this point in the patch stream. cgroup > > rmdir simplification will make them disappear anyway. > > So, hmm, ok. I'll drop patch 7 & 8. memcg may return -EBUSY in very very > race case but users will not see it in the most case. > I'll fix limit, move-charge and use_hierarchy problem first. IIUC, memcg can just return 0 when child creation races against pre_destroy(). cgroup will retry if child exists after pre_destroy() completion. If child comes and goes before cgroup checks its existence, some charges may be lost but that race already exists and it will be gone once the retry logic is removed. Also, returning -errno will trigger WARN_ON() w/o the legacy behavior flag. Thank you very much. -- tejun -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>