Bugzilla – Bug 10
cpuset_delete() return value treated incorrectly everywhere
Last modified: 2009-07-22 00:40:34 MDT
You need to log in before you can comment on or make changes to this bug.
Created an attachment (id=2) [details] Patch to change cpuset_delete() to just return the return code from rmdir() Currently all calls to cpuset_delete() treat the return value as if it is the return from rmdir(), when it is actually comparing that with 0 and thus inverting the return code. The symptom of this are lots of bogus messages about not being able to delete unused cpusets which are actually successfully being removed (fortunately that's the only effect)! The fix is trivial, just make cpuset_delete() return the value from rmdir() itself - no other changes are necessary. NOTE: This is different to how things are done in trunk, there there are lots of comparisons to 0 for its return value, so problems may still lurk there!
this patch has been applied. Is it OK to resolve, or should we check trunk to see what is going on there?
I think trunk could do with some attention for this. I'm on leave for the next week so may or may not get a chance to look at it.
Just had a look at trunk, it's far more sane, nothing needs doing there. I'll resolve this as fixed now, thanks Glen!