From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] app/testpmd: fix timeout in Rx queue flushing Date: Fri, 23 Sep 2016 20:20:45 +0200 Message-ID: <9780401.L0I3dXZThK@xps13> References: <1473328156-32488-1-git-send-email-james.g.poole@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Ferruh Yigit , pablo.de.lara.guarch@intel.com To: James Poole Return-path: Received: from mail-wm0-f42.google.com (mail-wm0-f42.google.com [74.125.82.42]) by dpdk.org (Postfix) with ESMTP id 2B3CE231C for ; Fri, 23 Sep 2016 20:20:51 +0200 (CEST) Received: by mail-wm0-f42.google.com with SMTP id b130so44394156wmc.0 for ; Fri, 23 Sep 2016 11:20:51 -0700 (PDT) In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-09-13 17:04, Ferruh Yigit: > On 9/8/2016 10:49 AM, James Poole wrote: > > When testpmd is run, the application would hang on the second time > > that "start" is executed. This is because the timer limit would get > > multiplied to an unreachably high number. > > > > At the start of flush_fwd_rx_queues(), the timer limit now resets > > to stop it from getting to this high number. > > > > The timer has been made local for this function. > > > > Fixes: f487715f36f5 ("app/testpmd: add timeout in Rx queue flushing") > > > > Signed-off-by: James Poole > > Acked-by: Applied, thanks