Thread (2 messages) 2 messages, 2 authors, 2015-08-29

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

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help