> On 十月 9, 2016, 5:16 a.m., Guangya Liu wrote: > > src/slave/containerizer/docker.cpp, lines 2105-2109 > > <https://reviews.apache.org/r/50841/diff/10/?file=1507913#file1507913line2105> > > > > Not yours, but I think that the comments can be removed as we have the > > logic of killing those containers forcely in #2118 and #2123. You can fix > > this in a separate patch.
Will move to another patch > On 十月 9, 2016, 5:16 a.m., Guangya Liu wrote: > > src/slave/containerizer/docker.cpp, lines 2162-2182 > > <https://reviews.apache.org/r/50841/diff/10/?file=1507913#file1507913line2162> > > > > What about moving this to a new function named as `____destroy` and use > > continuation for `deallocateNvidiaGpus` in #2159? Fixed, thanks! > On 十月 9, 2016, 5:16 a.m., Guangya Liu wrote: > > src/slave/containerizer/docker.cpp, lines 1554-1564 > > <https://reviews.apache.org/r/50841/diff/10/?file=1507913#file1507913line1554> > > > > I think we do not need the update here as `destroy` will always be > > called even if the container exit normally, relasing the GPU in > > `___destroy` is good enough. removed - Yubo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50841/#review151477 ----------------------------------------------------------- On 九月 22, 2016, 6:16 a.m., Yubo Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50841/ > ----------------------------------------------------------- > > (Updated 九月 22, 2016, 6:16 a.m.) > > > Review request for mesos, Benjamin Mahler, Guangya Liu, Kevin Klues, and > Rajat Phull. > > > Bugs: MESOS-5795 > https://issues.apache.org/jira/browse/MESOS-5795 > > > Repository: mesos > > > Description > ------- > > Added control logic to allocate/deallocate GPUs to GPU-related task > when the task is started/terminated. > > > Diffs > ----- > > src/slave/containerizer/docker.hpp 8da63101f951892e673612134770fc155d86112d > src/slave/containerizer/docker.cpp 1d27761fcb3f310cf954d45ed41f4c89ecbd5982 > > Diff: https://reviews.apache.org/r/50841/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Yubo Li > >