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 46841A0093; Mon, 18 May 2020 04:09:40 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id CAE211D513; Mon, 18 May 2020 04:09:39 +0200 (CEST) Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by dpdk.org (Postfix) with ESMTP id F31FF1C06A; Mon, 18 May 2020 04:09:37 +0200 (CEST) IronPort-SDR: mi0YwA+Um2mwNZRscj3ru3qB0Kpm28weAASmsu6IJJlB4hF7FUEM7y6P1jjSmOzW2LV9ozuzIY iK62p6LWk6cg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 May 2020 19:09:37 -0700 IronPort-SDR: MrTjMl2JAZghcdqrkd1mz/V1aZSpF/duccBuclYO7jHReyomFKqY3Iho2WQnJHvFii5DmAdbUP hngJPg6N3SuA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,405,1583222400"; d="scan'208";a="288407831" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.116.183]) by fmsmga004.fm.intel.com with ESMTP; 17 May 2020 19:09:35 -0700 Date: Mon, 18 May 2020 10:01:09 +0800 From: Ye Xiaolong To: Ting Xu Cc: dev@dpdk.org, beilei.xing@intel.com, jingjing.wu@intel.com, stable@dpdk.org Message-ID: <20200518020109.GB93932@intel.com> References: <20200511152748.21144-1-ting.xu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200511152748.21144-1-ting.xu@intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH v1] net/iavf: fix setting wrong RXDID value for Rx queue 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 05/11, Ting Xu wrote: >CVL kernel PF configures all reserved queues for VF, including >Rx queue RXDID. The number of reserved queues is the maximum >between Tx and Rx queues. If the number of the enabled Rx queues >is less than that of reserved queues, required RXDID will only >be set for those enabled, but default value (0) is set for others. >However, RXDID 0 (legacy 16byte descriptor) is not supported now, >PF will return error when configuring those disabled VF queues. > >In this patch, required RXDID is set for all reserved Rx queues, >no matter enabled or not. In this way, PF will configure Rx >queues correctly without reporting error. > >Fixes: b8b4c54ef9b0 ("net/iavf: support flexible Rx descriptor in normal path") >Cc: stable@dpdk.org b8b4c54ef9b0 is introduce in this release, no need to cc stable. > Acked-by: Xiaolong Ye Applied to dpdk-next-net-intel, Thanks.