From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH 1/2] bnx2x: fix error handling in bnx2x_loop_obtain_resources() Date: Mon, 8 Feb 2016 13:51:02 +0000 Message-ID: <20160208135101.GA13952@bricha3-MOBL3> References: <1451522271-16924-1-git-send-email-3chas3@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org, "Charles \(Chas\) Williams" , sony.chacko@qlogic.com To: Chas Williams <3chas3@gmail.com> Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 77AB33195 for ; Mon, 8 Feb 2016 14:51:21 +0100 (CET) Content-Disposition: inline In-Reply-To: <1451522271-16924-1-git-send-email-3chas3@gmail.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" On Wed, Dec 30, 2015 at 07:37:50PM -0500, Chas Williams wrote: > From: "Charles (Chas) Williams" > > bnx2x_loop_obtain_resources() returns a struct containing the status and > the error message. If bnx2x_do_req4pf() fails, it shouldn't return both > of these fields set to 0 indicating failure and no error. > > Further, bnx2x_do_req4pf() needs to be able fail and return NO_RESOURCES > so that bnx2x_loop_obtain_resources() can negotiate reduced resource > requirments. This requires additional checking around bnx2x_do_req4pf(). > > Signed-off-by: Chas Williams <3chas3@gmail.com> > --- Hi Chas, can you please provide a commit reference for a fixes line for this patch. [No need for a V2 just for that alone, just the reference is enough to save me diving into the git history. I'll manually add the fixes line when committing to the next-net tree] Maintainers, can you please review this patchset and provide feedback or an ACK. Regards, /Bruce