Thayne Harbaugh wrote:
[snip]
> >  static inline target_ulong get_sp_from_cpustate(CPUPPCState *state)
> > {
> >   return state->gpr[1];
> > }
> > 
> > This is no way related to CPU emulation then has nothing to do in cpu.h.
> > Furthermore, there no notion of sigaltstack or even stack pointer in the
> > PowerPC specification.
> > Revert this patch immediatly, please, and stop breaking others code...
> 
> My apologies.  I put get_sp_from_cpustate() in cpu.h because it is a
> generic function that isn't exclusive to sigaltstack().  If it's
> preferred it can be exclusive to sigaltstack().

I moved it over to target_signal.h, which seems to be the best place
for the time being. I also enabled do_sigaltstack processing for
MIPS, PPC and Alpha, as I don't see a reason not to support it.


Thiemo


Reply via email to