[Intel-wired-lan] [PATCH] ixgbevf: add control of skipping PF link-status check

[Wang, Liang-min]

You ACK the patch, and that's why I manually added reviewed-by. 

Larry

> -----Original Message-----
> From: Kirsher, Jeffrey T
> Sent: Wednesday, June 12, 2019 4:24 PM
> To: Wang, Liang-min <liang-min.wang at intel.com>; intel-wired-
> lan at lists.osuosl.org
> Cc: Guilherme Piccoli <gpiccoli at canonical.com>
> Subject: Re: [PATCH] ixgbevf: add control of skipping PF link-status check
> 
> On Tue, 2019-06-11 at 17:55 -0700, Wang, Liang-min wrote:
> > ixgbevf: add control of skipping PF link-status check
> >
> > The ixgbevf link status check always triggers an interrupt on Physical
> > Function (PF) driver because
> > it uses mailbox read to check PF link-status. For any Virtual Network
> > Function (VNF) implementation
> > where periodical link-status check is needed, this interrupt per request
> > becomes a significant
> > performance overhead. Therefore, this change uses the input parameter,
> > autoneg_wait_to_complete, to
> > conditionally skip the PF link-status check. For scenarios where only
> > quick Virtual Function (VF)
> > link status check is required, application could skip the PF interrupt by
> > setting autoneg_wait_to_complete
> > to "0". For cases where deep link status check (include both PF and VF
> > link status check) is required,
> > the application can set the autoneg_wait_to_complete to "1".
> >
> > Signed-off-by: Liang-Min Wang<liang-min.wang at intel.com>
> > Reviewed-by: Jeffrey T Kirsher<jeffrey.t.kirsher at intel.com>
> 
> Interesting, I know I did an initial review, but I do not remember adding
> my reviewed-by to this patch yet.  FYI, I never use my full legal name when
> signing any patch, which tells me you forged my Reviewed-by.
> 
> > ---
> > drivers/net/ethernet/intel/ixgbevf/vf.c | 10 ++++++++++
> > 1 file changed, 10 insertions(+)