diff options
| author |   Zhen Lei <thunder.leizhen@huawei.com> | 2020-11-20 17:20:57 +0800 |
|---|---|---|
| committer |   Dan Williams <dan.j.williams@intel.com> | 2020-12-17 13:52:28 -0800 |
| commit | 8bcbe3132c66c07d03f64d5da80be753359f2e92 (patch) | |
| tree | 1da7900e165214933e8780b5734b42b08c7233c0 /drivers | |
| parent | device-dax/core: Fix memory leak when rmmod dax.ko (diff) | |
| download | linux-dev-8bcbe3132c66c07d03f64d5da80be753359f2e92.tar.xz linux-dev-8bcbe3132c66c07d03f64d5da80be753359f2e92.zip | |
device-dax: delete a redundancy check in dev_dax_validate_align()
After we have done the alignment check for the length of each range, the
alignment check for dev_dax_size(dev_dax) is no longer needed, because it
get the sum of the length of each range.
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Link: https://lore.kernel.org/r/20201120092057.2144-1-thunder.leizhen@huawei.com
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/dax/bus.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c index 27513d311242..9761cb40d4bb 100644 --- a/drivers/dax/bus.c +++ b/drivers/dax/bus.c @@ -1114,16 +1114,9 @@ static ssize_t align_show(struct device *dev, static ssize_t dev_dax_validate_align(struct dev_dax *dev_dax) { - resource_size_t dev_size = dev_dax_size(dev_dax); struct device *dev = &dev_dax->dev; int i; - if (dev_size > 0 && !alloc_is_aligned(dev_dax, dev_size)) { - dev_dbg(dev, "%s: align %u invalid for size %pa\n", - __func__, dev_dax->align, &dev_size); - return -EINVAL; - } - for (i = 0; i < dev_dax->nr_range; i++) { size_t len = range_len(&dev_dax->ranges[i].range); |