Opened 8 years ago

Closed 3 years ago

#5324 closed enhancement (duplicate)

[PATCH] Define mcontext_t

Reported by: Andreas Färber Owned by: Nobody
Priority: normal Milestone: R1
Component: System/POSIX Version: R1/Development
Keywords: Cc: Alexander von Gluck
Blocked By: #1935 Blocking:
Has a Patch: no Platform: All

Description

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

http://www.opengroup.org/onlinepubs/009695399/basedefs/ucontext.h.html

Attachments (1)

ucontext.diff (561 bytes) - added by Andreas Färber 8 years ago.
patch extracted from #2695

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by Andreas Färber

Attachment: ucontext.diff added

patch extracted from #2695

comment:1 Changed 8 years ago by Andreas Färber

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 Changed 8 years ago by Stephan Aßmus

Resolution: fixed
Status: newclosed

Thanks! Applied in hrev35287.

comment:3 Changed 8 years ago by Stephan Aßmus

Resolution: fixed
Status: closedreopened

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

comment:4 Changed 8 years ago by Ingo Weinhold

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 Changed 3 years ago by Jessica

Resolution: duplicate
Status: closedreopened

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

comment:6 Changed 3 years ago by Ingo Weinhold

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.