Re: [PATCH 1/2] x86/fpu, fork: Dynamically allocate 'struct fpu'

[Ingo Molnar <mingo@kernel.org>]

* Ingo Molnar <mingo@kernel.org> wrote:

> From: Dave Hansen <dave@sr71.net>
> 
> The FPU rewrite removed the dynamic allocations of 'struct fpu'.
> But, this potentially wastes massive amounts of memory (2k per
> task on systems that do not have AVX-512 for instance).
> 
> Instead of having a separate slab, this patch just appends the
> space that we need to the 'task_struct' which we dynamically
> allocate already.  This saves from doing an extra slab
> allocation at fork().
> 
> The only real downside here is that we have to stick everything
> and the end of the task_struct.  But, I think the BUILD_BUG_ON()s
> I stuck in there should keep that from being too fragile.
> 
> Cc: Andy Lutomirski <luto@amacapital.net>
> Cc: Borislav Petkov <bp@alien8.de>
> Cc: H. Peter Anvin <hpa@zytor.com>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: dave@sr71.net
> Link: http://lkml.kernel.org/r/20150716191437.A334FF2E@viggo.jf.intel.com
> Signed-off-by: Ingo Molnar <mingo@kernel.org>

Dave, can I put your Signed-off-by into this patch? I have already tested these 
two patches and they are looking good here - so unless you can see some breakage 
or other problem I'd prefer it to keep it two patches.

Thanks,

	Ingo