Re: [PATCH, net-next] r8169: Deny functions membet of array rtl_work
From: Francois Romieu <romieu@fr.zoreil.com>
Date: 2015-08-29 11:26:55
From: Francois Romieu <romieu@fr.zoreil.com>
Date: 2015-08-29 11:26:55
Corcodel Marian [off-list ref] :
Deny functions member of array rtl_work to run when link is ok.After rtl8169_init_phy exist 10 sec timeout.On many cases link is ok and no need to destroy all work.
I won't figure what happens outside of the "many cases" statement. rtl_reset_work is a hard reset. It isn't supposed to go shopping just because the link seems to be established. Moreover you don't explain why you need to vandalize rtl_phy_work (unrelated to any 10 sec timeout). As usual nobody knows how it was tested (suspend/resume, change location and port, etc.). Please don't send me private only e-mail. -- Ueimor