CHANGE handling arguments of devdax_alloc()

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

src/provider/provider_devdax_memory.c

@@ -243,7 +243,9 @@ static umf_result_t devdax_alloc(void *provider, size_t size, size_t alignment,
     }
 
     if (IS_NOT_ALIGNED(size, DEVDAX_PAGE_SIZE_2MB)) {
-        size = ALIGN_UP(size, DEVDAX_PAGE_SIZE_2MB);
+        LOG_ERR("wrong size: %zu (not a multiple of the page size (%zu))", size,
+                DEVDAX_PAGE_SIZE_2MB);
+        return UMF_RESULT_ERROR_INVALID_ARGUMENT;
     }
 
     devdax_memory_provider_t *devdax_provider =

test/provider_devdax_memory.cpp

@@ -199,8 +199,8 @@ TEST_P(umfProviderTest, alloc_2page_align_page_size) {
                             PURGE_NONE);
 }
 
-TEST_P(umfProviderTest, alloc_page64_align_page_div_2) {
-    test_alloc_free_success(provider.get(), page_plus_64, page_size / 2,
+TEST_P(umfProviderTest, alloc_page_align_page_div_2) {
+    test_alloc_free_success(provider.get(), page_size, page_size / 2,
                             PURGE_NONE);
 }
 
@@ -214,6 +214,11 @@ TEST_P(umfProviderTest, purge_force) {
 
 // negative tests using test_alloc_failure
 
+TEST_P(umfProviderTest, alloc_page64_align_page_div_2) {
+    test_alloc_failure(provider.get(), page_plus_64, page_size / 2,
+                       UMF_RESULT_ERROR_INVALID_ARGUMENT, 0);
+}
+
 TEST_P(umfProviderTest, alloc_page64_align_page_minus_1_WRONG_ALIGNMENT_1) {
     test_alloc_failure(provider.get(), page_plus_64, page_size - 1,
                        UMF_RESULT_ERROR_INVALID_ARGUMENT, 0);