[dpdk-dev] [PATCH] net/ixgbe: fix busy wait in ixgbevf_check_link()

Zhang, Helin helin.zhang at intel.com
Fri Apr 6 17:16:24 CEST 2018



> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zhang, Helin
> Sent: Friday, April 6, 2018 11:09 PM
> To: Chas Williams; dev at dpdk.org
> Cc: Lu, Wenzhuo; Ananyev, Konstantin; Charles (Chas) Williams
> Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: fix busy wait in ixgbevf_check_link()
> 
> 
> 
> > -----Original Message-----
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Chas Williams
> > Sent: Friday, February 2, 2018 6:39 AM
> > To: dev at dpdk.org
> > Cc: Lu, Wenzhuo; Ananyev, Konstantin; Charles (Chas) Williams
> > Subject: [dpdk-dev] [PATCH] net/ixgbe: fix busy wait in
> > ixgbevf_check_link()
> >
> > From: "Charles (Chas) Williams" <chas3 at att.com>
> >
> > If we haven't set wait_to_complete, there is no need to busy wait
> > until we have a link status.  Applications, like bonding, use the
> > wait_to_complete flag to indicate that they will be doing their own
> > busy wait and will likely be polling again shortly.
> >
> > Fixes: dc66e5fd01b9 ("net/ixgbe: improve link state check on VF")
> >
> > Signed-off-by: Chas Williams <chas3 at att.com>
> Acked-by: Helin Zhang <helin.zhang at intel.com>
Applied to dpdk-next-net-intel, with title modifcations. Thanks!

/Helin
> 
> BTW, the title should be corrected by following the format rule of patches on
> DPDK community. Basically it shouldn't mention the function name in the title.


More information about the dev mailing list