Messages in this thread | | | Date | Fri, 19 Apr 2013 11:36:18 -0700 | From | Tejun Heo <> | Subject | Re: [PATCH 05/10] cpuset: don't update tasks' cpumask and nodemask in an empty cpuset |
| |
Hello, Li.
On Fri, Apr 19, 2013 at 08:27:05PM +0800, Li Zefan wrote: > I think this was introduced unintentionally when cpuset hotplug was > made asynchronous. Fortunately it does no harm, as updating tasks' > cpumask will just return failure and there's a guarantee_online_mems() > when updating nodemask, and then the tasks will be moved to an ancestor > cpuset.
Yeah, which will update the masks to the proper values anyway, so it was intentionally written that way as it didn't really matter either way. I suppose this helps future changes? Maybe update the description a bit?
Thanks.
-- tejun
| |