unified-memory-framework icon indicating copy to clipboard operation
unified-memory-framework copied to clipboard
verified publisher icon oneapi-src

Add devdax memory provider

Open ldorau opened this issue 1 year ago • 3 comments

Description

Add devdax memory provider.

Checklist

  • [x] Code compiles without errors locally
  • [x] All tests pass locally
  • [x] CI workflows execute properly

ldorau avatar Aug 09 '24 16:08 ldorau

What is a design and use case of the dax provider? Only to help "mmap" it? As free is not supported?

lplewa avatar Aug 12 '24 14:08 lplewa

What is a design and use case of the dax provider? Only to help "mmap" it? As free is not supported?

To be able to provide memory from a /dev/dax (not extendable file).

ldorau avatar Aug 13 '24 07:08 ldorau

@lplewa please resolve your issues.

ldorau avatar Sep 02 '24 09:09 ldorau

@bratpiorka please review

ldorau avatar Sep 02 '24 09:09 ldorau

it looks like one of the devdax test has failed

Fixed

ldorau avatar Sep 05 '24 11:09 ldorau

If we are not supporting free, why we just do not mmap entire devdax at init? We can remove half of the code from this provider? mmap before first access do not cost us, so it should not be a problem.

Because it is a memory provider that is supposed to handle many calls to alloc(size, alignment) ...

ldorau avatar Sep 10 '24 06:09 ldorau