Re: [PATCH] small cleanups + fixes for perfmon.c

From: Stephane Eranian <eranian_at_hpl.hp.com>
Date: 2003-10-09 03:52:38
John,

On Wed, Oct 08, 2003 at 03:20:02PM +0100, John Levon wrote:
> 
> The following patch, compile-tested only so far, does the following :
> 
> o removes the [UN]LOCK_BUF_FMT_LIST defines - there is no abstraction
>   benefit to them, and they just look ugly
> o fixes two KERN_ERR that should be KERN_INFO
> o removes the "nolock" parameter to the find function (was unused)
> o homogenises the names related to buffer formats
> o removes the hand-crafted list in favour of standard linux list methods
> o reduces code duplication
> 
> OK ?
> 
Yes, that one looks fine to me. I will apply it
to me version and then submit this to David.

Thanks for the cleanups.

-- 
-Stephane
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Oct 8 13:59:05 2003

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:19 EST