> -----Original Message----- > From: Ilya Maximets [mailto:[email protected]] > Sent: Thursday, November 1, 2018 11:05 AM > To: [email protected]; Zhang, Qi Z <[email protected]> > Cc: Lu, Wenzhuo <[email protected]>; Ananyev, Konstantin > <[email protected]>; Laurent Hardy > <[email protected]>; Wei Dai <[email protected]>; Yigit, Ferruh > <[email protected]>; Ilya Maximets <[email protected]>; > [email protected] > Subject: [PATCH v2] net/ixgbe: fix busy polling while fiber link update > > If the multispeed fiber link is in DOWN state, ixgbe_setup_link could take > around a second of busy polling. This is highly inconvenient for the case > where single thread periodically checks the link statuses. For example, OVS > main thread periodically updates the link statuses and hangs for a really long > time busy waiting on ixgbe_setup_link() for a DOWN fiber ports. For case > with 3 down ports it hangs for a 3 seconds and unable to do anything > including packet processing. > Fix that by shifting that workaround to a separate thread by alarm handler > that will try to set up link if it is DOWN. > > Fixes: c12d22f65b13 ("net/ixgbe: ensure link status is updated") > CC: [email protected] > > Signed-off-by: Ilya Maximets <[email protected]>
Applied to dpdk-next-net-intel. Thanks Qi
