Comment 7 for bug 827279

Revision history for this message
Jamie Strandboge (jdstrand) wrote :

While the patch seems generally ok, I am not familiar with cgroup code and would prefer if an upstream bug was filed with their ACK before applying the patch. Specifically, these bits:
+- if (ret < 0)
+- break;
++ if (ret < 0) {
++ if (errno != ESRCH)
++ break;
++ // the task just went away, ignore it
++ ret = 0;
++ continue;
++ }

seems to be a pretty big departure from the original where we are breaking only on ESRCH instead of all errors.