Bug 245774
| Summary: | graphics/mesa-dri: fix build on powerpc head | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Piotr Kubaj <pkubaj> | ||||
| Component: | Individual Port(s) | Assignee: | freebsd-x11 (Nobody) <x11> | ||||
| Status: | In Progress --- | ||||||
| Severity: | Affects Some People | CC: | bdragon, danfe, powerpc, x11, zeising | ||||
| Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(x11) |
||||
| Version: | Latest | ||||||
| Hardware: | powerpc | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
(In reply to Piotr Kubaj from comment #0) > powerpc head doesn't have 64 bit atomic functions Actually we have native 64-bit kernel atomics on ppc32 since src r356308, but at that time it was decided to keep the "status quo of userland doing its own locking". :-/ Should the patch be applied only in the powerpc32 case, or is it a noop on other architectures (given that they are compiled with clang)? (In reply to Niclas Zeising from comment #2) Sorry for the late answer. Yes, this is noop. There is a check for defined(MISSING_64BIT_ATOMICS). AFAIK other architectures have 64 bit atomics. If mesa has a use case for crossprocess 64 bit atomics, I am still carrying the userland parts of the atomic emulation around in my superglue tree and can resurrect it at any time. Is this still relevant? |
Created attachment 213615 [details] patch powerpc head doesn't have 64 bit atomic functions. Use this patch to fix build there.