Commit 01738448 authored by Luis de Bethencourt's avatar Luis de Bethencourt Committed by Greg Kroah-Hartman

staging: lustre: o2iblnd: fix misleading indentation

The code is correct, the indentation is misleading. Only the the return rc
is part of the conditional statement if rc != 0.

Fix a smatch warning:
drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c:2157
kiblnd_hdev_setup_mrs() warn: curly braces intended?
Signed-off-by: default avatarLuis de Bethencourt <luisbg@osg.samsung.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a77ddf7c
...@@ -2154,23 +2154,23 @@ static int kiblnd_hdev_setup_mrs(kib_hca_dev_t *hdev) ...@@ -2154,23 +2154,23 @@ static int kiblnd_hdev_setup_mrs(kib_hca_dev_t *hdev)
if (rc != 0) if (rc != 0)
return rc; return rc;
LIBCFS_ALLOC(hdev->ibh_mrs, 1 * sizeof(*hdev->ibh_mrs)); LIBCFS_ALLOC(hdev->ibh_mrs, 1 * sizeof(*hdev->ibh_mrs));
if (hdev->ibh_mrs == NULL) { if (hdev->ibh_mrs == NULL) {
CERROR("Failed to allocate MRs table\n"); CERROR("Failed to allocate MRs table\n");
return -ENOMEM; return -ENOMEM;
} }
hdev->ibh_mrs[0] = NULL; hdev->ibh_mrs[0] = NULL;
hdev->ibh_nmrs = 1; hdev->ibh_nmrs = 1;
mr = ib_get_dma_mr(hdev->ibh_pd, acflags); mr = ib_get_dma_mr(hdev->ibh_pd, acflags);
if (IS_ERR(mr)) { if (IS_ERR(mr)) {
CERROR("Failed ib_get_dma_mr : %ld\n", PTR_ERR(mr)); CERROR("Failed ib_get_dma_mr : %ld\n", PTR_ERR(mr));
kiblnd_hdev_cleanup_mrs(hdev); kiblnd_hdev_cleanup_mrs(hdev);
return PTR_ERR(mr); return PTR_ERR(mr);
} }
hdev->ibh_mrs[0] = mr; hdev->ibh_mrs[0] = mr;
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment