Skip to content
Snippets Groups Projects

Asynchronous memory allocation for GPUs

Merged Asynchronous memory allocation for GPUs
dpugmire/vtk-m:gpu_async_alloc into master
Merged Dave Pugmire requested to merge dpugmire/vtk-m:gpu_async_alloc into master

Syncronous memory allocation can have a lot of overhead on GPUs. This was noticed when doing tests for multi-block filters. This is supported in cuda versions > 11.3

This is a redo of !2759 (closed). Much has changed since that MR so this will be the replacement.

Edited by Dave Pugmire

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: e1bffa47

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    Errors:

    • commit 3380682d is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/filter/FilterParticleAdvection.hxx, vtkm/filter/FilterTemporalParticleAdvection.h, vtkm/filter/FilterTemporalParticleAdvection.hxx, vtkm/filter/ParticleAdvection.h, vtkm/filter/ParticleAdvection.hxx, and 4 others. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit e1bffa47 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/testing/UnitTestArrayHandle.cxx. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.

    Warnings:

    • the merge request is marked as a work-in-progress.

    The warnings do not need to be fixed, but it is recommended to do so.

    Please rewrite commits to fix the errors listed above (adding fixup commits will not resolve the errors) and force-push the branch again to update the merge request.

  • Dave Pugmire changed the description

    changed the description

  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: e1bffa47

  • Dave Pugmire added 1 commit

    added 1 commit

    • 9ef465b5 - Undo changes to arrayhandle unit test

    Compare with previous version

  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: 9ef465b5

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    Errors:

    • commit 3380682d is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/filter/FilterParticleAdvection.hxx, vtkm/filter/FilterTemporalParticleAdvection.h, vtkm/filter/FilterTemporalParticleAdvection.hxx, vtkm/filter/ParticleAdvection.h, vtkm/filter/ParticleAdvection.hxx, and 4 others. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit e1bffa47 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/testing/UnitTestArrayHandle.cxx. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.

    Warnings:

    • the merge request is marked as a work-in-progress.

    The warnings do not need to be fixed, but it is recommended to do so.

    Please rewrite commits to fix the errors listed above (adding fixup commits will not resolve the errors) and force-push the branch again to update the merge request.

  • Dave Pugmire added 1 commit

    added 1 commit

    • 24e63368 - Add Sync/Async memory allocator functions.

    Compare with previous version

  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: 24e63368

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    Errors:

    • commit 3380682d is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/filter/FilterParticleAdvection.hxx, vtkm/filter/FilterTemporalParticleAdvection.h, vtkm/filter/FilterTemporalParticleAdvection.hxx, vtkm/filter/ParticleAdvection.h, vtkm/filter/ParticleAdvection.hxx, and 4 others. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit e1bffa47 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/testing/UnitTestArrayHandle.cxx. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit 24e63368 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/cuda/internal/CudaAllocator.cu. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.

    Warnings:

    • the merge request is marked as a work-in-progress.

    The warnings do not need to be fixed, but it is recommended to do so.

    Please rewrite commits to fix the errors listed above (adding fixup commits will not resolve the errors) and force-push the branch again to update the merge request.

  • Dave Pugmire added 6 commits

    added 6 commits

    Compare with previous version

  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: 1c15ab8a

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    Errors:

    • commit 3380682d is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/filter/FilterParticleAdvection.hxx, vtkm/filter/FilterTemporalParticleAdvection.h, vtkm/filter/FilterTemporalParticleAdvection.hxx, vtkm/filter/ParticleAdvection.h, vtkm/filter/ParticleAdvection.hxx, and 4 others. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit e1bffa47 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/testing/UnitTestArrayHandle.cxx. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.
    • commit 24e63368 is not allowed because the following files are not formatted according to the 'clang-format' check: vtkm/cont/cuda/internal/CudaAllocator.cu. Post a comment ending in the line Do: reformat to rewrite the MR source branch automatically.

    Warnings:

    • the merge request is marked as a work-in-progress.

    The warnings do not need to be fixed, but it is recommended to do so.

    Please rewrite commits to fix the errors listed above (adding fixup commits will not resolve the errors) and force-push the branch again to update the merge request.

  • Dave Pugmire
    Dave Pugmire @dpugmire ·
    Author Developer

    Do: reformat

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    This topic has been reformatted and pushed; please fetch from the source repository and reset your local branch to continue with further development on the reformatted commits.

  • Kitware Robot added 8 commits

    added 8 commits

    Compare with previous version

    Toggle commit list
  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: d06c321a

  • Kitware Robot
    Kitware Robot @kwrobot ·
    Owner

    Warnings:

    • the merge request is marked as a work-in-progress.

    The warnings do not need to be fixed, but it is recommended to do so.

  • Dave Pugmire added 1 commit

    added 1 commit

    • 6697a615 - Run multiblock tests with and without Async GPU memory

    Compare with previous version

  • buildbot
    buildbot @buildbot ·
    Maintainer

    This merge request has been queued for testing. Test results may be viewed on:

    • Gitlab CI builders results can be viewed via the Pipelines for this merge request.
    • CDash links are also available in the Pipelines page
    • CDash (master)
    • Buildbot (master) (only visible from inside Kitware)

    Branch-at-master: 6697a615

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply