On Mon, Jun 15, 2015 at 03:12:21PM +0200, Peter Zijlstra wrote:
> On Mon, Mar 02, 2015 at 12:22:03AM +0900, Sergey Senozhatsky wrote:
>
> > +++ b/lib/cpumask.c
> > @@ -37,10 +37,11 @@ EXPORT_SYMBOL(__next_cpu_nr);
> > int cpumask_next_and(int n, const struct cpumask *src1p,
> > const struct cpumask *src2p)
> > {
> > + struct cpumask tmp;
> > +
> > + if (cpumask_and(&tmp, src1p, src2p))
> > + return cpumask_next(n, &tmp);
> > + return nr_cpu_ids;
> > }
> > EXPORT_SYMBOL(cpumask_next_and);
>
> Just ran into this; I though we were not supposed to put cpumasks on the
> stack because $BIG. ?!
>
> explain.
That's some fat stack with 8K CPUs:
cpumask_next_and:
pushq %rbp #
movq %rsp, %rbp #,
pushq %rbx #
movl %edi, %ebx # n, n
leaq -1040(%rbp), %rdi #, tmp106
subq $1032, %rsp #,
^^^^^
Lovely.
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
