Skip to content

Conversation

@PatKamin
Copy link
Contributor

@PatKamin PatKamin commented Mar 13, 2025

Fallback to fetching hwloc from source instead of failing UMF build.
Add a workflow for testing the fallback.
Unify correct paths to hwloc.
Use target name instead of a library path for hwloc linkage.
To test examples, the UMF_BUILD_EXAMPLES flag has to be set.

Nightly run: https://github.com/PatKamin/unified-memory-framework/actions/runs/13916556000

Description

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly
  • CI workflows, not executed per PR (e.g. Nightly), execute properly
  • New tests added, especially if they will fail without my changes
  • Added/extended example(s) to cover this functionality
  • Extended the README/documentation
  • All newly added source files have a license
  • All newly added source files are referenced in CMake files
  • Logger (with debug/info/... messages) is used
  • All API changes are reflected in docs and def/map files, and are tested

@PatKamin PatKamin requested a review from a team as a code owner March 13, 2025 07:08
@PatKamin PatKamin requested a review from KFilipek March 14, 2025 12:25
@PatKamin PatKamin force-pushed the hwloc-fallback branch 3 times, most recently from e48dfe5 to 42daf39 Compare March 17, 2025 08:14
Copy link
Contributor

@ldorau ldorau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lukaszstolarczuk please verify and resolve all your issues

@bratpiorka bratpiorka marked this pull request as draft March 17, 2025 10:19
@PatKamin PatKamin force-pushed the hwloc-fallback branch 3 times, most recently from 16ab7a1 to 55c2c9f Compare March 18, 2025 06:23
Fallback to fetching hwloc from source instead of failing UMF build.
Add a workflow for testing the fallback.
Unify correct paths to hwloc.
Use target name instead of a library path for hwloc linkage.
@PatKamin PatKamin marked this pull request as ready for review March 18, 2025 07:34
@ldorau
Copy link
Contributor

ldorau commented Mar 18, 2025

@lukaszstolarczuk are you going to approve it? ;-)

Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lukaszstolarczuk are you going to approve it? ;-)

Needed to discuss something with @PatKamin yet.

@lukaszstolarczuk lukaszstolarczuk merged commit d6d2da6 into oneapi-src:main Mar 18, 2025
85 checks passed
@PatKamin PatKamin deleted the hwloc-fallback branch June 27, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants