Skip to content

Review test_memory.py::test_vmm_allocator_policy_configuration xfail #1300

Open

Description

@rwgk

Review xfail added in test_memory.py with PR #1302

Activity

  1. added
    triageNeeds the team's attention
    cuda.coreEverything related to the cuda.core module
    on Dec 2, 2025
  2. changed the title [-]Review test_memory.py `xfail`[/-] [+]Review test_memory.py::test_vmm_allocator_policy_configuration `xfail`[/+] on Dec 2, 2025
  3. rwgk commented on Dec 2, 2025

    @rwgk
    ContributorAuthor

    @benhg the github UI won't let me formally assign this one to you. It'd be great if you could take a look at the added xfail. Are they OK, or is there an underlying issue we should fix?

  4. benhg commented on Dec 5, 2025

    @benhg
    Member

    Hi @rwgk the xfail is okay with me. I don't think there's an underlying issue but I am also not an expert on VMM for windows.

  5. rwgk commented on Dec 5, 2025

    @rwgk
    ContributorAuthor

    There is also the Jetson failure.

    Currently I don't have a Windows or Jetson system to reproduce those xfails (the systems I had were from a machine pool and got re-imaged in the meantime).

    I think it's good to leave the TODOs introduced with #1302 in the code, but what should we do about this issue? Mark as P2?

    @leofang for advice

  6. benhg commented on Dec 5, 2025

    @benhg
    Member

    I agree with you. I also don't have any understanding of VMM on Jetson systems, so I think it's fair to leave the xfails with TODO and follow up when we can text better. I'll defer to Leo's expertise as well.

  7. added this to the cuda.core backlog milestone on Jan 8, 2026
  8. removed this from the cuda.core backlog milestone on Apr 2, 2026
  9. aryanputta commented on Jun 23, 2026

    @aryanputta
    Contributor

    I took a look at this and found one possible underlying issue behind the Jetson xfail.

    test_vmm_allocator_policy_configuration requests gpu_direct_rdma=True, which sets CUmemAllocationProp.allocFlags.gpuDirectRDMACapable for cuMemCreate. The test and VirtualMemoryResource.__init__ currently gate that path on Device.properties.gpu_direct_rdma_supported, but cuda.core also exposes the more specific Device.properties.gpu_direct_rdma_with_cuda_vmm_supported property, documented as support for specifying the GPUDirect RDMA flag with cuMemCreate.

    Would a small PR that switches the VMM RDMA guard/test skip to gpu_direct_rdma_with_cuda_vmm_supported be useful here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingcuda.coreEverything related to the cuda.core module

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions