From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Elo, Matias (Nokia - FI/Espoo)" Subject: Re: eventdev: method for finding out unlink status Date: Wed, 5 Sep 2018 07:49:13 +0000 Message-ID: <285D18AA-EC23-4C04-824A-726F20681C61@nokia.com> References: <20180730142614.GA11265@jerin> <6D43DE84-583D-42E5-B298-0E7BDA0C17FB@nokia.com> <20180731083107.GA23233@jerin> <4C54AAAE-A872-47E1-B815-AF68965F9F3E@nokia.com> <20180809141814.GA15603@jerin> <4C66F1B2-02F0-4F6E-A6CD-6A3F44CDAB42@nokia.com> <20180810145209.GA2475@jerin> <20180810173510.GA6835@jerin> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Cc: Jerin Jacob , "dev@dpdk.org" To: "Van Haaren, Harry" Return-path: Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-eopbgr00137.outbound.protection.outlook.com [40.107.0.137]) by dpdk.org (Postfix) with ESMTP id A87E8F11 for ; Wed, 5 Sep 2018 09:49:15 +0200 (CEST) In-Reply-To: <20180810173510.GA6835@jerin> Content-Language: en-US Content-ID: <83921600B182E04DB24C1E40F9054D2E@eurprd07.prod.outlook.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" >>=20 >> I'm not sure I understand the issue here. >> Is anybody suggesting to make unlink() blocking? >>=20 >> For certain PMDs, perhaps it must be a synchronous handled unlink(). >> For other PMDs (eg event/sw) there are multiple threads involved, >> so it must be async. Hence, APIs should be async to avoid blocking the c= aller. >>=20 >> With an async API, if you don't want the async behaviuor, it is >> easy to build the sync version: call it in a loop, optionally with a del= ay(). >=20 > Correct. My point was, rte_event_port_unlink() can be blocking as it=20 > is a slow path API(does not really matter how long it waits). > If you think, it can be called in fastpath and/or application can > leverage some cpu cycles on completing the async call then you can add > at the cost of new API unlinks_in_progress() and make sure to update the = documentation > about unlink() that it can be async call(currently it is documented as a = sync > call). Did you come to an agreement how to solve this issue? Any solution (e.g. ma= ke rte_event_port_unlink() blocking with SW eventdev) would be welcomed since = this issue is currently blocking my work with eventdev.