Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature archdetect rpi #223

Closed
wants to merge 999 commits into from
Closed

Feature archdetect rpi #223

wants to merge 999 commits into from

Conversation

hmeiland
Copy link
Contributor

add raspberry pi support by adding feature to use CPU part for aarch64

@hmeiland hmeiland mentioned this pull request Jan 18, 2023
Closed
@hmeiland
Copy link
Contributor Author

hmeiland commented Jan 19, 2023 •
edited
Loading

checks failed due to HTTP rate limit, not code error...

ocaisa and others added 28 commits March 1, 2024 07:27
@ocaisa
Merge pull request EESSI#481 from TopRichard/eessi-2023.06-dask/9.2-f…
34f3be0 
…oss/2023a

{2023.06}[foss/2023a] dask v2023.9.2
@boegel
Merge pull request EESSI#486 from ocaisa/tweak_check_missing
5e7c6bb 
Make check_missing_installations.sh check against develop branch of EasyBuild
Adding OSU microbenchmarks with GPU support for 2023a, and one withou…
78ba8d8 
…t GPU support for 2023b
@bedroge
launch with --fakeroot for build jobs triggered by bot
e666e2c
@bedroge
remove existing dirs for rebuilds
34bc1c2
@bedroge
rebuild all OpenMPI 4.1.x versions
46e7532
@bedroge
add some comments/links
21959d6
@bedroge
fix syntax, add missing parentheses
7054a58
'which' is not working in the container, command -v is
1fd1609
@bedroge
remove spaces from blank lines
7b78861
@bedroge
rename file to be able to find EB version
87e5ff0
@bedroge
clarify comment about fakeroot
d07879f
@bedroge
add --allow-use-as-root-and-accept-consequences
135c3c8
@bedroge
add missing $
f1831b4
@bedroge
fix app_dir variable
68212a4
@bedroge
split loop, first remove everything that needs to be removed, then dr…
6bcd0e3 
…op back to regular user and do installations
@bedroge
add missing else and fi
c81c0de
@bedroge
su without hypen
f994583
@bedroge
rerun the script with a regular user after removing installations
21e3735
@bedroge
fix typo and add missing quote
4f436b3
@bedroge
fix syntax
0676fda
@bedroge
fix typo in app_dir variable
95d6aaa
Workaround for 'PSM3 can't open nic unit' error
3141789
@bedroge
run removal step without Prefix
e52afc1
@bedroge
move removal part to its own script, at least for now
be4c592
@ocaisa
Merge pull request EESSI#487 from casparvl/osu_gpu
a28e5ee 
{2023.06}[gompi/2023a,gompi/2023b] OSU Microbenchmarks v 7.1.1, OSU Microbenchmarks v 7.2 w/ CUDA 12.1.1
Use LMOD_CONFIG_DIR instead of LMOD_RC to allow user overrides of wha…
efbeb56 
…t we specify in our RC file, see search order on https://lmod.readthedocs.io/en/latest/145_properties.html?highlight=lmod_rc#the-properties-file-lmodrc-lua . Also, prefix our hooks with eessi_ to avoid unintentional clashes in namespace between site lmod configuration and eessi lmod configuration
Combine functions into a single function, before registering this as …
590cabd 
…a hook. Lmod only permits the registration of a single function, trying to register a second function will just overwrite the first, as can be seen from https://github.com/TACC/Lmod/blob/a97d68193a6f32ebca4b4bd70dac0f6ac8fddefe/src/Hook.lua#L87
bedroge and others added 20 commits April 4, 2024 17:04
@bedroge
Merge branch '2023.06-software.eessi.io' into 2023.06-sw-R-bundle-CRA…
a2669eb 
…N-2023.12-foss-2023a
@bedroge
change Highway hook in pre_prepare hook, and reset optarch in a post-…
6fa335b 
…prepare hook
@bedroge @boegel
Update eb_hooks.py
42e580b 
Co-authored-by: Kenneth Hoste <kenneth.hoste@ugent.be>
@bedroge @boegel
Update eb_hooks.py
3c4572c 
Co-authored-by: Kenneth Hoste <kenneth.hoste@ugent.be>
@ocaisa
Merge branch '2023.06-software.eessi.io' into elpa_libxc
13093d1
@boegel
Merge pull request EESSI#532 from bedroge/fix_highway_optarch_hook
ababa60 
Fix Highway hook: change optarch in `pre_prepare_hook` and reset it in `post_prepare_hook`
@bedroge
add from-pr 20238 for R-bundle-CRAN
7283d08
@bedroge
Merge branch '2023.06-software.eessi.io' into modflow644
dc69ba2
@bedroge
Merge pull request EESSI#508 from ocaisa/elpa_libxc
cd488b5
@bedroge
Merge branch '2023.06-software.eessi.io' into 2023.06-sw-R-bundle-CRA…
88448a5 
…N-2023.12-foss-2023a
@bedroge
Merge branch '2023.06-software.eessi.io' into modflow644
23b400a
@bedroge
Merge pull request EESSI#522 from casparvl/modflow644
3371bc2 
{2023.06}[2023a] Modflow 6.4.4 and deps
@bedroge
Merge branch '2023.06-software.eessi.io' into 2023.06-sw-R-bundle-CRA…
7b4d8a5 
…N-2023.12-foss-2023a
@bedroge
add comment, retrigger CI
f6012f7
@boegel
Merge pull request EESSI#428 from trz42/2023.06-sw-R-bundle-CRAN-2023…
8bc14b9 
….12-foss-2023a

{2023.06}[foss/2023a] R-bundle-CRAN 2023.12
@bedroge
add EB 4.9.1
1998a7c
@ocaisa
Merge pull request EESSI#537 from bedroge/easybuild_4.9.1
e54ae92 
{2023.06}[system] EasyBuild v4.9.1
@ocaisa
Merge branch 'main' into feature-archdetect-rpi
1c3dc26
@ocaisa
Update eessi_arch_arm_part.spec
68632f3
@ocaisa
Update debian-rpi4.output
31ad134
@ocaisa
Copy link
Member

ocaisa commented Apr 12, 2024 •
edited
Loading

@hmeiland Can you sync this with upstream, you still have some older CI that's now failing

@ocaisa
Copy link
Member

ocaisa commented Apr 12, 2024

@hmeiland You are targeting the wrong branch with this, it should be 2023.06-software.eessi.io

@ocaisa
Copy link
Member

ocaisa commented Apr 12, 2024 •
edited
Loading

@hmeiland Unfortunately, this is not working as intended, archdetect is no longer recognising neoverse_n1 as an alternative to neoverse_v1 before choosing generic

See https://github.com/EESSI/software-layer/actions/runs/8660373452/job/23748193970?pr=223 , correct output should be
as per https://github.com/EESSI/software-layer/blob/2023.06-software.eessi.io/tests/archdetect/aarch64/neoverse_v1/AWS-awslinux-graviton3.all.output

@ocaisa
Copy link
Member

ocaisa commented Apr 12, 2024 •
edited
Loading

Are we not just better off adding additional required features to eessi_arch_arm.spec to ensure that the Pi 4 doesn't match the condition?

https://marcin.juszkiewicz.com.pl/2022/11/08/from-a-diary-of-aarch64-porter-arm-cpu-features-table/ could help here

@ocaisa
Copy link
Member

ocaisa commented Apr 14, 2024

Alternative to this is merged in #540 where I've ensured that an instruction from the 8.2 instruction set is used to decide on the arch (rather than asimd which already appears in the 8.0 instruction set, and is why the Pi 4 was making a false match). If we wanted an explicit build for the 8.0 instruction set, we'd have to do a bit of work with archdetect (reintroduce asimd as another distinction)

@ocaisa ocaisa closed this Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@hmeiland @ocaisa @boegel @bedroge @casparvl @trz42 @Neves-P @truib