From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0063.outbound.protection.outlook.com [104.47.0.63]) by dpdk.org (Postfix) with ESMTP id CFF47F60E for ; Sat, 11 Feb 2017 10:49:00 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=lWWk2aZvi1RJAWm8RJybBJMOGPg7oUAZ1vsR8tEimh8=; b=t2uYTqFf4eGCXJjLTbJxUbarIGw1741c5/XZ2v+vtd1uHHyfi1yPkP9UKofKRtJuO+CBO/iH/EUD65yCoun8FbxG6fGyyn1dfWKUUuStkdfZq92fcTWrTwGzwhxK+OS4PMEKtPb0ApVJatJEGG2IBOEMiOJF2FJqdZCDRjfJ/yw= Received: from AM5PR0401MB2514.eurprd04.prod.outlook.com (10.169.244.146) by VI1PR04MB1616.eurprd04.prod.outlook.com (10.164.84.150) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.888.16; Sat, 11 Feb 2017 09:48:59 +0000 Received: from AM5PR0401MB2514.eurprd04.prod.outlook.com ([10.169.244.146]) by AM5PR0401MB2514.eurprd04.prod.outlook.com ([10.169.244.146]) with mapi id 15.01.0888.030; Sat, 11 Feb 2017 09:48:57 +0000 From: Nipun Gupta To: "Van Haaren, Harry" , "dev@dpdk.org" , "jerin.jacob@caviumnetworks.com" CC: Hemant Agrawal , "Richardson, Bruce" , "Eads, Gage" Thread-Topic: [PATCH v2] eventdev: amend comments for events limit and threshold Thread-Index: AQHSg4K2ijftExo5jkqsdvb1vxwXSqFiCqkAgAF7ArA= Date: Sat, 11 Feb 2017 09:48:57 +0000 Message-ID: References: <1486739104-10953-1-git-send-email-nipun.gupta@nxp.com> <1486741801-18258-1-git-send-email-nipun.gupta@nxp.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=nipun.gupta@nxp.com; x-originating-ip: [182.68.60.190] x-microsoft-exchange-diagnostics: 1; VI1PR04MB1616; 7:9tp99ZrJAk4F/mjxtPZ0qmlmG77MwEeqLuGoVFleONlB3X88lo3i4VXeKPoxXbKXIXgrxd3jIhTqMRlkgYLFN1OPiTsqk4/6rYSvp2EUZT1c6dTSU5V9otWn54JH5TWv75O6LC9FJUJ/5D1HCSJ7z7OlhFN05uSi0Yy/YW8NbYHmK37VfKCbrmtNYZ6hFBqXlvCXe/E38IhXSDkzapGUxaqhVwZ7OA8ek8yh7HyOSr54WxWds86U0W5v05IGRMspbFjImSGLa7UW0x6KCs0aPDehPF1L20G34Ros6CGv77m4HYu/HGpOvHUx+foWAlVxB9m4l7dZeWUyEkl4M4SnNlvys8ruJn0mmPNJNhNIfi+NZ1sP7LInjiEnRAncJlny75IibbSJbYrjzND4o0/qMbZKA8Ksh8txf5Quo96uBzV5xkILNJOTBjajVwST8TrbG+RrzhQswlzMGqxLdepb4D95SG2r9kezET6grnX5YxjWSOlnFzrEZDlIIK1u9hXddH8GxCH49DaCEmCL7o9I3g== x-forefront-antispam-report: SFV:SKI; SCL:-1SFV:NSPM; SFS:(10009020)(6009001)(7916002)(39860400002)(39840400002)(39410400002)(39450400003)(39850400002)(199003)(43544003)(76104003)(377454003)(189002)(13464003)(305945005)(53936002)(105586002)(97736004)(77096006)(7736002)(55016002)(2900100001)(99286003)(106356001)(2501003)(5660300001)(106116001)(66066001)(38730400002)(76176999)(54356999)(50986999)(54906002)(81166006)(6506006)(3280700002)(189998001)(7696004)(25786008)(229853002)(68736007)(92566002)(6116002)(102836003)(3846002)(33656002)(4326007)(3660700001)(101416001)(2950100002)(122556002)(74316002)(9686003)(2201001)(2906002)(86362001)(81156014)(8936002)(8676002)(6246003)(6436002); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR04MB1616; H:AM5PR0401MB2514.eurprd04.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; x-ms-office365-filtering-correlation-id: dff6daa6-9ee6-4e6f-7238-08d4526332c0 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081); SRVR:VI1PR04MB1616; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(185117386973197)(228905959029699); x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(6041248)(20161123560025)(20161123562025)(20161123558025)(20161123555025)(20161123564025)(6072148); SRVR:VI1PR04MB1616; BCL:0; PCL:0; RULEID:; SRVR:VI1PR04MB1616; x-forefront-prvs: 0215D7173F received-spf: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Feb 2017 09:48:57.0463 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB1616 Subject: Re: [dpdk-dev] [PATCH v2] eventdev: amend comments for events limit and threshold 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: Sat, 11 Feb 2017 09:49:01 -0000 > -----Original Message----- > From: Van Haaren, Harry [mailto:harry.van.haaren@intel.com] > Sent: Friday, February 10, 2017 16:02 > To: Nipun Gupta ; dev@dpdk.org > Cc: Hemant Agrawal ; > jerin.jacob@caviumnetworks.com; Richardson, Bruce > ; Eads, Gage > Subject: RE: [PATCH v2] eventdev: amend comments for events limit and > threshold >=20 > > From: Nipun Gupta [mailto:nipun.gupta@nxp.com] > > Sent: Friday, February 10, 2017 3:50 PM > > To: dev@dpdk.org > > Cc: hemant.agrawal@nxp.com; jerin.jacob@caviumnetworks.com; > Richardson, Bruce > > ; Eads, Gage ; Van > Haaren, Harry > > ; Nipun Gupta > > Subject: [PATCH v2] eventdev: amend comments for events limit and thres= hold > > > > Updated the comments on 'nb_events_limit' of 'struct rte_event_dev_conf= ig' > > and 'new_event_threshold' of 'struct rte_event_port_conf' for open syst= em > > configuration. > > > > Signed-off-by: Nipun Gupta > > --- > > Changes for v2: > > - Fix errors reported by check-git-log.sh > > > > lib/librte_eventdev/rte_eventdev.h | 12 +++++++----- > > 1 file changed, 7 insertions(+), 5 deletions(-) > > > > diff --git a/lib/librte_eventdev/rte_eventdev.h > b/lib/librte_eventdev/rte_eventdev.h > > index c2f9310..171e52e 100644 > > --- a/lib/librte_eventdev/rte_eventdev.h > > +++ b/lib/librte_eventdev/rte_eventdev.h > > @@ -404,11 +404,12 @@ struct rte_event_dev_config { > > * @see RTE_EVENT_DEV_CFG_PER_DEQUEUE_TIMEOUT > > */ > > int32_t nb_events_limit; > > - /**< Applies to *closed system* event dev only. This field indicates = a > > - * limit to ethdev-like devices to limit the number of events injecte= d > > - * into the system to not overwhelm core-to-core events. > > + /**< In a *closed system* this field indicates a limit to ethdev-like > > + * devices to limit the number of events injected into the system to > > + * not overwhelm core-to-core events. > > * This value cannot exceed the *max_num_events* which previously > > - * provided in rte_event_dev_info_get() > > + * provided in rte_event_dev_info_get(). > > + * This should be set to '-1' for *open system*. >=20 >=20 > I don't think we should mention ethdev explicitly here - it applies to an= y > port that is attempting to enqueue work into a closed-system eventdev. >=20 > What do you think of the following wording? (Suggestion only, feel free t= o > re-word if required). >=20 > /**< In a closed system this field is the limit on the maximum number of = events > that can be inflight in the eventdev at a given time. The limit is r= equired > to ensure that the finite space in a closed system is not overwhelme= d. The > value cannot exceed the *max_num_events* as provided by > rte_event_dev_info_get(). > This value should be set to -1 for open systems. > */ I agree with you Harry. For *closed systems*, this limit should be valid on= all the enqueues, whether from ethdev type devices or enqueue's from core. BTW, do you use event limit both on event device and on event ports in your software implementation? Because I think the limit may be only per event po= rt. Jerin, Your views on this for open systems would be very helpful. Thanks. >=20 > > */ > > uint8_t nb_event_queues; > > /**< Number of event queues to configure on this device. > > @@ -633,7 +634,8 @@ struct rte_event_port_conf { > > * can have a lower threshold so as not to overwhelm the device, > > * while ports used for worker pools can have a higher threshold. > > * This value cannot exceed the *nb_events_limit* > > - * which previously supplied to rte_event_dev_configure() > > + * which previously supplied to rte_event_dev_configure(). > > + * This should be set to '-1' for *open system*. > > */ >=20 > Minor grammer issue (that was previously there too, but worth fixing anyw= ay), > there is a _was_ missing from the sentence: Sure. I'll take care of this in v3. >=20 > + which was previously supplied to rte_event_dev_configure(). >=20