[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 31 Dec 2015 15:11:38 +0800
From: <zyjzyj2000@...il.com>
To: <jeffrey.t.kirsher@...el.com>, <jesse.brandeburg@...el.com>,
<shannon.nelson@...el.com>, <carolyn.wyborny@...el.com>,
<donald.c.skidmore@...el.com>, <bruce.w.allan@...el.com>,
<john.ronciak@...el.com>, <mitch.a.williams@...el.com>,
<intel-wired-lan@...ts.osuosl.org>, <netdev@...r.kernel.org>,
<e1000-devel@...ts.sourceforge.net>, <mark.d.rustad@...el.com>
CC: <venkat.viswanathan@...driver.com>,
<Boris.Shteinbock@...driver.com>, <Vincent.Bourg@...driver.com>
Subject: [PATCH V6] ixgbe: synchronize link_up and link_speed of a slave
Hi, all
Thanks for the reply from Jeff.
V2: Based on feedback from Jeff Kirsher, it is not appropriate to continue
in the function ixgbe_watchdog_link_is_up without link_speed since this will
make some virtual NICs not work well.
V3: Based on feedback from Emil Tantilov, the time span between link_up
and link_speed is not important when the X540 NIC acts as an independent
interface.
V4: According to Rustad, Mark D, maybe it is related with copper phy. To make
fiber phy more robust, synchronize both the link_up and link_speed of a slave
interface in ixgbe driver.
V5: Based on feedback from Rustad, Mark D, simplify code comment and if statement
to only test for IFF_SLAVE flag and unknown link speed.
V6: Based on feedback from Jeff Kirsher, the patch format is adjusted
and change log of this patch are added.
Best Regards!
Zhu Yanjun
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists