Skip to content

Solaris porting changes v3 - #15576

Closed
l1gi wants to merge 6 commits into
OISF:mainfrom
l1gi:solaris-7005-v3
Closed

Solaris porting changes v3#15576
l1gi wants to merge 6 commits into
OISF:mainfrom
l1gi:solaris-7005-v3

Conversation

@l1gi

@l1gi l1gi commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

This updates the Solaris porting changes on top of current upstream main as a small series of logical
commits instead of a single squashed patch.

The series includes:

  • util/path: handle missing d_type / DT_REG definitions
  • build/solaris: use __sun and Solaris byteorder helpers
  • util/time: avoid tm_gmtoff on Solaris
  • util/syslog: guard LOG_FTP
  • util/cpu: enable SPARC misalignment emulation at startup
  • suricata: guard RLIMIT_NPROC usage

Ticket: 7005

l1gi added 6 commits June 7, 2026 22:39
Guard DT_REG usage so directory walking code still builds on platforms where dirent does not expose d_type or DT_REG, such as Solaris.
Replace uses of the non-standard sun macro with __sun and use Solaris byte swap helpers from <sys/byteorder.h>. This keeps the Solaris-specific code paths reachable when building with standard-conforming toolchains.
Skip tm_gmtoff and tm_zone initialization on Solaris where those struct tm members are not available.
Only expose the ftp facility when the platform syslog headers define LOG_FTP.
Call a SPARC-specific helper during pre-init so the kernel emulates unaligned accesses for this process instead of terminating on alignment faults. This matches the Solaris/SPARC portability fix that motivated the original patch.
Only use RLIMIT_NPROC when the platform exposes that resource limit constant.
@l1gi
l1gi requested a review from victorjulien as a code owner June 7, 2026 20:46
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

NOTE: This PR may contain new authors.

@victorjulien victorjulien added this to the 9.0 milestone Jun 10, 2026
@victorjulien

Copy link
Copy Markdown
Member

Merged in #15601, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants