[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