From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id A3BDD1B24E for ; Tue, 31 Oct 2017 01:12:40 +0100 (CET) Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Oct 2017 17:12:38 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,321,1505804400"; d="scan'208";a="169202489" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.241.225.254]) ([10.241.225.254]) by fmsmga005.fm.intel.com with ESMTP; 30 Oct 2017 17:12:38 -0700 To: xiangxia.m.yue@gmail.com, dev@dpdk.org Cc: Tonghao Zhang References: <1509026701-17478-1-git-send-email-xiangxia.m.yue@gmail.com> From: Ferruh Yigit Message-ID: <9e5c260f-20db-38f9-c92f-37aaed168b73@intel.com> Date: Mon, 30 Oct 2017 17:12:38 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <1509026701-17478-1-git-send-email-xiangxia.m.yue@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v2 1/4] net/ixgbevf: unregister irq handler when other interrupts 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, 31 Oct 2017 00:12:41 -0000 On 10/26/2017 7:04 AM, xiangxia.m.yue@gmail.com wrote: > From: Tonghao Zhang > > When we bind the ixgbe VF (e.g 82599 card) to igb_uio and enable the > rx-interrupt, there will be more than one epoll_wait on intr_handle.fd. > One is in "eal-intr-thread" thread, and the others are in the thread > which call the "rte_epoll_wait". The problem is that sometiems > "eal-intr-thread" thread will process the rx interrupt, and rte_epoll_wait > can’t get the event any more, and the packets may be lost. > > We should unregister the status interrupt handler in "eal-intr-thread" > thread and the ixgbe is in the same case. > > Signed-off-by: Tonghao Zhang Thanks for the patches! We are very close to 17.11 release and only fix patches are considered at this phase, this patchset will be evaluated for next release. Thanks, ferruh