Opened 12 years ago

Closed 7 years ago

#5324 closed enhancement (duplicate)

[PATCH] Define mcontext_t

Reported by: andreasf Owned by: nobody
Priority: normal Milestone: R1
Component: System/POSIX Version: R1/Development
Keywords: Cc: kallisti5
Blocked By: #1935 Blocking:
Platform: All


Haiku is missing a ucontext.h header file with its ucontext_t and mcontext_t types.

Attachments (1)

ucontext.diff (561 bytes ) - added by andreasf 12 years ago.
patch extracted from #2695

Download all attachments as: .zip

Change History (7)

by andreasf, 12 years ago

Attachment: ucontext.diff added

patch extracted from #2695

comment:1 by andreasf, 12 years ago

As described in my 2008 mailing list message, mcontext_t can be typedef'ed to our existing, BeOS-compatible struct vregs.

That way the header remains platform-independent, while avoiding ugly conditionals in multi-platform code already using mcontext_t.

comment:2 by stippi, 12 years ago

Resolution: fixed
Status: newclosed

Thanks! Applied in hrev35287.

comment:3 by stippi, 12 years ago

Resolution: fixed
Status: closedreopened

I've removed the header again in hrev35291 as recommended by Ingo.

comment:4 by bonefish, 12 years ago

Blocked By: 1935 added
Blocking: 2695 removed
Resolution: duplicate
Status: reopenedclosed

ucontext_t and mcontext_t should rather be added as part of implementing SA_SIGINFO (the only feature using these structures), which in turn should be implemented as part of real-time signal support. Cf. #1935.

comment:5 by jessicah, 7 years ago

Resolution: duplicate
Status: closedreopened

This needs revisiting, as we have real-time signal support as of hrev42116.

comment:6 by bonefish, 7 years ago

Resolution: duplicate
Status: reopenedclosed

The types were added to signal.h as mandated by the current standard (ucontext.h was removed in issue 7).

Note: See TracTickets for help on using tickets.