[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [PATCH net-next v2 1/2] xen-netback: Rename map ops



Rename identifiers to state explicitly that they refer to map ops.

Signed-off-by: Zoltan Kiss <zoltan.kiss@xxxxxxxxxx>
---
diff --git a/drivers/net/xen-netback/netback.c 
b/drivers/net/xen-netback/netback.c
index 0efa32d..a650899 100644
--- a/drivers/net/xen-netback/netback.c
+++ b/drivers/net/xen-netback/netback.c
@@ -915,9 +915,9 @@ static inline void xenvif_grant_handle_reset(struct xenvif 
*vif,
 
 static int xenvif_tx_check_gop(struct xenvif *vif,
                               struct sk_buff *skb,
-                              struct gnttab_map_grant_ref **gopp)
+                              struct gnttab_map_grant_ref **gopp_map)
 {
-       struct gnttab_map_grant_ref *gop = *gopp;
+       struct gnttab_map_grant_ref *gop_map = *gopp_map;
        u16 pending_idx = XENVIF_TX_CB(skb)->pending_idx;
        struct skb_shared_info *shinfo = skb_shinfo(skb);
        struct pending_tx_info *tx_info;
@@ -926,11 +926,11 @@ static int xenvif_tx_check_gop(struct xenvif *vif,
        struct sk_buff *first_skb = NULL;
 
        /* Check status of header. */
-       err = gop->status;
+       err = gop_map->status;
        if (unlikely(err))
                xenvif_idx_release(vif, pending_idx, XEN_NETIF_RSP_ERROR);
        else
-               xenvif_grant_handle_set(vif, pending_idx , gop->handle);
+               xenvif_grant_handle_set(vif, pending_idx , gop_map->handle);
 
        /* Skip first skb fragment if it is on same page as header fragment. */
        start = (frag_get_pending_idx(&shinfo->frags[0]) == pending_idx);
@@ -943,10 +943,12 @@ check_frags:
                tx_info = &vif->pending_tx_info[pending_idx];
 
                /* Check error status: if okay then remember grant handle. */
-               newerr = (++gop)->status;
+               newerr = (++gop_map)->status;
 
                if (likely(!newerr)) {
-                       xenvif_grant_handle_set(vif, pending_idx , gop->handle);
+                       xenvif_grant_handle_set(vif,
+                                               pending_idx,
+                                               gop_map->handle);
                        /* Had a previous error? Invalidate this fragment. */
                        if (unlikely(err))
                                xenvif_idx_unmap(vif, pending_idx);
@@ -998,7 +1000,7 @@ check_frags:
                }
        }
 
-       *gopp = gop + 1;
+       *gopp_map = gop_map + 1;
        return err;
 }
 
@@ -1374,7 +1376,7 @@ static int xenvif_handle_frag_list(struct xenvif *vif, 
struct sk_buff *skb)
 
 static int xenvif_tx_submit(struct xenvif *vif)
 {
-       struct gnttab_map_grant_ref *gop = vif->tx_map_ops;
+       struct gnttab_map_grant_ref *gop_map = vif->tx_map_ops;
        struct sk_buff *skb;
        int work_done = 0;
 
@@ -1387,7 +1389,7 @@ static int xenvif_tx_submit(struct xenvif *vif)
                txp = &vif->pending_tx_info[pending_idx].req;
 
                /* Check the remap error code. */
-               if (unlikely(xenvif_tx_check_gop(vif, skb, &gop))) {
+               if (unlikely(xenvif_tx_check_gop(vif, skb, &gop_map))) {
                        netdev_dbg(vif->dev, "netback grant failed.\n");
                        skb_shinfo(skb)->nr_frags = 0;
                        kfree_skb(skb);
@@ -1586,21 +1588,21 @@ static inline void xenvif_tx_dealloc_action(struct 
xenvif *vif)
 /* Called after netfront has transmitted */
 int xenvif_tx_action(struct xenvif *vif, int budget)
 {
-       unsigned nr_gops;
+       unsigned nr_mops;
        int work_done, ret;
 
        if (unlikely(!tx_work_todo(vif)))
                return 0;
 
-       nr_gops = xenvif_tx_build_gops(vif, budget);
+       nr_mops = xenvif_tx_build_gops(vif, budget);
 
-       if (nr_gops == 0)
+       if (nr_mops == 0)
                return 0;
 
        ret = gnttab_map_refs(vif->tx_map_ops,
                              NULL,
                              vif->pages_to_map,
-                             nr_gops);
+                             nr_mops);
        BUG_ON(ret);
 
        work_done = xenvif_tx_submit(vif);

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.