From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 97A03A0561; Tue, 21 Apr 2020 05:22:07 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6ED511D5C9; Tue, 21 Apr 2020 05:22:06 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 6554D1D5B4 for ; Tue, 21 Apr 2020 05:22:04 +0200 (CEST) IronPort-SDR: 1btD4SGKleX+G5b4vy21szX8xGBcX2tvkpvu9OxMV55aNhD7hnXeSmHZjoH+Htr6nDPwDncfnL hOYavpK+gN8g== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Apr 2020 20:22:03 -0700 IronPort-SDR: CT6VXwaLf+rCV18dfgxeh9FCrw1+nSgj/zI3TyMmH7JPqCSFhbToRmQGZZkRzW3zpy9Fo5sANb bx8wDl5fh0qw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,408,1580803200"; d="scan'208";a="402048836" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.117.17]) by orsmga004.jf.intel.com with ESMTP; 20 Apr 2020 20:22:02 -0700 Date: Tue, 21 Apr 2020 11:17:43 +0800 From: Ye Xiaolong To: Haiyue Wang Cc: dev@dpdk.org, qi.z.zhang@intel.com, qiming.yang@intel.com Message-ID: <20200421031743.GH33101@intel.com> References: <20200416031122.59640-1-haiyue.wang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200416031122.59640-1-haiyue.wang@intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH v1] net/ice: check the DSN package file firstly 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 04/16, Haiyue Wang wrote: >Align the package file search sequence with PF only for DCF mode. Get >the DSN through the virtual channel firstly to check the package file >accessible. > >Signed-off-by: Haiyue Wang >--- > drivers/net/ice/ice_dcf_parent.c | 30 ++++++++++++++++++++++++++---- > 1 file changed, 26 insertions(+), 4 deletions(-) > >diff --git a/drivers/net/ice/ice_dcf_parent.c b/drivers/net/ice/ice_dcf_parent.c >index e05b6b3e5..d4b4ededb 100644 >--- a/drivers/net/ice/ice_dcf_parent.c >+++ b/drivers/net/ice/ice_dcf_parent.c >@@ -226,12 +226,34 @@ ice_dcf_request_pkg_name(struct ice_hw *hw, char *pkg_name) > { > struct ice_dcf_adapter *dcf_adapter = > container_of(hw, struct ice_dcf_adapter, parent.hw); >+ struct virtchnl_pkg_info pkg_info; >+ struct dcf_virtchnl_cmd vc_cmd; >+ uint64_t dsn; > >- /* TODO: check with DSN firstly by iAVF */ >- PMD_INIT_LOG(DEBUG, >- "DCF VSI_ID = %u", >- dcf_adapter->real_hw.vsi_id); >+ vc_cmd.v_op = VIRTCHNL_OP_DCF_GET_PKG_INFO; >+ vc_cmd.req_msglen = 0; >+ vc_cmd.req_msg = NULL; >+ vc_cmd.rsp_buflen = sizeof(pkg_info); >+ vc_cmd.rsp_msgbuf = (uint8_t *)&pkg_info; > >+ if (ice_dcf_execute_virtchnl_cmd(&dcf_adapter->real_hw, &vc_cmd)) >+ goto pkg_file_direct; >+ >+ rte_memcpy(&dsn, pkg_info.dsn, sizeof(dsn)); >+ >+ snprintf(pkg_name, ICE_MAX_PKG_FILENAME_SIZE, >+ ICE_PKG_FILE_SEARCH_PATH_UPDATES "ice-%016llX.pkg", >+ (unsigned long long)dsn); >+ if (!access(pkg_name, 0)) >+ return 0; >+ >+ snprintf(pkg_name, ICE_MAX_PKG_FILENAME_SIZE, >+ ICE_PKG_FILE_SEARCH_PATH_DEFAULT "ice-%016llX.pkg", >+ (unsigned long long)dsn); >+ if (!access(pkg_name, 0)) >+ return 0; >+ >+pkg_file_direct: > snprintf(pkg_name, > ICE_MAX_PKG_FILENAME_SIZE, "%s", ICE_PKG_FILE_UPDATES); > if (!access(pkg_name, 0)) >-- >2.26.1 > Applied to dpdk-next-net-intel, Thanks.