Page MenuHomeFreeBSD

libprocstat: improve conditional for 32-bit compat
ClosedPublic

Authored by brooks on Oct 13 2023, 6:46 PM.
Tags
None
Referenced Files
F172276339: D42201.diff
Thu, Sep 17, 10:22 AM
F172260270: D42201.id.diff
Thu, Sep 17, 7:56 AM
Unknown Object (File)
Wed, Sep 9, 7:35 AM
Unknown Object (File)
Sat, Aug 29, 6:45 PM
Unknown Object (File)
Aug 12 2026, 8:35 PM
Unknown Object (File)
Aug 12 2026, 8:35 PM
Unknown Object (File)
Aug 8 2026, 12:24 PM
Unknown Object (File)
Aug 7 2026, 10:17 PM
Subscribers

Details

Summary

Include support for translating 32-bit auxv vectors on non-64-bit
platforms that aren't riscv (which has no 32-bit ABI support).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I don't see a problem, but why bother hard-coding the platforms that lack 32-bit compat support?

I don't see a problem, but why bother hard-coding the platforms that lack 32-bit compat support?

In this case it's not a lack of 32-bit compat support so much as there will never be 32-bit support. Why compile code gated by an always false, dynamic check? I guess we might end up with an x32 interface if someone really needed the smaller pointer size, but I suspect that would have a different sv_name and this ifdef would be the least of a developer implementing x32's concerns.

This revision is now accepted and ready to land.Oct 13 2023, 7:22 PM