[Prism54-devel] Re: CVS Update

Margit Schubert-While margitsw at t-online.de
Fri Aug 6 13:22:09 UTC 2004


At 15:11 06.08.2004 +0200, you wrote:
>After reading related code, I think the current code is OK. We could 
>change "if(!priv->mib) return;" to "BUG_ON(!priv->mib);".
>But if we want the driver safe, we need to ensure nobody accesses  the 
>priv struct after netdev is unregistered:
>* I think that unregistering netdevice syncs and disable ioctl (not sure 
>but it 'should' be true).
>* We must be sure that no wireless-stats update remains scheduled. 
>(use  "flush_scheduled_work();")
>Then it is safe to call "mgt_clean();"
>
>The only thing missing is thus a "flush_scheduled_work();" in 
>prism54_remove, right after "unregister_netdev()".
>
>         ...And justice for all
>         Aurelien

What is with suspend/resume/PSM ?

Cheers

Margit
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://prism54.org/pipermail/prism54-devel/attachments/20040806/e14492bb/attachment-0001.html


More information about the Prism54-devel mailing list