From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-it0-f67.google.com (mail-it0-f67.google.com [209.85.214.67]) by dpdk.org (Postfix) with ESMTP id 28D771B025 for ; Wed, 17 Jan 2018 09:47:09 +0100 (CET) Received: by mail-it0-f67.google.com with SMTP id c16so8255262itc.5 for ; Wed, 17 Jan 2018 00:47:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=6ffCzSWxnp7XTAXu7wTtYzf7VFzbZ1BdtrI9e0kd5ig=; b=XFwYB4lGl/XMATdI+eoxp+pKvOOTIy9gT8uhVb2X5++AZDv27UpJxubKVqJ1X1cS/2 WBQhM4gbp0rP9obodO70VuUxgsZxMyNfR9OMZVVLT46uy5SPe++d/hwdiXEIU93ZZMY2 LNMGVthfKIixXcewoQKsH4Ots7LVwFIfZpRs7rthL4Btasfg2e28pWbW3HZQU5hco7s9 5v2yuyl2Yzh1GNl4AWiIOPx1Hyss3wj3c4ftI3Lvv+rTESHx4NkFaznbsSddZ4rJ2Z2C NzyyspIZYFaWHLFC2KNtvkdAbd5HekE/vOH6TlF+BUxkpMEUJE5D9cWRbcHlOCeP8k5T 8j2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=6ffCzSWxnp7XTAXu7wTtYzf7VFzbZ1BdtrI9e0kd5ig=; b=hBx7+t+7kfB+QSLAw8F/F2eSm+58MXrX6957k0U0riizga9RvQ7HBHZLSwAL7T8zAo XmdqnwXg9CZXrL5ctiyQCsiy/nY9eBJMQ70xDwi9/yJ7fkn/1CaC848BHmqmOLrmz1Ry FWOBWt5gmcKkkeOkX35X3PHY7jKBpQo3bVjx3c49WTholuZsMVnuBIfDaMGJT//33S2f k3cenh4lTchHrU/aoDp/Sd52YmxH1shjtwD4KVcOqxCNNOIrzbk/s5UdL5Ngcc1/vLww gBS3BC50xmsDVRwhVomejfLOzrdNf+7mQtSbA7X5evp3Btu/0toBDzRKP/zsTAzeLg86 T/CQ== X-Gm-Message-State: AKwxytdmWo9yy1rregUaX/nVbNURER2YKQ112tpDi1AXC4bTDWyPoQpz xcjFIdrAJTw2MaX0pYlkpuY= X-Google-Smtp-Source: ACJfBotOFLIIcB/xvEu+gy5tg+TMMDkJS8st8JkN5YBueK0Nu7Z5rkocvgSfr7nAW8i0XE/LTP/NeQ== X-Received: by 10.36.65.88 with SMTP id x85mr12872221ita.108.1516178828677; Wed, 17 Jan 2018 00:47:08 -0800 (PST) Received: from [0.0.0.0] (67.209.179.165.16clouds.com. [67.209.179.165]) by smtp.gmail.com with ESMTPSA id x196sm2162089iod.12.2018.01.17.00.47.02 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 17 Jan 2018 00:47:08 -0800 (PST) To: Thomas Monjalon Cc: dev@dpdk.org, Jerin Jacob , Jianbo Liu , Jan Viktorin , Olivier Matz , konstantin.ananyev@intel.com, hemant.agrawal@nxp.com, Jia He References: <1516161831-28719-1-git-send-email-hejianet@gmail.com> <1516161831-28719-4-git-send-email-hejianet@gmail.com> <2512442.13rg75T8JE@xps> From: Jia He Message-ID: <23810751-921d-4e3a-bac6-95d251cc122f@gmail.com> Date: Wed, 17 Jan 2018 16:47:04 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: <2512442.13rg75T8JE@xps> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v8 3/3] ring: introduce new header file to support C11 memory model X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Jan 2018 08:47:10 -0000 Hi Thomas On 1/17/2018 4:24 PM, Thomas Monjalon Wrote: > 17/01/2018 05:03, Jia He: >> To support C11 memory model barrier, 2 options are suggested by Jerin: >> 1. use rte_smp_rmb >> 2. use load_acquire/store_release(refer to [1]). >> CONFIG_RTE_RING_USE_C11_MEM_MODEL is provided, and by default it is "n" >> on any architectures so far. > In previous patches, it was enabled for ARM. > You decided to not enable it at all? Sorry, maybe I misunderstand your previous mail. >This config option should be added in the common file (as disabled). Do you mean  CONFIG_RTE_RING_USE_C11_MEM_MODEL=n in comm_base and "y" in armv8 config? Cheers, Jia > >> config/common_linuxapp | 2 + > It should be defined in common_base, not common_linuxapp. > >> --- /dev/null >> +++ b/lib/librte_ring/rte_ring_c11_mem.h >> @@ -0,0 +1,193 @@ >> +/*- >> + * Copyright(c) 2017 Intel Corporation. All rights reserved. >> + * All rights reserved. >> + * >> + * SPDX-License-Identifier: BSD-3-Clause >> + */ > It is not complying with the template. > Please check the license/ directory. > > Why is it Intel Copyright? > "All rights reserved" is probably not needed. > >> +/* >> + * Derived from FreeBSD's bufring.h >> + * >> + ************************************************************************** >> + * >> + * Copyright (c) 2007-2009 Kip Macy kmacy@freebsd.org >> + * All rights reserved. >> + * >> + * Redistribution and use in source and binary forms, with or without >> + * modification, are permitted provided that the following conditions are met: >> + * >> + * 1. Redistributions of source code must retain the above copyright notice, >> + * this list of conditions and the following disclaimer. >> + * >> + * 2. The name of Kip Macy nor the names of other >> + * contributors may be used to endorse or promote products derived from >> + * this software without specific prior written permission. >> + * >> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" >> + * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE >> + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE >> + * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE >> + * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR >> + * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF >> + * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS >> + * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN >> + * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) >> + * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE >> + * POSSIBILITY OF SUCH DAMAGE. >> + * >> + ***************************************************************************/ > This double license may be an issue. > Hemant, comment? > -- Cheers, Jia