From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id B593647CE for ; Thu, 11 May 2017 19:00:09 +0200 (CEST) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 May 2017 10:00:08 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,325,1491289200"; d="scan'208";a="98804334" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.252.13.2]) ([10.252.13.2]) by orsmga005.jf.intel.com with ESMTP; 11 May 2017 10:00:07 -0700 To: Ajit Khaparde , Stephen Hurd Cc: dev@dpdk.org References: <20170328034903.41482-1-ajit.khaparde@broadcom.com> <20170328034903.41482-6-ajit.khaparde@broadcom.com> From: Ferruh Yigit Message-ID: <6a55004d-4787-c6b0-27db-b9da9fb466b5@intel.com> Date: Thu, 11 May 2017 18:00:06 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <20170328034903.41482-6-ajit.khaparde@broadcom.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 05/28] bnxt: add support for PF/VF communications 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: Thu, 11 May 2017 17:00:11 -0000 On 3/28/2017 4:48 AM, Ajit Khaparde wrote: > Set up interrupts and default completion ring during device init to > allow VF communications. > > Request most VF HWRM commands be forwarded to the PF driver. In the > future, this should be configurable. > > Fix reporting of active VFs. > > Add a log message when a completion is ignored on the default completion > ring since this will indicate support that needs to be added. > > Do not initialize the default completion ring in bnxt_alloc_hwrm_rings(), > but move it out to bnxt_dev_init(). > > Create a structure bnxt_child_vf_info for PF to cache VF specific info. > > Signed-off-by: Stephen Hurd > Signed-off-by: Ajit Khaparde Hi Ajit, Stephen, This patch doesn't cleanly apply on top of latest next-net, can you please rebase the patchset? Also as reminder, there was previously reported 32bit and shared library builds. And there are some checkpatch warnings in patch 25 and 28. Thanks, ferruh