From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id C2D641B968 for ; Tue, 10 Apr 2018 15:53:28 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Apr 2018 06:53:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,432,1517904000"; d="scan'208";a="219258207" Received: from tanjianf-mobl.ccr.corp.intel.com (HELO [10.255.30.205]) ([10.255.30.205]) by fmsmga005.fm.intel.com with ESMTP; 10 Apr 2018 06:53:26 -0700 To: Anatoly Burakov , dev@dpdk.org References: From: "Tan, Jianfeng" Message-ID: Date: Tue, 10 Apr 2018 21:53:26 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH] eal/ipc: stop async IPC loop on callback request 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: Tue, 10 Apr 2018 13:53:29 -0000 On 4/10/2018 6:03 PM, Anatoly Burakov wrote: > EAL did not stop processing further asynchronous requests on > encountering a request that should trigger the callback. This > resulted in erasing valid requests but not triggering them. That means one wakeup could process multiple replies, and following process_async_request() will erase some valid requests? > > Fix this by stopping the loop once we have a request that we > can trigger. Also, remove unnecessary check for trigger > request being NULL. > > Fixes: f05e26051c15 ("eal: add IPC asynchronous request") > Cc: anatoly.burakov@intel.com > > Signed-off-by: Anatoly Burakov > --- > lib/librte_eal/common/eal_common_proc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/librte_eal/common/eal_common_proc.c b/lib/librte_eal/common/eal_common_proc.c > index f98622f..1ea3b58 100644 > --- a/lib/librte_eal/common/eal_common_proc.c > +++ b/lib/librte_eal/common/eal_common_proc.c > @@ -510,11 +510,11 @@ async_reply_handle(void *arg __rte_unused) > TAILQ_REMOVE(&pending_requests.requests, > sr, next); > free(sr); > - } else if (action == ACTION_TRIGGER && > - trigger == NULL) { > + } else if (action == ACTION_TRIGGER) { > TAILQ_REMOVE(&pending_requests.requests, > sr, next); > trigger = sr; > + break; If I understand it correctly above, break here, we will trigger an async action, and then go back to sleep with some ready requests not handled? Seems that we shall unlock, process, and lock here. Right? Thanks, Jianfeng > } > } > }