Re: [Intel-wired-lan] [PATCHv2] ixgbe: Wait for 1ms, not 1us, after RST

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

--------------------------------------------------------------------------------