From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH] examples: fix symmetric_mp, set NIC rx_drop_en bit Date: Thu, 4 Dec 2014 14:03:24 +0000 Message-ID: <20141204140324.GA4828@bricha3-MOBL3> References: <1417625819-3024-1-git-send-email-bruce.richardson@intel.com> <1770523.OPzYA99vBi@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev-VfR2kkLFssw@public.gmane.org To: Thomas Monjalon Return-path: Content-Disposition: inline In-Reply-To: <1770523.OPzYA99vBi@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On Thu, Dec 04, 2014 at 02:37:15PM +0100, Thomas Monjalon wrote: > 2014-12-03 16:56, Bruce Richardson: > > The symmetric_mp example app is set up to allow two processes to > > share a NIC port, with each pulling packets from one queue. In order > > to have the app continue working when one of the process dies, the > > drop_en bit should be set in the NIC configuration. Without this bit > > set, the NIC will stall once any queue fills. With the bit set, once > > a queue fills, all subsequent packets for that queue are discarded > > allowing other queues to continue operating as normal. > > > > This setting was missed when converting to use standardised defaults > > in commit 81f7ecd9. > > I don't see rx_drop_en in commit 81f7ecd9. > Why do you say it was missed? > This is the one app that needs the drop_en bit set, and the bit is not set in the defaults. Therefore, when this app was converted to use the defaults from the driver, the drop_en bit should have been explicitly set as in this patch. [The fact of there being no drop_en bit in the commit is proof of its absense :-)]. /Bruce