Whamcloud - gitweb
LU-4423 lnet: Don't ignore error code for lst_group_info_ioctl 52/18952/2
authorCristina Moraru <cristina.moraru09@gmail.com>
Wed, 16 Mar 2016 15:43:27 +0000 (11:43 -0400)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 23 Mar 2016 06:03:40 +0000 (06:03 +0000)
Error stored in rc was never reported, so directly return it here.

Linux-commit : 2a96be0d46825689ac9b6078056e523ba96acec4

Change-Id: I0535d0a101c1142f035e6dfba240e95fc9040eb2
Signed-off-by: Cristina Moraru <cristina.moraru09@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-on: http://review.whamcloud.com/18952
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Doug Oucharek <doug.s.oucharek@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lnet/selftest/conctl.c

index bd438eb..804a7a0 100644 (file)
@@ -436,7 +436,7 @@ lst_group_info_ioctl(lstio_group_info_args_t *args)
        if (args->lstio_grp_dentsp != NULL &&
            (copy_to_user(args->lstio_grp_idxp, &index, sizeof(index)) ||
             copy_to_user(args->lstio_grp_ndentp, &ndent, sizeof(ndent))))
-               rc = -EFAULT;
+               return -EFAULT;
 
        return 0;
 }