xianlubird / mydocker

<<自己动手写docker>> 源码
Apache License 2.0
1.98k stars 560 forks source link

code 4.1中的问题以及一些其他建议 #89

Open shejialuo opened 2 years ago

shejialuo commented 2 years ago

https://github.com/xianlubird/mydocker/blob/8ea47de6bf52b3a116b51e8e094500fdc3d9d173/run.go#L22-L28

上述代码中的os.Exit(0)是不应该存在的,cgroupManager.Destroy()并不会执行。

其他的一些建议:

整个仓库的代码的Tag有点混乱,比如说code3.3的tag代码是不全的,是直接添加到code4.1中修复的。如果有时间的话,还是希望作者能够整理一下。再次感谢作者的贡献,这本书写的很好,弥补了我实战的缺少。