diff options
author | Leon Romanovsky <leonro@mellanox.com> | 2018-05-29 08:14:05 -0400 |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2018-06-01 11:16:24 -0400 |
commit | 2cb4079188a1421520372f5e57ddaceab39435aa (patch) | |
tree | 0a4bc04cb8d6ea5a4734d911b4fd7956f40c6909 | |
parent | 2468b82d69e3a53d024f28d79ba0fdb8bf43dfbf (diff) |
RDMA/mlx5: Don't check return value of zap_vma_ptes()
There is no need to check return value of zap_vma_ptes()
because there is nothing to do with this knowledge.
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
-rw-r--r-- | drivers/infiniband/hw/mlx5/main.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c index a182d19c557e..0541581c5d84 100644 --- a/drivers/infiniband/hw/mlx5/main.c +++ b/drivers/infiniband/hw/mlx5/main.c | |||
@@ -1969,7 +1969,6 @@ static int mlx5_ib_set_vma_data(struct vm_area_struct *vma, | |||
1969 | 1969 | ||
1970 | static void mlx5_ib_disassociate_ucontext(struct ib_ucontext *ibcontext) | 1970 | static void mlx5_ib_disassociate_ucontext(struct ib_ucontext *ibcontext) |
1971 | { | 1971 | { |
1972 | int ret; | ||
1973 | struct vm_area_struct *vma; | 1972 | struct vm_area_struct *vma; |
1974 | struct mlx5_ib_vma_private_data *vma_private, *n; | 1973 | struct mlx5_ib_vma_private_data *vma_private, *n; |
1975 | struct mlx5_ib_ucontext *context = to_mucontext(ibcontext); | 1974 | struct mlx5_ib_ucontext *context = to_mucontext(ibcontext); |
@@ -1978,9 +1977,7 @@ static void mlx5_ib_disassociate_ucontext(struct ib_ucontext *ibcontext) | |||
1978 | list_for_each_entry_safe(vma_private, n, &context->vma_private_list, | 1977 | list_for_each_entry_safe(vma_private, n, &context->vma_private_list, |
1979 | list) { | 1978 | list) { |
1980 | vma = vma_private->vma; | 1979 | vma = vma_private->vma; |
1981 | ret = zap_vma_ptes(vma, vma->vm_start, | 1980 | zap_vma_ptes(vma, vma->vm_start, PAGE_SIZE); |
1982 | PAGE_SIZE); | ||
1983 | WARN_ONCE(ret, "%s: zap_vma_ptes failed", __func__); | ||
1984 | /* context going to be destroyed, should | 1981 | /* context going to be destroyed, should |
1985 | * not access ops any more. | 1982 | * not access ops any more. |
1986 | */ | 1983 | */ |