Skip to content

Commit

Permalink
Remove real time signals references
Browse files Browse the repository at this point in the history
At this moment uksignal doesn't support real time
signals (signal codes greater than 31). However, Go runtime
assumes that it does and it tries to init signals up to SIGRTMAX,
causing any go application to fail. This commit avoids this situation.
These changes should be removed when real time signals support will
be added to uksignals.

Signed-off-by: Răzvan Vîrtan <virtanrazvan@gmail.com>
  • Loading branch information
razvanvirtan committed Oct 4, 2021
1 parent a4d3e96 commit 17683fc
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 34 deletions.
3 changes: 2 additions & 1 deletion generated/runtime_sysinfo.go
Original file line number Diff line number Diff line change
Expand Up @@ -2408,7 +2408,8 @@ const __POSIX_SYNCHRONIZED_IO = 200809
const _SYS_close = ___NR_close
const _PR_SET_UNALIGN = 6
const _SYS_brk = ___NR_brk
const __NSIG = (___SIGRTMAX + 1)
// TO DO : const __NSIG=(___SIGRTMAX + 1) when RT signals are added to uksignal
const __NSIG = 32
const __POSIX_FD_SETSIZE = __POSIX_OPEN_MAX
const _ENOTTY = 25
const _RTA_PRIORITY = 6
Expand Down
67 changes: 34 additions & 33 deletions generated/sigtab.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,37 +35,38 @@ var sigtable = [...]sigTabT{
_SIGSTKFLT: {_SigThrow + _SigUnblock, "SIGSTKFLT: stack fault"},
_SIGPWR: {_SigNotify, "SIGPWR: power failure restart"},
_SIGPOLL: {_SigNotify, "SIGPOLL: pollable event occurred"},
32: {_SigSetStack + _SigUnblock, "signal 32"}, /* SIGCANCEL; see issue 6997 */
33: {_SigSetStack + _SigUnblock, "signal 33"}, /* SIGSETXID; see issues 3871, 9400, 12498 */
34: {_SigNotify, "signal 34"},
35: {_SigNotify, "signal 35"},
36: {_SigNotify, "signal 36"},
37: {_SigNotify, "signal 37"},
38: {_SigNotify, "signal 38"},
39: {_SigNotify, "signal 39"},
40: {_SigNotify, "signal 40"},
41: {_SigNotify, "signal 41"},
42: {_SigNotify, "signal 42"},
43: {_SigNotify, "signal 43"},
44: {_SigNotify, "signal 44"},
45: {_SigNotify, "signal 45"},
46: {_SigNotify, "signal 46"},
47: {_SigNotify, "signal 47"},
48: {_SigNotify, "signal 48"},
49: {_SigNotify, "signal 49"},
50: {_SigNotify, "signal 50"},
51: {_SigNotify, "signal 51"},
52: {_SigNotify, "signal 52"},
53: {_SigNotify, "signal 53"},
54: {_SigNotify, "signal 54"},
55: {_SigNotify, "signal 55"},
56: {_SigNotify, "signal 56"},
57: {_SigNotify, "signal 57"},
58: {_SigNotify, "signal 58"},
59: {_SigNotify, "signal 59"},
60: {_SigNotify, "signal 60"},
61: {_SigNotify, "signal 61"},
62: {_SigNotify, "signal 62"},
63: {_SigNotify, "signal 63"},
64: {_SigNotify, "signal 64"},
// TODO: uncomment when RT signals support is added to uksignal
// 32: {_SigSetStack + _SigUnblock, "signal 32"}, /* SIGCANCEL; see issue 6997 */
// 33: {_SigSetStack + _SigUnblock, "signal 33"}, /* SIGSETXID; see issues 3871, 9400, 12498 */
// 34: {_SigNotify, "signal 34"},
// 35: {_SigNotify, "signal 35"},
// 36: {_SigNotify, "signal 36"},
// 37: {_SigNotify, "signal 37"},
// 38: {_SigNotify, "signal 38"},
// 39: {_SigNotify, "signal 39"},
// 40: {_SigNotify, "signal 40"},
// 41: {_SigNotify, "signal 41"},
// 42: {_SigNotify, "signal 42"},
// 43: {_SigNotify, "signal 43"},
// 44: {_SigNotify, "signal 44"},
// 45: {_SigNotify, "signal 45"},
// 46: {_SigNotify, "signal 46"},
// 47: {_SigNotify, "signal 47"},
// 48: {_SigNotify, "signal 48"},
// 49: {_SigNotify, "signal 49"},
// 50: {_SigNotify, "signal 50"},
// 51: {_SigNotify, "signal 51"},
// 52: {_SigNotify, "signal 52"},
// 53: {_SigNotify, "signal 53"},
// 54: {_SigNotify, "signal 54"},
// 55: {_SigNotify, "signal 55"},
// 56: {_SigNotify, "signal 56"},
// 57: {_SigNotify, "signal 57"},
// 58: {_SigNotify, "signal 58"},
// 59: {_SigNotify, "signal 59"},
// 60: {_SigNotify, "signal 60"},
// 61: {_SigNotify, "signal 61"},
// 62: {_SigNotify, "signal 62"},
// 63: {_SigNotify, "signal 63"},
// 64: {_SigNotify, "signal 64"},
}

0 comments on commit 17683fc

Please sign in to comment.