Re: [PATCH] net: ethernet : stmicro: fixed power suspend and resume failure in stmmac driver

From: Giuseppe CAVALLARO
Date: Thu Oct 16 2014 - 04:58:46 EST


Hello Hao Liang

On 10/1/2014 7:45 PM, David Miller wrote:
From: Hao Liang <hliang1025@xxxxxxxxx>
Date: Wed, 1 Oct 2014 14:08:28 +0800

I double-check my patch and the ->mac->xxx calls are still under the lock.
I think that lock is trying to protect priv struct and related data, so i
just remove some functions have no bearing on priv struct.

It's preventing parallel invocations of the ->mac->xxx calls.

The other instances are in device open/close, where RTNL semaphore is
held, and no other code paths in the driver can be active.

You need the lock.

Do you have a new patch for this problem after David's advice?

I am reviewing the patches sent some weeks ago for driver looking
and I can also try to fix this in case of you have no news.

Let me know,

Regards,
peppe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/