[Intel-wired-lan] [next PATCH] igb/igbvf: don't give up

Brown, Aaron F aaron.f.brown at intel.com
Fri Dec 18 01:49:47 UTC 2015


> From: Intel-wired-lan [intel-wired-lan-bounces at lists.osuosl.org] on behalf of Mitch Williams [mitch.a.williams at intel.com]
> Sent: Friday, December 11, 2015 4:45 PM
> To: intel-wired-lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [next PATCH] igb/igbvf: don't give up
> 
> The driver shouldn't just give up if it fails to get the hardware
> mailbox lock. This can happen in a situation where the PF-VF
> communication channel is heavily loaded and causes complete
> communications failure between the PF and VF drivers.
> 
> Add a counter and a delay. The driver will now retry ten times, waiting
> one millsecond between retries.

> Signed-off-by: Mitch Williams <mitch.a.williams at intel.com>
> ---
>  drivers/net/ethernet/intel/igb/e1000_mbx.c | 18 ++++++++++++------
>  drivers/net/ethernet/intel/igbvf/mbx.c     | 20 +++++++++++++-------
>  2 files changed, 25 insertions(+), 13 deletions(-)

This is getting flagged by checkpatch for using udelay rather than usleep_range.  But it does not seem to be considering it an error or even a warning, rather it just seems to come up as a "CHECK:".

Is there any reason not to use usleep_range here?
--------------------------------------------------------------------------------------------
u1458:[1]/usr/src/kernels/next-queue> git format-patch $item -1 --stdout|./scripts/checkpatch.pl -
CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt
#46: FILE: drivers/net/ethernet/intel/igb/e1000_mbx.c:337:
+		udelay(1000);

CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt
#77: FILE: drivers/net/ethernet/intel/igbvf/mbx.c:248:
+		udelay(1000);

total: 0 errors, 0 warnings, 2 checks, 52 lines checked

Your patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
u1458:[1]/usr/src/kernels/next-queue> 


More information about the Intel-wired-lan mailing list