diff options
author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2015-10-17 23:58:19 +0300 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2015-11-23 22:08:52 +0300 |
commit | 0286c67e710cc82260b8c826e5a7b75143fedbd6 (patch) | |
tree | 90e58f084703ecbcd22a15db0e51b9e411d2989e /net | |
parent | 465fc643c2dcbe08e0debac80c225f6750b40d3c (diff) | |
download | linux-0286c67e710cc82260b8c826e5a7b75143fedbd6.tar.xz |
intel: i40e: fix confused code
This code is pretty confused. The variable name 'bytes_not_copied'
clearly indicates that the programmer knew the semantics of
copy_{to,from}_user, but then the return value is checked for being
negative and used as a -Exxx return value.
I'm not sure this is the proper fix, but at least we get rid of the
dead code which pretended to check for access faults.
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Acked-by: Shannon Nelson <shannon.nelson@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions