From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f172.google.com (mail-wi0-f172.google.com [209.85.212.172]) by dpdk.org (Postfix) with ESMTP id E7A5E8D93 for ; Tue, 22 Sep 2015 15:38:19 +0200 (CEST) Received: by wiclk2 with SMTP id lk2so24361415wic.1 for ; Tue, 22 Sep 2015 06:38:19 -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=IUM+MYg8ZdfOr2Qw1omSqf0H/6sasuiMP4xbyDiUsE4=; b=Y3oAJjWXIaLbBOmZ2STgg7jPBGjhs0AHk8JGie9qGlbaBaLamkUW6Gq0BIRzQbw8Y7 KVCK2PFp2w7B+17oKjiIFfcZXrqOqq7o1HsG4smn3t6apCcJhugiBonemy86TFaAPfi0 emKijPOgZrsarPukPfs/MVeegnPV6oCwvsUkhl/nqgswMCX6wCS27d0OVIbGDMTerOln r3iIqHdFleeZwIBkRrUNhEJPVShg6IBejnbpyuKvTpZkCuPvwaDaoL3gkgVzoXyK6CoL HtXbNvoAj033dn0anQqO4qWJqBb0HxJDcLSv0ovtm1/TuyNq/z6Ff/j0o7ntoFGB9jQy SEsQ== X-Gm-Message-State: ALoCoQmsz3brgWCozDTdobBbm+ku6i297CmEcJYpQrGQiayO8B4YHoPrjtl5GEieuKRzLqOhL515 X-Received: by 10.180.107.130 with SMTP id hc2mr18461095wib.92.1442929099710; Tue, 22 Sep 2015 06:38:19 -0700 (PDT) Received: from xps13.localnet (136-92-190-109.dsl.ovh.fr. [109.190.92.136]) by smtp.gmail.com with ESMTPSA id uc12sm3214367wib.13.2015.09.22.06.38.18 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 22 Sep 2015 06:38:19 -0700 (PDT) From: Thomas Monjalon To: "Xie, Huawei" Date: Tue, 22 Sep 2015 15:37:13 +0200 Message-ID: <4377815.VgSsvvaXDb@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: References: <1442506651-16279-1-git-send-email-huawei.xie@intel.com> <4527609.XBoLQSHpyv@xps13> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [RFC PATCH] virtio: virtio ring layout optimization and vectorization rx 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: Tue, 22 Sep 2015 13:38:20 -0000 2015-09-22 09:29, Xie, Huawei: > On 9/22/2015 4:38 PM, Thomas Monjalon wrote: > > I thought it was clear we must avoid #ifdef for configuration. > > Please check how to add a runtime flag. > > Or better: keep only one path which works everywhere and well optimized. > > Thomas, i have this in mind when i am developing this feature and i plan > to discuss this with you. In theory, the best solution would be using > run time flag and the result would be normally we always choose the > optimized path. > The situation in real world is, in my mind, this feature isn't mature > enough(Of course this doesn't mean it has known issues). One good > solution in kernel community is experimental features. > Could we also have experimental/development tag? I would like to put > this as experimental/development feature temporarily, turn it off by > default and continue to improve it. I think it is a rework, not an experimental feature. > The advantage is we have it in tree and every developers could pull it > and help improve it. Also, it is the base for continuous vhost > optimization. We should have a performance optimized driver for the > device side optimization. If there are some acceptance problem, it can be solved on the list. If there are some bugs after integration, it can be fixed before the release, as there are some validation and non-regression tests. If the new Rx/Tx code can replace the old one, please replace it. If the use cases are different, please use a runtime flag. It it is not ready, please do more tests and re-submit later. By the way, thanks for your work.