Opened 10 years ago

Closed 4 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:
Has a Patch: no 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 10 years ago.
patch extracted from #2695

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by andreasf

Attachment: ucontext.diff added

patch extracted from #2695

comment:1 Changed 10 years ago by andreasf

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 10 years ago by stippi

Resolution: fixed
Status: newclosed

Thanks! Applied in hrev35287.

comment:3 Changed 10 years ago by stippi

Resolution: fixed
Status: closedreopened

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

comment:4 Changed 10 years ago by bonefish

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 4 years ago by jessicah

Resolution: duplicate
Status: closedreopened

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

comment:6 Changed 4 years ago by bonefish

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.