From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Falcon Subject: Re: [PATCH 0/3] ibmvnic: Fix bugs and memory leaks Date: Wed, 16 May 2018 15:59:41 -0500 Message-ID: References: <1526503745-14421-1-git-send-email-tlfalcon@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: jallen@linux.vnet.ibm.com, nfont@linux.vnet.ibm.com To: netdev@vger.kernel.org Return-path: Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:56412 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751013AbeEPU7q (ORCPT ); Wed, 16 May 2018 16:59:46 -0400 Received: from pps.filterd (m0098404.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id w4GKx7sa088604 for ; Wed, 16 May 2018 16:59:46 -0400 Received: from e16.ny.us.ibm.com (e16.ny.us.ibm.com [129.33.205.206]) by mx0a-001b2d01.pphosted.com with ESMTP id 2j0r7ktbbb-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 16 May 2018 16:59:46 -0400 Received: from localhost by e16.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 16 May 2018 16:59:44 -0400 In-Reply-To: <1526503745-14421-1-git-send-email-tlfalcon@linux.vnet.ibm.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 05/16/2018 03:49 PM, Thomas Falcon wrote: > This is a small patch series fixing up some bugs and memory leaks > in the ibmvnic driver. The first fix frees up previously allocated > memory that should be freed in case of an error. The second fixes > a reset case that was failing due to TX/RX queue IRQ's being > erroneously disabled without being enabled again. The final patch > fixes incorrect reallocated of statistics buffers during a device > reset, resulting in loss of statistics information and a memory leak. > > Thomas Falcon (3): > ibmvnic: Free coherent DMA memory if FW map failed > ibmvnic: Fix non-fatal firmware error reset > ibmvnic: Fix statistics buffers memory leak Sorry, these are meant for the 'net' tree. Tom > > drivers/net/ethernet/ibm/ibmvnic.c | 28 +++++++++++++++++----------- > 1 file changed, 17 insertions(+), 11 deletions(-) >