use cuda allocation alignment as default

[rongou]

This is a strict refactoring to make the calling code a bit simpler.

[jrhemstad]

Once #883 is complete, there will be nothing special about CUDA_ALLOCATION_ALIGNMENT vs any other specified alignment value that warrants defaulting it like this.

[rongou]

Hmm, but in #883 we are still calling (e.g. in binning_memory_resource):

allocation_size =
  rmm::detail::align_up(allocation_size, rmm::detail::CUDA_ALLOCATION_ALIGNMENT)

Don't we still need to do this type of alignments?

[jrhemstad]

Hmm, but in #883 we are still calling (e.g. in binning_memory_resource):

allocation_size =
  rmm::detail::align_up(allocation_size, rmm::detail::CUDA_ALLOCATION_ALIGNMENT)

Don't we still need to do this type of alignments?

The new interface allows the user to specify the desired alignment. The default will be max_align_t, but that default is provided at the interface level, not internally in a function like this.

[rongou]

Wouldn't it still make sense to do this simple cleanup while we wait for the new interface?

[github-actions]

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

[harrism]

Wouldn't it still make sense to do this simple cleanup while we wait for the new interface?

Possibly, given that #883 and cuda::memory_resource are dragging...

[harrism]

Moving to 22.04

[rongou]

rerun tests

[rongou]

@harrism should this be merged?

[harrism]

@jrhemstad needs to approve.

[jjacobelli]

rerun tests

[github-actions]

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.