I've put together a new patch. On Thursday 11 October 2007 19:04:28 Daniel Jacobowitz wrote: > Trivial comments: you've added #includes so Makefile.in needs an > update (maybe I'll do automatic dependencies sometime soon); please > include a ChangeLog with all patches; Both done. On Thursday 11 October 2007 20:13:14 Mark Kettenis wrote: > > From: Nick Hudson > > I think I need to add the instructions for when the > > setcontext/sigreturn fails as well. Is this correct? > > Probably a good idea, since it makes the false positives even less > likely. Done. > By not calling i386bsd_init_abi(), tdep->jb_pc_offset is no longer > set. You could initialize tdep->jb_pc_offset in i386nbsd_init_abi(), > but I'd prefer leaving in the i386bsd_init_abi() call, because it > shows the inheritance structure a bit more clearly. OK, no problems - I've put it back. > It'd also be nice to have some documentation about what versions of > NetBSD use which sigtramp code (best to stick to official releases > here). Perhaps that information could be used to give somewhat more > meaningfull names to the tramp_frame varaibles. Each variable is numbered with the NetBSD dot release that the code was first seen. You'll have to forgive the ChangeLog as it's completely alien to me. Thanks, Nick