On Thu, Oct 11, 2018 at 10:50 AM Christophe de Dinechin <christophe.de.dinechin@xxxxxxxxx> wrote: > > > Sebastian Andrzej Siewior writes: > > > From: Rik van Riel <riel@xxxxxxxxxxx> > > > > copy_fpstate_to_sigframe() has two callers and both invoke the function only if > > fpu->initialized is set. > > One of the callers is in a different file. > Not sure if that warrants some kind of check that > fpu->initialized is set? I still kind of think that fpu->initialized should be removed entirely as part of this series. Keeping unnecessary complication around in the FPU code increaseds the scope for interesting bugs and makes understanding the code considerably harder.
