RE: [Intel-wired-lan] [PATCH 4/5] e1000e: Separate signaling for link check/link up

2017-09-14 Thread Brown, Aaron F
On 7/21/2017 21:36, Benjamin Poirier wrote: > Lennart reported the following race condition: > > \ e1000_watchdog_task > \ e1000e_has_link > \ hw->mac.ops.check_for_link() === e1000e_check_for_copper_link > /* link is up */ > mac->get_link_status = false; >

Re: [Intel-wired-lan] [PATCH 4/5] e1000e: Separate signaling for link check/link up

2017-08-02 Thread Benjamin Poirier
On 2017/08/02 10:34, Lennart Sorensen wrote: > On Wed, Aug 02, 2017 at 02:28:07PM +0300, Neftin, Sasha wrote: > > On 7/21/2017 21:36, Benjamin Poirier wrote: > > > Lennart reported the following race condition: > > > > > > \ e1000_watchdog_task > > > \ e1000e_has_link > > > \ hw->mac

Re: [Intel-wired-lan] [PATCH 4/5] e1000e: Separate signaling for link check/link up

2017-08-02 Thread Lennart Sorensen
On Wed, Aug 02, 2017 at 02:28:07PM +0300, Neftin, Sasha wrote: > On 7/21/2017 21:36, Benjamin Poirier wrote: > > Lennart reported the following race condition: > > > > \ e1000_watchdog_task > > \ e1000e_has_link > > \ hw->mac.ops.check_for_link() === e1000e_check_for_copper_link > >

Re: [Intel-wired-lan] [PATCH 4/5] e1000e: Separate signaling for link check/link up

2017-08-02 Thread Neftin, Sasha
On 7/21/2017 21:36, Benjamin Poirier wrote: Lennart reported the following race condition: \ e1000_watchdog_task \ e1000e_has_link \ hw->mac.ops.check_for_link() === e1000e_check_for_copper_link /* link is up */ mac->get_link_status = false;