diff options
author | Wang Chen <wangchen@cn.fujitsu.com> | 2008-11-13 02:37:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-13 02:37:49 -0500 |
commit | 454d7c9b14e20fd1949e2686e9de4a2926e01476 (patch) | |
tree | fae8bd1bfb5fd496977a0639ad71c54a2ee278ae /drivers/net/atp.c | |
parent | 7a12122c7a00347da9669cfcde82954c9e3d6f5e (diff) |
netdevice: safe convert to netdev_priv() #part-1
We have some reasons to kill netdev->priv:
1. netdev->priv is equal to netdev_priv().
2. netdev_priv() wraps the calculation of netdev->priv's offset, obviously
netdev_priv() is more flexible than netdev->priv.
But we cann't kill netdev->priv, because so many drivers reference to it
directly.
This patch is a safe convert for netdev->priv to netdev_priv(netdev).
Since all of the netdev->priv is only for read.
But it is too big to be sent in one mail.
I split it to 4 parts and make every part smaller than 100,000 bytes,
which is max size allowed by vger.
Signed-off-by: Wang Chen <wangchen@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/atp.c')
-rw-r--r-- | drivers/net/atp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/atp.c b/drivers/net/atp.c index 405c2dc5f3e4..7028b276dfd3 100644 --- a/drivers/net/atp.c +++ b/drivers/net/atp.c | |||
@@ -912,7 +912,8 @@ static void __exit atp_cleanup_module(void) { | |||
912 | struct net_device *next_dev; | 912 | struct net_device *next_dev; |
913 | 913 | ||
914 | while (root_atp_dev) { | 914 | while (root_atp_dev) { |
915 | next_dev = ((struct net_local *)root_atp_dev->priv)->next_module; | 915 | struct net_local *atp_local = netdev_priv(root_atp_dev); |
916 | next_dev = atp_local->next_module; | ||
916 | unregister_netdev(root_atp_dev); | 917 | unregister_netdev(root_atp_dev); |
917 | /* No need to release_region(), since we never snarf it. */ | 918 | /* No need to release_region(), since we never snarf it. */ |
918 | free_netdev(root_atp_dev); | 919 | free_netdev(root_atp_dev); |