On Tue, Sep 10, 2013 at 12:12:47PM -0500, Alexander Graf wrote: > > On 10.09.2013, at 05:21, Paul Mackerras wrote: > > > @@ -212,6 +212,7 @@ struct thread_struct { > > #endif > > #endif > > struct thread_fp_state fp_state; > > + struct thread_fp_state *fp_save_area; > > Why do you need these pointers? Couldn't you handle everything you need through preempt notifiers? As you note in your review of a later patch, no, I need the pointer so that if in-kernel code wants to use FP or VSX, potentially in the context of this same process, it knows where to save the FP/VSX state away to. Paul. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html