[Bug 262513] math/oink: update and make atomics portable

From: <bugzilla-noreply_at_freebsd.org>
Date: Sat, 12 Mar 2022 23:45:51 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=262513

            Bug ID: 262513
           Summary: math/oink: update and make atomics portable
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Only Me
          Priority: ---
         Component: Individual Port(s)
          Assignee: yuri@freebsd.org
          Reporter: fuz@fuz.su
                CC: yuri@freebsd.org
             Flags: maintainer-feedback?(yuri@freebsd.org)
          Assignee: yuri@freebsd.org
 Attachment #232413 maintainer-approval?(yuri@freebsd.org)
             Flags:
                CC: yuri@freebsd.org

Created attachment 232413
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=232413&action=edit
math/oink: update and make atomics portable

The attached patch replaces the mfence() macro with something that should work
on any architecture.  Unfortunately upstream doesn't seem to understand atomics
and uses volatile variables instead, so it's still somewhat of a fingers
crossed situation.

I've also took the liberty to advance the port to the current commit which
integrates the two patches you have.

Tested with Poudriere on armv7 arm64 FreeBSD 13.  The testsuite passes on armv7
except for the following 3 cases:

          6 - TestSolverPSIp (Bus error)
         27 - TestSolverZLKpar (Bus error)
         29 - TestSolverUZLK (Bus error)

Looks like an unaligned memory access maybe.  On arm64 there are no errors.

-- 
You are receiving this mail because:
You are the assignee for the bug.