Skip to content

Commit

Permalink
sched: convert sys_sched_getaffinity() to cpumask_var_t.
Browse files Browse the repository at this point in the history
Impact: stack usage reduction

Dynamically allocating cpumasks (when CONFIG_CPUMASK_OFFSTACK) saves
space in the stack.  cpumask_var_t is just a struct cpumask for
!CONFIG_CPUMASK_OFFSTACK.

Some jiggling here to make sure we always exit at the bottom (so we hit
the free_cpumask_var there).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
  • Loading branch information
Rusty Russell authored and Ingo Molnar committed Nov 24, 2008
1 parent a0e9024 commit f17c860
Showing 1 changed file with 13 additions and 8 deletions.
21 changes: 13 additions & 8 deletions kernel/sched.c
Original file line number Diff line number Diff line change
Expand Up @@ -5499,19 +5499,24 @@ asmlinkage long sys_sched_getaffinity(pid_t pid, unsigned int len,
unsigned long __user *user_mask_ptr)
{
int ret;
cpumask_t mask;
cpumask_var_t mask;

if (len < sizeof(cpumask_t))
if (len < cpumask_size())
return -EINVAL;

ret = sched_getaffinity(pid, &mask);
if (ret < 0)
return ret;
if (!alloc_cpumask_var(&mask, GFP_KERNEL))
return -ENOMEM;

if (copy_to_user(user_mask_ptr, &mask, sizeof(cpumask_t)))
return -EFAULT;
ret = sched_getaffinity(pid, mask);
if (ret == 0) {
if (copy_to_user(user_mask_ptr, mask, cpumask_size()))
ret = -EFAULT;
else
ret = cpumask_size();
}
free_cpumask_var(mask);

return sizeof(cpumask_t);
return ret;
}

/**
Expand Down

0 comments on commit f17c860

Please sign in to comment.