From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-DM3-obe.outbound.protection.outlook.com (mail-dm3nam03on0049.outbound.protection.outlook.com [104.47.41.49]) by dpdk.org (Postfix) with ESMTP id D441C36E for ; Mon, 6 Mar 2017 14:45:45 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=QCPMRimwSgc1H6tGdKK7ArSESl7ppozsXJztcUByD5Q=; b=OPLNLpSDl/HFxiB4UtGVYoDHkG3ysteNoSaomuK7tiBXFnNOJQ0SGAsDdrgDwZTcNKSyATnsOh/M4fGopbbNN0nWlr/yK/oBzfAGhjpuHzxet7Xali3KEulGQOzrVsfvckOyyv9dueiRBSWrSJt29RsIzlbyl5CULJMwrZ0/eZ0= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=caviumnetworks.com; Received: from localhost.localdomain (106.200.230.99) by BLUPR0701MB1713.namprd07.prod.outlook.com (10.163.85.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.947.12; Mon, 6 Mar 2017 13:45:42 +0000 Date: Mon, 6 Mar 2017 19:15:20 +0530 From: Jerin Jacob To: Harry van Haaren Cc: dev@dpdk.org Message-ID: <20170306134519.GA7397@localhost.localdomain> References: <1488372542-156763-1-git-send-email-harry.van.haaren@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1488372542-156763-1-git-send-email-harry.van.haaren@intel.com> User-Agent: Mutt/1.7.1 (2016-10-04) X-Originating-IP: [106.200.230.99] X-ClientProxiedBy: MA1PR01CA0104.INDPRD01.PROD.OUTLOOK.COM (10.174.56.148) To BLUPR0701MB1713.namprd07.prod.outlook.com (10.163.85.14) X-MS-Office365-Filtering-Correlation-Id: e40097b3-91c9-4580-d18e-08d464971617 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BLUPR0701MB1713; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 3:JnrG5rthnS6f6Z/vQPQ9FinNb4bD1FrKR0gTyTNLEQtFdlj0owyVnNpv6G4e7VEnTQt0/WhSc2LmWurwn7HQluw+Y7DiPhN5YEiW2I2BmqeJKvTpZ+nc5rj1bjFykbappazTNw8wGI0z1ZrvhJNwetsX0oBuBvOsd5r9MNEL05sc4mkcE6+i7WgbkTskncKFnhFXgHfjBdDpICJSZMiJcT5PV59toYTJ7C0eTD9f/4Bx+ce3ixTIZAjSzm+WMEQYvxKY+kLNhn0wgtwH9wgbyw==; 25:/G5FVnbCn8jHPNoLfh11z0eutUUnxKeYW3HPZuBzD5qaFYV1BZjtgrV9cwUaQPOSUFBgveN+Ig/y/MOKmJRrfc14bFZPtXAXcLo7TbCqpFF8P3uhCzPtlFVtfOKHitPllGmc9jLuUC6YNhQhjs0y7ukiZZ6jQ3q0+/beXT9heCoSAPkhGFNE/WnAN+0Mf8NWadsZz25S/RArp01+b+c0HVUcRmUwIhj676Ta78/3CaV8qXD55FUSJd/1lPvovHJKbVacUoggA9rRrLuDACyyCvxLh4LqBA5+cITegJBd4Z+/X8zJ16qVNhztTL+v5oXvHpBGLX+Q2PHRr7QCEY82lMKghgK77+6tqn1xG9aHxTO3hyNNNealpNuwWBNYG65kwPb1y2ZYMvYlneCNMcQqxyDDHg3799LFtsEhHLrNyBtz8e7wAq01JcT57FAfCFXifivpGcLKrll2jIxqseDRfg== X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 31:kyAWZ2n5co/pku20IeSLVW/wOZ3eWJBKs6B728tS/xfp5pGZZ8GMopR/hUZblPqI66cnncpdwdafykaCCZfIsMtIIfH5Ept0qH5HS7nuqj53/Okb88lE36BaWyTnYVNQhO0YpOxIzhjTwGIAMfcALZ8FmGPdY8D7lIgib1/diiOZaVgLDs7GltK42/qK+R724bRbUOrwRhCU6ZBDsfjpJuQKIoaXiOJ4mZPH7sYAR+bs9lH/qIWjbtqxKnFFVfpT091FIsTwhP0pDwuVwxBJgQ==; 20:AfCW69EHW7ktVpgFdhL3wuF3R1wal6ENlzHmcE2qCfp29ETmeOqZzjQfqtln5665ReJ+5+pOdcB9BVmLrS3P2cplmwgduBriYfwR/KDKuGqoq0m28sUgsEmUEPCXtJSuIVhtlJ7763e4Ls5AFNTvS0V/2UUpeTcNIJPDaKAhLs8yd3zwauzqKvCWOdbOMc2xXiwF/b7Sg05HwbArUmP+EWC2jysyhRpghkCii5kHCZhyQp40A4h++O15XMC+JiQItQ2QhQdD8ezRsabmWgFjbQVydc7WjFknkhFy3BM2eEAF8TqpMBGbKodRAiS73EwP7eXR5T9qDGplolrUAbihW6AgBgcXgX2G40YyqhH5/JGkyhHkzjTjxnuOq4Xs2FL5rqfueT5pmKwf3o/hAPy1hQUVP9IgksVMWM2H3s3gSHFAp6bsIaBFkEeoPnEFD1LhSPvhu67cg+4IkBPNnxyf9/p53ztFo+9mOL1LQgw9wZuGemj4PEoxWMMl3OqfMrJ7tLaz7VWYqEiLRTqyp2upZ4MUO1WKqOrf1o+NQqYtZVG4WLG+jYDGOeWzIY9Ec7XWh/jJ1iFttm4HU34/MMEhVic5h1Ddr3aZyWBeltj8D2M= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(278428928389397)(228905959029699); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041248)(20161123564025)(20161123558025)(20161123555025)(20161123562025)(20161123560025)(6072148); SRVR:BLUPR0701MB1713; BCL:0; PCL:0; RULEID:; SRVR:BLUPR0701MB1713; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 4:N87tITYUMYbcFsvVPuj+bIS8VGdt8/82UZrDJvFVRG4dLtCfJ4MUYzesFIP+9+rMkwwHRMkYd2KLtBEB/2OXj/xajrRlZ9P/5jg2iGTQvryEJp4Vzk3JpjFOeAmjnfrzdEeJxasSfshVH6Yf4WZxHhl9yQYA4wQWwC901wpDjskOm9mcVjDi/oxKZyfdpBiQfA8PUSLOsaU4guG6Qhe+iaBUnG0CCL0uO6U5ujkbxOQSxgzSxN2HztdkVbelC5K/R8tMXRTC8v0zaunW8gZqj3lew9RwDZOukKyEPzvf0VNkBXNyld0RA8CQJlhKjzvvgn3EP+ml9PfaAg+plBHT31783pbY9GscJkZlmH5e+uNx6EuM2YRlYD0ZkfQt4a9JF45hpI49NFu+MWkSVIndV22ydIa01mOt03Gd4xdQjTbpqiUNSmIJ/zgGUOnVoq3c/9nmkjzJnBp5VdbzDYPFRx4ZHNJ5c0Ee9NKLBJY12BJicGNnzlQnUBOW7gxWuzUqU3FgPPJ3fV99FxGYoGrmuw1usV/Pnm2Uk+DzSG9cR/ZHgst9O9yRN6aNRa0RWAz6VAvr5fl/KcZuMUicqLXgazhfZVAyY1Bg5NeKCVvshxjeKKyB0Pn5BhAF/YKggufVYmwIInqOl7L46fsVA9Rkc/MvpFB/a07WTCFbbIrB+5yUl+yWOjHc2hPPP1yJZsnx X-Forefront-PRVS: 0238AEEDB0 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(7916002)(39450400003)(24454002)(8676002)(81166006)(305945005)(3846002)(6116002)(7736002)(23726003)(5009440100003)(50986999)(76176999)(54356999)(5660300001)(42186005)(4001350100001)(6916009)(42882006)(6666003)(2950100002)(1076002)(47776003)(66066001)(189998001)(2906002)(50466002)(33656002)(55016002)(53936002)(92566002)(229853002)(4326008)(6506006)(110136004)(38730400002)(9686003)(25786008)(83506001)(6246003)(61506002)(18370500001); DIR:OUT; SFP:1101; SCL:1; SRVR:BLUPR0701MB1713; H:localhost.localdomain; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BLUPR0701MB1713; 23:WE2Y+KP3+EazxIJ4kH7sikdj8y6M7HODcE6rfVY?= =?us-ascii?Q?vDdyoua4+FGUKG+ZwTAwD5VTbKz2nTBfbr0bR44yqMX6YJCcBYfa0NDXNlaC?= =?us-ascii?Q?qusWGOIi/PZY4hVq9Dv8o+Om8p2FXPFd8unVEV4G2Mr7dQVc7ck0rAUfmnZe?= =?us-ascii?Q?dTItscepngL/86a0LiAXJ+1x1qo3tym9KC3gMqjbIB0xzgnh4MtsAC0CF/90?= =?us-ascii?Q?hPqnuUlz3kuYK5alfb+wcxhBL802C2m5WahCDwhPI8QFeXdmSdBN8bGvp65h?= =?us-ascii?Q?QTAK+/Cqcq4mbdIyD/aORdteHBBZNZNNM264SEgXY5z4iLv3sV8QFTd6lNXv?= =?us-ascii?Q?yHKywOAaDz2Q3QgPAyBZqdugz4utjxi9MYGnlSW4b1q3RBZlUgFZtzX/9jLG?= =?us-ascii?Q?U8qpnR2kkasCEXYYN+W6JdUY4rEPYXVxfKQPbPPlPMzcHC7OGwcMV0pcNBVP?= =?us-ascii?Q?8eNsRexUTddicbvZIWbHgJs4T0ugWeaEbgoPG9qfcoubOGgVPFSzhfrecojr?= =?us-ascii?Q?9HEDeSCRWmvkzHv5PDs/V+AavPd08tUh5wEjKCpHo2fiL1Kxqiy+vLujwgAW?= =?us-ascii?Q?LYWyGAWmuEjF6Sz8k8FrYnpLFeJFGVpRatZB7waEdKepKYkKZ9L+4VO2uzpB?= =?us-ascii?Q?dCq2GrlZ/osx0P8QREfmC4fXkDjVDRhq9K7tjoHSm/O89H929LyZPig3CjDH?= =?us-ascii?Q?NVVojXj8BCzm3TZmddNkb3w0COEvyqXggyiwK9Bn7ogRMP1duKApt0Ze7EkI?= =?us-ascii?Q?mgtWQgqxdLMC5Z98ZOtdYeGG8WEFrQOI9XBj+ljkPX7msyU2hEBHKDwnP1Zr?= =?us-ascii?Q?FSLYJ0s6XTh1vfX6/Y+utf5i6zpVXkgqZV+UQZ1GHkEwXhDR0e8e0JC8l3wC?= =?us-ascii?Q?u4Sc8njwonA943I5naI7ayzTl0bPGe69iWlmJx5eVZky9WcJ+yTHv+D9AilH?= =?us-ascii?Q?3/v0k/PJVLQoprlAfa744vd8k37WAkLid0AfDtavV9G+ktsJ2d8If+kKbEQ1?= =?us-ascii?Q?Q2rMomFXSQZpfNXhlgU/l7K+vQbH0Tr/llJKz5vHTCN6eZggCBCYa84rwiOD?= =?us-ascii?Q?Ondr6vegVukmSB9fZs4VaMWDLZbRYN+lO8CulAynWdvDp0sUbOw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 6:9LO5O5R+L3dqzbnwgFsFYxNGLsNV1qLhBmm3oQzXGQfWapHgR7JPn7LAB3fIq8dMM6f6CoQTvxk9quHT0qaEi/ZjGJOzCm8sXL7PHu9Xsiern30GoXGks5Se2vWH5i/ktJQWWyvwbRarvTowmde3lqM2tGPp3497csxcQwF7dp3XdcXCngPeNe82rrqpRcrbr4Rz2rBTP1eAd+2AoL//afKVKooHoKcN8w7JVNb0+wQeqSSXyqh/K/cX23k9AlqDpHjlcXwbQS+b7jTvLOMPsXOtOi0kSM0h5h/9Lax34v7I72Y1g6vzTL3nYZryaAz55rqrOl/O4em7J1lftD8EuVIYHnV3YKFSAD/KRHt98srIm6xvefd28aHDSXkIhRucP4H5p9H8t6s8So7D8koIrg==; 5:hS64QEOjOb2N8Dn+8LvimAp5QZVh8KaQTmM8SIQi8QTwkvgmUIBGZ0U77RR3SLgmWEMnWkjQ8jk0EJN/R8p+N3npap3uXaui2EFmaH3l5AQa1c3dDzGi0eu9PaBZdoWmUrlBABzBhlN14ftV7MjwYg==; 24:6tbwfS/fmm+Syg2Zj4IuYPT7B1etn9qWyDb+r6P1zdULoszhUGPaH9+cr4IMFp9ijMsREARdsGwUwsd0ESejcGybnxQo5kkBP5A6SjHBXgE= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1713; 7:eMUe5js0DCm+f3opGiibyq5mOA5NF3mHXDhnS1n7q8nr6R0El5aEbtXorwnQDqXGHYThJyEtYES9aYd/3n4YZIteTZp9lHVSsJZDlW12GWBdb4pClHdsJzHgq7J1NCxzEpsvqjJmaXGrK806/zTryjNbUvBcjtleM7jaAML7xrWEKxUANZAO16bJCe97bTnicOeKUaHBWRINW9SxSs/0Lz+HVxlt3OjrNvlHfagwT2J8nhAzd1C1cowyNKUHsgPgb/UIcWGOX+LReQ32CMkvZSUIT3gMtdPPVFLyOoj3QSxmkx50Pg63swJsbzeJp8GOuaSKIH5Q9KRhSDFUKRPmfg== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Mar 2017 13:45:42.0674 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR0701MB1713 Subject: Re: [dpdk-dev] [PATCH] eventdev: remove default queue overriding 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: Mon, 06 Mar 2017 13:45:46 -0000 On Wed, Mar 01, 2017 at 12:49:02PM +0000, Harry van Haaren wrote: > PMDs that only do a specific type of scheduling cannot provide > CFG_ALL_TYPES, so the Eventdev infrastructure should not demand > that every PMD supports CFG_ALL_TYPES. Sure. Then I think then we can enumerate CFG_ALL_TYPES as capability. Meaning, New flag in event_dev_cap to denote PMD can support all the sched type per queue. My reasoning for the capability flag is because, The application flow will be depended on the logic of creating the queue with different flags. Thoughts ? I thought, In SW implementation, We can create 3 virtual queues per queue. Based on en-queue's shed_type, implementation can choose the correct underneath virtual queue. I guess, it has performance issues, if so, Maybe capability is the way forward. Setting to CFG_ALL_TYPES will be useful for flow based event pipeling as I mentioned earlier in other email. > By not overriding the default configuration of the queue as > suggested by the PMD, the eventdev_common unit tests can pass > on all PMDs, regardless of thier capabilities. Make sense. We can remove the default as CFG_ALL_TYPES. > > RTE_EVENT_QUEUE_CFG_DEFAULT is no longer used by the eventdev layer > it can be removed now. Applications should use CFG_ALL_TYPES > if they require enqueue of all types a queue. > > Signed-off-by: Harry van Haaren > --- > lib/librte_eventdev/rte_eventdev.c | 1 - > lib/librte_eventdev/rte_eventdev.h | 6 ------ > 2 files changed, 7 deletions(-) > > diff --git a/lib/librte_eventdev/rte_eventdev.c b/lib/librte_eventdev/rte_eventdev.c > index 68bfc3b..c32a776 100644 > --- a/lib/librte_eventdev/rte_eventdev.c > +++ b/lib/librte_eventdev/rte_eventdev.c > @@ -593,7 +593,6 @@ rte_event_queue_setup(uint8_t dev_id, uint8_t queue_id, > RTE_FUNC_PTR_OR_ERR_RET(*dev->dev_ops->queue_def_conf, > -ENOTSUP); > (*dev->dev_ops->queue_def_conf)(dev, queue_id, &def_conf); > - def_conf.event_queue_cfg = RTE_EVENT_QUEUE_CFG_DEFAULT; > queue_conf = &def_conf; > } > > diff --git a/lib/librte_eventdev/rte_eventdev.h b/lib/librte_eventdev/rte_eventdev.h > index 7073987..d836f61 100644 > --- a/lib/librte_eventdev/rte_eventdev.h > +++ b/lib/librte_eventdev/rte_eventdev.h > @@ -471,12 +471,6 @@ rte_event_dev_configure(uint8_t dev_id, > /* Event queue specific APIs */ > > /* Event queue configuration bitmap flags */ > -#define RTE_EVENT_QUEUE_CFG_DEFAULT (0) > -/**< Default value of *event_queue_cfg* when rte_event_queue_setup() invoked > - * with queue_conf == NULL > - * > - * @see rte_event_queue_setup() > - */ > #define RTE_EVENT_QUEUE_CFG_TYPE_MASK (3ULL << 0) > /**< Mask for event queue schedule type configuration request */ > #define RTE_EVENT_QUEUE_CFG_ALL_TYPES (0ULL << 0) > -- > 2.7.4 >