From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f178.google.com (mail-wi0-f178.google.com [209.85.212.178]) by dpdk.org (Postfix) with ESMTP id F322458DC for ; Mon, 13 Oct 2014 21:44:22 +0200 (CEST) Received: by mail-wi0-f178.google.com with SMTP id h11so4800020wiw.11 for ; Mon, 13 Oct 2014 12:52:04 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding:content-type; bh=As3Czh2UwMyTER3+w0LtnK1CB9wqYImVqIuSMdHX9hs=; b=kLlkqHOvve8oxSTk4V73ZiKp+mUTFRVwltjuPHEXNYbxCbbkOvsZ4MdSfRCPNHdQUt kJFy8maNmEd2cs0EB/ub9k8dth2IiQuLtxZPT8iPTQsOFFsH9Kg5POTFz/pR/Et+vx1T w0vIEHNhNFC0wbmFoMFuZT0VkhGGVTCAJIl+GxVg5PvXQoNcJqI34d2IsRCs1sZgH4TG fE95CAeUnFEJ6zN/tLNF0jgCiBD9KV3tjENTQMCCE++V3B9eoLeEDYW2Q1H30ZQ5zBiq tz0WsCWALOZUSOwNlUU1CKi3fKwHw0gJN+TSUhYU51onD2T8dZM831kDDGKSJzsB+sLa fYmA== X-Gm-Message-State: ALoCoQkmXlgEGnibBHd87g28vGyaajcjOair+KpViWlcpxCNb7Me4lJ41flN1z0l09ytNNjh5pDr X-Received: by 10.194.249.163 with SMTP id yv3mr428995wjc.91.1413229924115; Mon, 13 Oct 2014 12:52:04 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by mx.google.com with ESMTPSA id wk5sm17740686wjb.12.2014.10.13.12.52.02 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 13 Oct 2014 12:52:03 -0700 (PDT) From: Thomas Monjalon To: Huawei Xie Date: Mon, 13 Oct 2014 21:51:48 +0200 Message-ID: <4458709.TeG1xlHSS8@xps13> Organization: 6WIND User-Agent: KMail/4.14.1 (Linux/3.16.4-1-ARCH; KDE/4.14.1; x86_64; ; ) In-Reply-To: <1412794499-4332-1-git-send-email-huawei.xie@intel.com> References: <1412794499-4332-1-git-send-email-huawei.xie@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v6 00/25] user space vhost library X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 Oct 2014 19:44:23 -0000 Hi Huawei, 2014-10-09 02:54, Huawei Xie: > This set of patches transforms and refactors vhost example to a user > space vhost library. > This library implements a user space vhost cuse driver, and provides > generic APIs for user space ethernet vSwitch to integrate us-vhost for > fast packet switching with guest virtio. > > vhost lib consists of five APIs plus several helper routines > for feature disable/enable. > 1) rte_vhost_driver_register initializes vhost driver. > 2) rte_vhost_driver_callback_register registers the callbacks. > Callbacks are called from vhost driver when a virtio device is ready > to be added to the data processing core or is de-activated by guest. > 3) rte_vhost_driver_session_start, a blocking API to start vhost > message handler session loop. > 4) rte_vhost_enqueue_burst and rte_vhost_dequeue_burst for > enqueue/dequeue packets to/from virtio ring respectively. > > v2) turn off vhost lib by default > > v3) fixed checkpatch issues > > v4) split the patch per thomas' requirement > > v5) fine granularity split of the patch > regenerate patches based on latest commit > this patchset removes vhost example patches, which will be > submitted later. > > Huawei Xie (25): > move src files from examples/vhost to lib/librte_vhost > rename main.c to vhost_rxtx.c and virtio-net.h to rte_vhost_net.h > keep virtio_dev_(merge_)rx, copy_from_mbuf_to_vring and virtio_dev_merge_tx; remove anything else in vhost_rxtx.c > remove mac learning, mac/vlan, VMDQ and other switching related logic > remove host memory region region related logic > remove retry > patch virtio_dev_merge_tx to make it return packets to app > patch vhost_dev_merge_tx about buf_size > add queue_id parameter to vhost rx/tx functions > define PACKET_BURST > rte_vhost_en/dequeue_burst API > move virtio_net_config_ll strcture to virtio_net.c > remove index > call get_virtio_net_callbacks to get the ops in register_cuse_device > rte_vhost_driver_register and rte_vhost_session_start API > rte_vhost_callback_register API > add debug print > define VHOST_SUPPORTED_FEATURES > header file cleanup > static fix > add priv field in virtio_net to store application specific context > coding style fixes > add TODO/FIXME > add vhost support in Makefile Thanks for your hard work. There are still few things to clean in this patch splitting but I've did it to apply them. I won't describe all the changes I've done, you can check them in the git repository. In short, some split or merge were needed, some lines were removed and re-added later, build dependencies were not correct and doc generation was missing. You did the big work by really splitting all these stuff. Working on small commits was far easier. Thanks Applied Now you can add the new example. I hope we'll have more reviews and cleanup now that the first version of this library is integrated. -- Thomas