Opened 15 years ago

Closed 10 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

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 andreasf 15 years ago.
patch extracted from #2695

Download all attachments as: .zip

Change History (7)

by andreasf, 15 years ago

Attachment: ucontext.diff​ added

patch extracted from #2695

comment:1 by andreasf, 15 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, 15 years ago

Resolution: → fixed
Status: new → closed

Thanks! Applied in hrev35287.

comment:3 by stippi, 15 years ago

Resolution: fixed
Status: closed → reopened

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

comment:4 by bonefish, 15 years ago

Blocked By: 1935 added
Blocking: 2695 removed
Resolution: → duplicate
Status: reopened → closed

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, 10 years ago

Resolution: duplicate
Status: closed → reopened

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

comment:6 by bonefish, 10 years ago

Resolution: → duplicate
Status: reopened → closed

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.