From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (Postfix) with ESMTP id 086167F5F for ; Tue, 1 Oct 2013 10:00:46 -0500 (CDT) Message-ID: <524AE39B.3040505@sgi.com> Date: Tue, 01 Oct 2013 10:00:43 -0500 From: Mark Tinguely MIME-Version: 1.0 Subject: Re: [PATCH] xfs: Use kmem_free() instead of free() References: <1380638873-5181-1-git-send-email-treding@nvidia.com> In-Reply-To: <1380638873-5181-1-git-send-email-treding@nvidia.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Thierry Reding Cc: Ben Myers , Alex Elder , linux-kernel@vger.kernel.org, xfs@oss.sgi.com On 10/01/13 09:47, Thierry Reding wrote: > This fixes a build failure caused by calling the free() function which > does not exist in the Linux kernel. > > Signed-off-by: Thierry Reding > --- > fs/xfs/xfs_log_recover.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c > index 4324058..3979749 100644 > --- a/fs/xfs/xfs_log_recover.c > +++ b/fs/xfs/xfs_log_recover.c > @@ -1585,7 +1585,7 @@ xlog_recover_add_to_trans( > "bad number of regions (%d) in inode log format", > in_f->ilf_size); > ASSERT(0); > - free(ptr); > + kmem_free(ptr); > return XFS_ERROR(EIO); > } > Looks good. I will remove the other list items in another patch. Reviewed-by: Mark Tinguely _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs