Re: [PATCH 13/16] caif_virtio: use simplified virtqueue accessors.

From: Wanlong Gao
Date: Wed Feb 20 2013 - 06:12:42 EST


On 02/19/2013 03:56 PM, Rusty Russell wrote:
> We never add buffers with input and output parts, so use the new accessors.
>
> Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx>

Reviewed-by: Wanlong Gao <gaowanlong@xxxxxxxxxxxxxx>

> ---
> drivers/net/caif/caif_virtio.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/net/caif/caif_virtio.c b/drivers/net/caif/caif_virtio.c
> index df832e7..9360055 100644
> --- a/drivers/net/caif/caif_virtio.c
> +++ b/drivers/net/caif/caif_virtio.c
> @@ -364,8 +364,7 @@ static int cfv_netdev_tx(struct sk_buff *skb, struct net_device *netdev)
> /* Add buffer to avail ring. Flow control below should ensure
> * that this always succeedes
> */
> - ret = virtqueue_add_buf(cfv->vq_tx, &sg, 1, 0,
> - buf_info, GFP_ATOMIC);
> + ret = virtqueue_add_outbuf(cfv->vq_tx, &sg, 1, buf_info, GFP_ATOMIC);
>
> if (unlikely(WARN_ON(ret < 0))) {
> kfree(buf_info);
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/