From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] fm10k: fix a crash bug when quit from testpmd Date: Mon, 23 Nov 2015 23:55:05 +0100 Message-ID: <3695173.dQDnAloRu5@xps13> References: <1447304251-4145-1-git-send-email-jing.d.chen@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Chen Jing D(Mark)" Return-path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id 31169568A for ; Mon, 23 Nov 2015 23:56:22 +0100 (CET) Received: by wmec201 with SMTP id c201so183552714wme.0 for ; Mon, 23 Nov 2015 14:56:22 -0800 (PST) In-Reply-To: <1447304251-4145-1-git-send-email-jing.d.chen@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-11-12 12:57, Chen Jing D: > From: "Chen Jing D(Mark)" > > When the fm10k port is closed, both func tx_queue_clean() and > fm10k_tx_queue_release_mbufs_vec() will try to release buffer in > SW ring. The latter func won't do sanity check on those pointers > and cause crash. > > The fix include 2 parts. > 1. Remove Vector TX buffer release func since it can share the > release functions with regular TX. > 2. Add log to print out what actual Rx/Tx func is used. 2 parts mean 2 patches. [...] > + if (rx_using_sse) > + PMD_INIT_LOG(ERR, "Use vector Rx func"); > + else > + PMD_INIT_LOG(ERR, "Use regular Rx func"); Why using en error log level?