Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 935875 - sys-cluster/openmpi and friends: rekeyword
Summary: sys-cluster/openmpi and friends: rekeyword
Status: IN_PROGRESS
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Keywording (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo Cluster Team
URL:
Whiteboard:
Keywords: CC-ARCHES
Depends on:
Blocks:
 
Reported: 2024-07-12 06:01 UTC by Eli Schwartz
Modified: 2025-01-17 06:01 UTC (History)
3 users (show)

See Also:
Package list:
sys-cluster/openmpi arm64 loong ppc64 sparc sys-cluster/prrte arm64 loong ppc64 sparc sys-cluster/pmix arm64 loong ppc64 sparc sys-cluster/ucx arm64 loong ppc64 sparc
Runtime testing required: ---
nattka: sanity-check+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Eli Schwartz gentoo-dev 2024-07-12 06:01:14 UTC
Having just bumped openmpi with new dependencies, pmix etc need to be keyworded for compatibility.
Comment 1 NATTkA bot gentoo-dev Security 2024-07-12 06:04:09 UTC Comment hidden (obsolete)
Comment 2 Eli Schwartz gentoo-dev 2024-07-12 06:08:43 UTC
manual list it is, then... interesting that nattka make-package-list didn't solve for this.
Comment 3 Eli Schwartz gentoo-dev 2024-07-12 06:15:37 UTC
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=8773617f69643dfbae7ae8e2fb05ce67617f0642


commit 8773617f69643dfbae7ae8e2fb05ce67617f0642
Author:     Marek Szuba <marecki@gentoo.org>
AuthorDate: Mon Jul 5 13:13:03 2021
Commit:     Marek Szuba <marecki@gentoo.org>
CommitDate: Mon Jul 5 15:38:04 2021

    sys-cluster/ucx: keyword 1.10.0_rc5 for -riscv
    
    "#error: Unsupported architecture". According to the source code (i.e.
    git master on GitHub), ucx currently only supports amd64, arm64 and
    ppc64.
    
    Signed-off-by: Marek Szuba <marecki@gentoo.org>

 sys-cluster/ucx/ucx-1.10.0_rc5.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


If this is true it may mean we need to drop more arches here too. I will leave that to the arch teams to figure out.
Comment 4 Eli Schwartz gentoo-dev 2024-08-23 17:33:57 UTC
new unbundled dep
Comment 5 Jakov Smolić archtester gentoo-dev 2024-08-23 23:37:47 UTC
ppc64 done
Comment 6 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2024-08-24 06:41:48 UTC
arm64 done
Comment 7 NATTkA bot gentoo-dev Security 2024-09-12 18:41:26 UTC Comment hidden (obsolete)
Comment 8 Arthur Zamarin archtester Gentoo Infrastructure gentoo-dev Security 2024-09-12 18:42:36 UTC
ia64 was dropped from gentoo
Comment 9 NATTkA bot gentoo-dev Security 2024-09-12 18:44:28 UTC
All sanity-check issues have been resolved
Comment 10 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-12-06 17:38:02 UTC
ping loong, sparc
Comment 11 WANG Xuerui gentoo-dev 2025-01-16 16:30:36 UTC
Sorry for the delay.

Turns out all versions of sys-cluster/ucx contain non-trivial arch-specific code, like the "bistro" component that JITs indirect jumps to pointers. And it's only ported to amd64, arm64 and ppc64.

Given sys-cluster/pmix unconditionally RDEPENDs on sys-cluster/ucx, and sys-cluster/openmpi unconditionally RDEPENDs on sys-cluster/pmix, I think it's probably best to globally mask openmpi and only unmask on those 3 supported arches. And of course sys-cluster/ucx should be KEYWORDS="-* ~amd64 ~arm64 ~ppc64 ~x86" to reflect the reality...
Comment 12 Eli Schwartz gentoo-dev 2025-01-16 17:13:06 UTC
(In reply to WANG Xuerui from comment #11)
> Sorry for the delay.
> 
> Turns out all versions of sys-cluster/ucx contain non-trivial arch-specific
> code, like the "bistro" component that JITs indirect jumps to pointers. And
> it's only ported to amd64, arm64 and ppc64.


All versions of the upstream software too, I guess.

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=978c8e081d9b625ce7414fa19839afe80cac52f1

commit 978c8e081d9b625ce7414fa19839afe80cac52f1
Author:     Justin Lecher <jlec@gentoo.org>
AuthorDate: Sun Apr 1 16:36:57 2018
Commit:     Justin Lecher <jlec@gentoo.org>
CommitDate: Sun Apr 1 16:51:33 2018

    sys-cluster/ucx: New package
    
    Signed-off-by: Justin Lecher <jlec@gentoo.org>
    Package-Manager: Portage-2.3.27, Repoman-2.3.9

 sys-cluster/ucx/Manifest         |  1 +
 sys-cluster/ucx/metadata.xml     | 12 ++++++++++++
 sys-cluster/ucx/ucx-1.2.2.ebuild | 29 +++++++++++++++++++++++++++++
 3 files changed, 42 insertions(+)
 create mode 100644 sys-cluster/ucx/Manifest
 create mode 100644 sys-cluster/ucx/metadata.xml
 create mode 100644 sys-cluster/ucx/ucx-1.2.2.ebuild



Originally added with KEYWORDS="~amd64 ~x86 ~amd64-linux ~x86-linux"


These upstream "error unsupported architectures" didn't include i686, just x86_64 / ppc64 / aarch64, since at least 

https://github.com/openucx/ucx/commit/a6b55450c81b90661ff45d05bb836241a9545c41#diff-e78e8403c1cc6f845126743fbbc7fd4c2b95c67a309d9acb8726060e35921c07R32

Which is version 1.2.0

I was hoping that this was just keywords being carried forward despite no longer working, but in fact someone simply added a brand new package with keywords that have never been tested a single time in almost a decade. Sad...
Comment 13 WANG Xuerui gentoo-dev 2025-01-17 06:01:18 UTC
There is a PR adding loong support to sys-cluster/ucx upstream, but the work is stalled. I've contacted the PR author and plan to take over the work myself.