Browse Source

netdev: use list_first_entry macro

Use list_first_entry macro; no longer any need to use
'next' directly in list to find first entry.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
stephen hemminger 15 years ago
parent
commit
e5e26d75f4
1 changed files with 3 additions and 3 deletions
  1. 3 3
      net/core/dev.c

+ 3 - 3
net/core/dev.c

@@ -3018,7 +3018,7 @@ static void net_rx_action(struct softirq_action *h)
 		 * entries to the tail of this list, and only ->poll()
 		 * entries to the tail of this list, and only ->poll()
 		 * calls can remove this head entry from the list.
 		 * calls can remove this head entry from the list.
 		 */
 		 */
-		n = list_entry(list->next, struct napi_struct, poll_list);
+		n = list_first_entry(list, struct napi_struct, poll_list);
 
 
 		have = netpoll_poll_lock(n);
 		have = netpoll_poll_lock(n);
 
 
@@ -4882,7 +4882,7 @@ static void rollback_registered_many(struct list_head *head)
 	}
 	}
 
 
 	/* Process any work delayed until the end of the batch */
 	/* Process any work delayed until the end of the batch */
-	dev = list_entry(head->next, struct net_device, unreg_list);
+	dev = list_first_entry(head, struct net_device, unreg_list);
 	call_netdevice_notifiers(NETDEV_UNREGISTER_BATCH, dev);
 	call_netdevice_notifiers(NETDEV_UNREGISTER_BATCH, dev);
 
 
 	synchronize_net();
 	synchronize_net();
@@ -5268,7 +5268,7 @@ void netdev_run_todo(void)
 
 
 	while (!list_empty(&list)) {
 	while (!list_empty(&list)) {
 		struct net_device *dev
 		struct net_device *dev
-			= list_entry(list.next, struct net_device, todo_list);
+			= list_first_entry(&list, struct net_device, todo_list);
 		list_del(&dev->todo_list);
 		list_del(&dev->todo_list);
 
 
 		if (unlikely(dev->reg_state != NETREG_UNREGISTERING)) {
 		if (unlikely(dev->reg_state != NETREG_UNREGISTERING)) {