Project

General

Profile

Actions

Bug #16032

open

SMF service programs should be 64-bit

Added by Toomas Soome 4 months ago. Updated 4 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
cmd - userland programs
Start date:
Due date:
% Done:

0%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
External Bug:

Description

We have number of service startup programs for SMF, build them 64-bit.


Related issues

Related to illumos gate - Feature #14128: ipf: only build 64-bit commandsClosedToomas Soome

Actions
Actions #1

Updated by Toomas Soome 4 months ago

Actions #2

Updated by Gordon Ross 4 months ago

Of course this will require fixing the libsqlite problem (it's only 32-bit)

Actions #3

Updated by Rich Lowe 4 months ago

There's a fully 64bit SMF in arm64-gate (obviously), but there is a specific bug in SMF's use of the alignment macros and type promotion that needs a more careful fix than is there (if I recall, I just cast things to avoid the promotion).

Hayashi had a much better -- but harder for me to test -- fix using typeof in these macros to avoid unintended promotion.

Actions #4

Updated by Rich Lowe 4 months ago

Gordon Ross wrote in #note-2:

Of course this will require fixing the libsqlite problem (it's only 32-bit)

A fun quirk of this, which did not apply on arm, is sqlite's lack of real typing may mean migration from a 32bit to a 64bit binary may change values if those values are floating point (and possibly otherwise too, I forgot the specifics).

That might be difficult to test.

I'm not sure from the terse description though whether Toomas means to make configd/startd 64bit, or to make some of the sundry executables that run the services themselves 64bit. If it's the latter, sqlite (and the P2ALIGN issue I mentioned) don't apply

Actions #5

Updated by Toomas Soome 4 months ago

Rich Lowe wrote in #note-4:

Gordon Ross wrote in #note-2:

Of course this will require fixing the libsqlite problem (it's only 32-bit)

A fun quirk of this, which did not apply on arm, is sqlite's lack of real typing may mean migration from a 32bit to a 64bit binary may change values if those values are floating point (and possibly otherwise too, I forgot the specifics).

That might be difficult to test.

I'm not sure from the terse description though whether Toomas means to make configd/startd 64bit, or to make some of the sundry executables that run the services themselves 64bit. If it's the latter, sqlite (and the P2ALIGN issue I mentioned) don't apply

I think, it is good umbrella for both cases, as we need to get both done. So for time being, it is good to collect all related information in single point.

Actions

Also available in: Atom PDF