From: Tomer Tayar <tomer.tayar@intel.com>
Date: Sun, 26 May 2024 13:32:32 +0000 (+0300)
Subject: accel/habanalabs: return ENOMEM if less than requested pages were pinned
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=9f5067531c9b79318c4e48a933cb2694f53f3de2;p=users%2Fjedix%2Flinux-maple.git

accel/habanalabs: return ENOMEM if less than requested pages were pinned

EFAULT is currently returned if less than requested user pages are
pinned. This value means a "bad address" which might be confusing to
the user, as the address of the given user memory is not necessarily
"bad".

Modify the return value to ENOMEM, as "out of memory" is more suitable
in this case.

Signed-off-by: Tomer Tayar <tomer.tayar@intel.com>
Reviewed-by: Koby Elbaz <koby.elbaz@intel.com>
Signed-off-by: Koby Elbaz <koby.elbaz@intel.com>
---

diff --git a/drivers/accel/habanalabs/common/memory.c b/drivers/accel/habanalabs/common/memory.c
index 61472a381904..48d2d598a387 100644
--- a/drivers/accel/habanalabs/common/memory.c
+++ b/drivers/accel/habanalabs/common/memory.c
@@ -2332,7 +2332,7 @@ static int get_user_memory(struct hl_device *hdev, u64 addr, u64 size,
 		if (rc < 0)
 			goto destroy_pages;
 		npages = rc;
-		rc = -EFAULT;
+		rc = -ENOMEM;
 		goto put_pages;
 	}
 	userptr->npages = npages;