Project

General

Profile

Bug #13487

want mapfile-based assertions about symbol properties

Added by Rich Lowe 2 months ago. Updated about 13 hours ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
tools - gate/build tools
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

In our constant quest to avoid breaking the ABI, we should assert that any global data is the size we expect both as a check, and a warning to anyone changing it (perhaps inadvertently) that they must consider the repercussions.

We can do this by implementing a version of the Solaris ld(1) mapfile assertions.

#1

Updated by Electric Monk 2 months ago

  • Gerrit CR set to 1214
#2

Updated by Rich Lowe 19 days ago

testing:

Builds of illumos, additions to the test suite.  Merges with illumos showing up additional/changed global data I've had to adapt to, showing that both the guidance and assertions fire.
#3

Updated by Electric Monk 3 days ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 8222814ef8560ee0ba222eca8ca5acffc6cd0e44

commit  8222814ef8560ee0ba222eca8ca5acffc6cd0e44
Author: Richard Lowe <richlowe@richlowe.net>
Date:   2021-04-09T20:48:05.000Z

    13487 want mapfile-based assertions about symbol properties
    Reviewed by: Robert Mustacchi <rm@fingolfin.org>
    Reviewed by: Jason King <jason.brian.king@gmail.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

#4

Updated by Toomas Soome about 13 hours ago

SPARC build and testing: building on sparc did reveal issue in libld and additional mapfile entries were needed (as expected). build/install/boot on my build system. Build is also reported by Klaus Ziegler.

Also available in: Atom PDF