Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1162546AbaDCIIR (ORCPT ); Thu, 3 Apr 2014 04:08:17 -0400 Received: from smtp.ctxuk.citrix.com ([185.25.65.24]:31140 "EHLO SMTP.EU.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162517AbaDCIIH convert rfc822-to-8bit (ORCPT ); Thu, 3 Apr 2014 04:08:07 -0400 X-IronPort-AV: E=Sophos;i="4.97,785,1389744000"; d="scan'208";a="13059245" From: Paul Durrant To: Zoltan Kiss , Ian Campbell , Wei Liu , "xen-devel@lists.xenproject.org" CC: "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Jonathan Davies" Subject: RE: [PATCH net-next v3 1/2] xen-netback: Rename map ops Thread-Topic: [PATCH net-next v3 1/2] xen-netback: Rename map ops Thread-Index: AQHPTpWyUMj3QEgjiEWONrNjWuMRMJr/ipTw Date: Thu, 3 Apr 2014 08:08:03 +0000 Message-ID: <9AAE0902D5BC7E449B7C8E4E778ABCD02BB0F1@AMSPEX01CL01.citrite.net> References: <1396458298-30041-1-git-send-email-zoltan.kiss@citrix.com> In-Reply-To: <1396458298-30041-1-git-send-email-zoltan.kiss@citrix.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.80.2.29] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-DLP: AMS1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: Zoltan Kiss > Sent: 02 April 2014 18:05 > To: Ian Campbell; Wei Liu; xen-devel@lists.xenproject.org > Cc: Paul Durrant; netdev@vger.kernel.org; linux-kernel@vger.kernel.org; > Jonathan Davies; Zoltan Kiss > Subject: [PATCH net-next v3 1/2] xen-netback: Rename map ops > > Rename identifiers to state explicitly that they refer to map ops. > > Signed-off-by: Zoltan Kiss Reviewed-by: Paul Durrant > --- > v3: > - rename xenvif_tx_create_gop to xenvif_tx_create_map_op > > diff --git a/drivers/net/xen-netback/netback.c b/drivers/net/xen- > netback/netback.c > index ae34f5f..8f468ab 100644 > --- a/drivers/net/xen-netback/netback.c > +++ b/drivers/net/xen-netback/netback.c > @@ -819,13 +819,13 @@ struct xenvif_tx_cb { > > #define XENVIF_TX_CB(skb) ((struct xenvif_tx_cb *)(skb)->cb) > > -static inline void xenvif_tx_create_gop(struct xenvif *vif, > - u16 pending_idx, > - struct xen_netif_tx_request *txp, > - struct gnttab_map_grant_ref *gop) > +static inline void xenvif_tx_create_map_op(struct xenvif *vif, > + u16 pending_idx, > + struct xen_netif_tx_request *txp, > + struct gnttab_map_grant_ref *mop) > { > - vif->pages_to_map[gop-vif->tx_map_ops] = vif- > >mmap_pages[pending_idx]; > - gnttab_set_map_op(gop, idx_to_kaddr(vif, pending_idx), > + vif->pages_to_map[mop-vif->tx_map_ops] = vif- > >mmap_pages[pending_idx]; > + gnttab_set_map_op(mop, idx_to_kaddr(vif, pending_idx), > GNTMAP_host_map | GNTMAP_readonly, > txp->gref, vif->domid); > > @@ -879,7 +879,7 @@ static struct gnttab_map_grant_ref > *xenvif_get_requests(struct xenvif *vif, > shinfo->nr_frags++, txp++, gop++) { > index = pending_index(vif->pending_cons++); > pending_idx = vif->pending_ring[index]; > - xenvif_tx_create_gop(vif, pending_idx, txp, gop); > + xenvif_tx_create_map_op(vif, pending_idx, txp, gop); > frag_set_pending_idx(&frags[shinfo->nr_frags], > pending_idx); > } > > @@ -899,7 +899,7 @@ static struct gnttab_map_grant_ref > *xenvif_get_requests(struct xenvif *vif, > shinfo->nr_frags++, txp++, gop++) { > index = pending_index(vif->pending_cons++); > pending_idx = vif->pending_ring[index]; > - xenvif_tx_create_gop(vif, pending_idx, txp, gop); > + xenvif_tx_create_map_op(vif, pending_idx, txp, > gop); > frag_set_pending_idx(&frags[shinfo->nr_frags], > pending_idx); > } > @@ -939,9 +939,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; > @@ -950,11 +950,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); > @@ -967,10 +967,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); > @@ -1022,7 +1024,7 @@ check_frags: > } > } > > - *gopp = gop + 1; > + *gopp_map = gop_map + 1; > return err; > } > > @@ -1291,7 +1293,7 @@ static unsigned xenvif_tx_build_gops(struct xenvif > *vif, int budget) > } > } > > - xenvif_tx_create_gop(vif, pending_idx, &txreq, gop); > + xenvif_tx_create_map_op(vif, pending_idx, &txreq, gop); > > gop++; > > @@ -1398,7 +1400,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; > > @@ -1411,7 +1413,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); > @@ -1610,21 +1612,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); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/