Re: [Intel-wired-lan] [PATCHv2] ixgbe: Wait for 1ms, not 1us, after RST
From: Peter Hurley <hidden>
Date: 2015-10-27 22:45:12
On 10/27/2015 02:35 PM, ND Linux CI Server wrote:
Greetings, This email is automatically generated by ND's Linux Patch Testing framework based on aiaiai. I have performed some automatic testing of a patch (series) you submitted to intel-wired-lan@lists.osuosl.org The following contains output of any tests which failed to pass, and might be the result of developer error. The tests performed include but may not be limited to checkpatch.pl, bisection testing, compilation on a default kernel config, coccinelle scripts, cppcheck, and smatch. If you have received this email in error, or believe that aiaiai has detected a false positive, please email Jacob Keller [off-list ref].
False positive. As long as the delay is at least 1ms (which is guaranteed), slightly longer delays (relative to the existing reset delay of 100ms) are not harmful. Use of usleep_range() would be unnecessary overkill for the purpose. Regards, Peter Hurley
--- I have tested your changes [Intel-wired-lan] [PATCHv2] ixgbe: Wait for 1ms, not 1us, after RST Project: net (net-current development queue) Configurations: intel_defconfig,x86 Tested the patch(es) on top of the following commits: 505b857 ixgbe: Reset interface after enabling SR-IOV ce9d9b8 net: sysctl: fix a kmemleak warning 1acea4f ppp: fix pppoe_dev deletion condition in pppoe_release() f6b8dec9 af_key: fix two typos -------------------------------------------------------------------------------- Successfully built configuration "intel_defconfig,x86", no issues. -------------------------------------------------------------------------------- checkpatch.pl has some complaints: -------------------------------------------------------------------------------- checkpatch.pl results for patch "[PATCH] ixgbe: Wait for 1ms, not 1us, after RST" WARNING:MSLEEP: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt #29: FILE: drivers/net/ethernet/intel/ixgbe/ixgbe_x540.c:119: + msleep(1); total: 0 errors, 1 warnings, 0 checks, 13 lines checked --------------------------------------------------------------------------------