From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf0-f172.google.com (mail-pf0-f172.google.com [209.85.192.172]) by dpdk.org (Postfix) with ESMTP id 413038BA7 for ; Thu, 26 Apr 2018 17:37:23 +0200 (CEST) Received: by mail-pf0-f172.google.com with SMTP id j5so18689660pfh.2 for ; Thu, 26 Apr 2018 08:37:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=oE80QiqKsHhuU39H5PdkMLuSyOqY75FtY/rExSS6Qek=; b=f6J7jG5jZ3goJGT0o5JXR4vUYaHk1WZH8FR8j5ZAqU31JRLwDjfe7D4o4+If6y8Noz AaiPG9rTNAZ9ocfUnquWteS2lsviJHJULYPMCLk/XtFlrmePONbT8n0IJY06kDLi3rau 0RwE5/wODOPddHK75lyS6gimwCi4sBFxMhfej3ffkbjQxmbt4PdxJEUph2e2e3RLMOCa XCOA9Ko0WED/qzpOnXUovzMhnT39pTNXGuqpFlyvuBNoFNVgf7xwtky6mYD/7KF62fYg Xib+bPx14vqLjKqLIfUAK2jRl67VJ61dOHs4YDgqpoOdFTaMDdYb3507Zq60eWhz+Bsa ENiw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=oE80QiqKsHhuU39H5PdkMLuSyOqY75FtY/rExSS6Qek=; b=PQD3DrLj7ANmlUEPPvlF44WpJ7KioC3oHl9LGzYIBfj1BjogAhNZaCZWqMU8hafrxD bloT3b7b6S2Ea6CzGU1FTsU+s5/Tft72k0kw5S0VJP5qVwdwyLDxZgOuyHIlDA2/CB2z RrWfGL/ZHoCkD6XnHKItVGtwJNtHGRwRVU43cK9nsX5XGD4UvOjQcbquSjQRt2n28Tjy MJ+29jle/ud2ddxnjXPVU1PpM45LFWunwCG04Kx+8fTvV7Ha9hTvT83BHj8RjfI7eh3k t3jzAMBKB3oZ/asi0i1rTRkhpFEt4AypaDiQAut4+Ec3sdkHRE/JcXGetCe9oQjmYYYi OBcg== X-Gm-Message-State: ALQs6tDleYFriVkwksubGPz2mJYFpfm2KMEin8MM9MLJIhaAXuN0GlO7 pQOmM2umvkmTPWr+BKjQOoxo4Q== X-Google-Smtp-Source: AIpwx4/VLM9tuKlKzrpZ4W+3eqr2wZLTtjSJ2G6fTUNK6YOs8bckTR4rar5aNiBn5ytZyyDR71eGpA== X-Received: by 10.98.163.145 with SMTP id q17mr33232854pfl.87.1524757042369; Thu, 26 Apr 2018 08:37:22 -0700 (PDT) Received: from xeon-e3 (204-195-71-95.wavecable.com. [204.195.71.95]) by smtp.gmail.com with ESMTPSA id p13sm39205687pgn.18.2018.04.26.08.37.21 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 26 Apr 2018 08:37:22 -0700 (PDT) Date: Thu, 26 Apr 2018 08:37:19 -0700 From: Stephen Hemminger To: "Carrillo, Erik G" Cc: Thomas Monjalon , "Burakov, Anatoly" , "dev@dpdk.org" , Neil Horman , "Mcnamara, John" , "Kovacevic, Marko" , Robert Sanford , "olivier.matz@6wind.com" , "shreyansh.jain@nxp.com" Message-ID: <20180426083719.599c15a2@xeon-e3> In-Reply-To: References: <2584582.pWCQVUZt55@xps> <5414097.qiRGovPonW@xps> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH] lcore: make semantics of lcore role function more intuitive 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: Thu, 26 Apr 2018 15:37:23 -0000 On Thu, 26 Apr 2018 14:56:19 +0000 "Carrillo, Erik G" wrote: > Great, thanks Thomas. > > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas@monjalon.net] > > Sent: Thursday, April 26, 2018 9:55 AM > > To: Carrillo, Erik G > > Cc: Burakov, Anatoly ; dev@dpdk.org; Neil > > Horman ; Mcnamara, John > > ; Kovacevic, Marko > > ; Robert Sanford ; > > olivier.matz@6wind.com; shreyansh.jain@nxp.com > > Subject: Re: [dpdk-dev] [PATCH] lcore: make semantics of lcore role function > > more intuitive > > > > 26/04/2018 16:44, Carrillo, Erik G: > > > Thanks, Anatoly and Thomas. I had also considered the following chunk for > > the release notes: > > > > > > diff --git a/doc/guides/rel_notes/release_18_05.rst > > b/doc/guides/rel_notes/release_18_05.rst > > > index 04ff4fe..127a7e2 100644 > > > --- a/doc/guides/rel_notes/release_18_05.rst > > > +++ b/doc/guides/rel_notes/release_18_05.rst > > > @@ -72,6 +72,11 @@ API Changes > > > Also, make sure to start the actual text at the margin. > > > > > ========================================================= > > > > > > +* **rte_lcore_has_role() return values changed** > > > + > > > + This function now returns 1 or 0 for true or false, respectively, rather > > > + than 0 or <0 for success or failure to make use of the function more > > > + intuitive. > > > > > > ABI Changes > > > ----------- > > > > > > Do we want this note? Also, it looks like the Doxygen documentation of > > the function in the header file didn't get updated. > > > > > > Oh, you are right, this patch is not complete. > > I've fixed it: > > > > --- a/doc/guides/rel_notes/release_18_05.rst > > +++ b/doc/guides/rel_notes/release_18_05.rst > > @@ -161,6 +161,12 @@ API Changes > > announced at least one release before the ABI change is made. There are > > no > > ABI breaking changes planned. > > > > +* eal: ``rte_lcore_has_role()`` return value changed. > > + > > + This function now returns true or false, respectively, rather than 0 > > + or <0 for success or failure. > > + It makes use of the function more intuitive. > > + > > * mempool: capability flags and related functions have been removed. > > > > Flags ``MEMPOOL_F_CAPA_PHYS_CONTIG`` and diff --git > > a/lib/librte_eal/common/include/rte_lcore.h > > b/lib/librte_eal/common/include/rte_lcore.h > > index 334a0629e..1a2f37eaa 100644 > > --- a/lib/librte_eal/common/include/rte_lcore.h > > +++ b/lib/librte_eal/common/include/rte_lcore.h > > @@ -311,7 +311,7 @@ rte_ctrl_thread_create(pthread_t *thread, const char > > *name, > > * @param role > > * The role to be checked against. > > * @return > > - * On success, return 0; otherwise return a negative value. > > + * Boolean value: positive if test is true; otherwise returns 0. > > */ > > > > Thanks Erik! > > > Usually the safest way to introduce this is introduce a new function with a different name. Then retire the old one.