diff options
| author | Ira Weiny <ira.weiny@intel.com> | 2021-05-25 10:24:28 -0700 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2021-07-07 22:10:04 -0700 |
| commit | b05d4c576b697b9f462b9c532c997171d5c3b067 (patch) | |
| tree | 78caf3c0f16a323d5aca440d72d20e1917be6514 /drivers/dax | |
| parent | 44788591c3cfb81d9315b8ee5c2076e51bfe8a39 (diff) | |
| download | lwn-b05d4c576b697b9f462b9c532c997171d5c3b067.tar.gz lwn-b05d4c576b697b9f462b9c532c997171d5c3b067.zip | |
dax: Ensure errno is returned from dax_direct_access
If the caller specifies a negative nr_pages that is an invalid
parameter.
Return -EINVAL to ensure callers get an errno if they want to check it.
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Ira Weiny <ira.weiny@intel.com>
Link: https://lore.kernel.org/r/20210525172428.3634316-4-ira.weiny@intel.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/dax')
| -rw-r--r-- | drivers/dax/super.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dax/super.c b/drivers/dax/super.c index 5fa6ae9dbc8b..44736cbd446e 100644 --- a/drivers/dax/super.c +++ b/drivers/dax/super.c @@ -313,7 +313,7 @@ long dax_direct_access(struct dax_device *dax_dev, pgoff_t pgoff, long nr_pages, return -ENXIO; if (nr_pages < 0) - return nr_pages; + return -EINVAL; avail = dax_dev->ops->direct_access(dax_dev, pgoff, nr_pages, kaddr, pfn); |