From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] mk: using initial-exec model for thread local variable Date: Fri, 18 May 2018 11:46:57 +0200 Message-ID: <2401955.4lgGG0bg7M@xps> References: <1524909247-6581-1-git-send-email-yong.liu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, "Wang, Zhihong" To: "Yang, Zhiyong" , "Liu, Yong" Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 1A9C43989 for ; Fri, 18 May 2018 11:47:00 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 28/04/2018 06:39, Yang, Zhiyong: > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Marvin Liu > Sent: Saturday, April 28, 2018 5:54 PM > > > > When building share library, thread-local storage model will be changed to > > global-dynamic. It will cost additional protect for read thread local variable. By > > now only lcore id is this kind of varaible and not need to dynamic share with > > other threads. So make TLS model back to initial-exec like static library for > > better performance. > > > > Signed-off-by: Marvin Liu > > > > For vhost-user, > If no this pach, vhost user in shared lib perf drops 14.3% than working in static. > after applying the patch , vhost-user in shared lib can achieve the similar perf as in static lib. > > Tested-by: Zhiyong Yang For the record, I have decided to not try this optimization in the last weeks of the 18.05 release. However we could try it in 18.08 by applying the patch early in the cycle. Is there anyone against this patch?