Review test_memory.py::test_vmm_allocator_policy_configuration `xfail`

Open rwgk opened this issue 3 months ago • 4 comments

Review xfail added in test_memory.py with PR #1302

Dec 02 '25 21:12 rwgk

@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?

Dec 02 '25 22:12 rwgk

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.

Dec 05 '25 16:12 benhg

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

Dec 05 '25 18:12 rwgk

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.

Dec 05 '25 23:12 benhg