From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] Update release notes for container power policies Date: Wed, 17 Oct 2018 13:43:21 +0200 Message-ID: <19031454.2TYKyzlM3p@xps> References: <20181017112350.19811-1-david.hunt@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, john.mcnamara@intel.com, marko.kovacevic@intel.com To: David Hunt Return-path: Received: from wout1-smtp.messagingengine.com (wout1-smtp.messagingengine.com [64.147.123.24]) by dpdk.org (Postfix) with ESMTP id 6F7445B1C for ; Wed, 17 Oct 2018 13:43:21 +0200 (CEST) In-Reply-To: <20181017112350.19811-1-david.hunt@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 17/10/2018 13:23, David Hunt: > Signed-off-by: David Hunt > --- > doc/guides/rel_notes/release_18_11.rst | 7 +++++++ > 1 file changed, 7 insertions(+) Why this patch is alone? It is for a change already applied? If you forgot the release notes, you should add a Fixes: line so we know which code change it refers. Thanks > diff --git a/doc/guides/rel_notes/release_18_11.rst b/doc/guides/rel_notes/release_18_11.rst > index 9c00e33cc..3174a0f80 100644 > --- a/doc/guides/rel_notes/release_18_11.rst > +++ b/doc/guides/rel_notes/release_18_11.rst > @@ -88,6 +88,13 @@ New Features > the specified port. The port must be stopped before the command call in order > to reconfigure queues. > > +* **Added JSON power policy interface for containers .** > + > + Extended the Power Library and vm_power_manger sample app to allow power > + policies to be submitted via a FIFO using JSON formatted strings. Previously > + limited to Virtual Machines, this feature extends power policy functionality > + to containers and host applications that need to have their cores frequency > + controlled based on the rules contained in the policy. > > API Changes > ----------- You must keep a double blank line before before next heading. Anyway you should move it before testpmd note, as explained in the comment: Suggested order in release notes items: * Core libs (EAL, mempool, ring, mbuf, buses) * Device abstraction libs and PMDs - ethdev (lib, PMDs) - cryptodev (lib, PMDs) - eventdev (lib, PMDs) - etc * Other libs * Apps, Examples, Tools (if significative)