From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id DAC3BC1C0 for ; Thu, 18 Feb 2016 05:06:57 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP; 17 Feb 2016 20:06:42 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,463,1449561600"; d="scan'208";a="905722659" Received: from adambynes.sh.intel.com (HELO adambynes) ([10.239.129.240]) by fmsmga001.fm.intel.com with ESMTP; 17 Feb 2016 20:06:35 -0800 Date: Thu, 18 Feb 2016 12:06:20 +0800 From: Zhe Tao To: David Marchand Message-ID: <20160218040620.GA17470@intel.com> References: <1452688307-20213-1-git-send-email-jingjing.wu@intel.com> <1453859378-23912-1-git-send-email-jingjing.wu@intel.com> <1453859378-23912-3-git-send-email-jingjing.wu@intel.com> <9BB6961774997848B5B42BEC655768F8D98B60@SHSMSX104.ccr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v2 2/2] i40evf: support interrupt based pf reset request X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Feb 2016 04:06:58 -0000 On Mon, Feb 15, 2016 at 02:16:16PM +0100, David Marchand wrote: Hello, > > On Sun, Feb 14, 2016 at 4:25 AM, Wu, Jingjing wrote: > >> -----Original Message----- > >> From: David Marchand [mailto:david.marchand@6wind.com] > >> Having this infrastructure is one thing, but the initial problem was that the > >> driver did not recover from this reset event. > >> The linux i40e vf driver handles this kind of event itself. > >> Could we have something similar ? > >> > > > > Considering about the how to use DPDK PMD, and how to setup resource, we can > > know that lots of resources are managed by application. I think based on current > > PMD driver framework, driver cannot reset without application's help. > > I reported an issue on ixgbe. > What you provide here is a workaround for i40e. > I am not even sure this can be applied to ixgbe. > > Does it mean that anytime we have a problem with drivers, workarounds > should be applied to ethdev / eal ... so that you don't have to handle > anything in the drivers ? I think this patch provides a necessary framework for i40e VF to handle the asynchronous event, maybe Jingjing can help to change the description of this patch to let it does not limit to the VF reset event. > This is not the first time I complain about this kind of design issues. > > > > If we need to support driver recovery automatically, we'd better to find a way to do that. > > Do you have any idea? > > First, list those "lots of resources" that "are managed by application". > If your driver needs to keep track of those, this is i40e driver job > to do this internally without requiring ethdev to be modified. > > If this proves to be generic enough, maybe moving part of this to > ethdev will then make sense. > > > Thanks. > > -- > David Marchand Thanks, Zhe Tao