[lkml]   [2004]   [Aug]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: 2.6.8-rc2-mm2
On Mon, Aug 02, 2004 at 01:55:27AM -0700, Andrew Morton wrote:
> - Tons of new fixes and cleanups and updates to various things as usual.
> - Again, 2.6.8 is close, so if people have critical fixes in here which
> need flushing out, please ping me.

Speaking of cleanups, I've got a little something.
65 files changed, 274 insertions(+), 1076 deletions(-)

This mass slaughter of duplicated code is a cleanup of /proc/profile
that consolidates code across all arches and privatizes private state.
Compiletested on x86-64. Prior incarnations of earlier cleanups this
is based on were runtime tested on ia32, x86-64, sparc64, and alpha.

The purpose of these cleanups in their prior incarnations has been for
use as a preparatory cleanup for profiling other kinds of events in
/proc/profile's buffer (or similar buffers). There has recently been a
need to discover which codepaths were responsible for leaking inodes
that were leaking that similar cleanups in combination with some slab
profiling hooks are being used to instrument.

In order, the patches of the series (included as a series of separate
MIME attachments to this post) are:
Handling of prof_cpu_mask is grossly inconsistent. Some arches
have it as a cpumask_t, others unsigned long, and even within
arches it's treated inconsistently. This makes it cpumask_t
across the board, and consolidates the handling in kernel/profile.c
The program counter calculation from pt_regs is the only
portion of profile accounting that differs across various
architectures. This is usually instruction_pointer(regs), but
to handle the few arches where it isn't, introduce profile_pc().
With prof_cpu_mask and profile_pc() in hand, the core is now
able to perform all the profile accounting work on behalf of
arches. Consolidate the profile accounting and convert all
arches to call the core function.
proc_misc.c is a trainwreck. Move the file_operations for
/proc/profile into kernel/profile.c and call the profiling
setup from proc_misc_init().
Make the various bits of state no longer used anywhere else
static to kernel/profile.c
Convert prof_buffer to an array of atomic_t instead of
sometimes atomic_t, sometimes unsigned int. Also, bootmem
rounds up internally, so blow away some crap code there.

-- wli
[unhandled content-type:application/octet-stream][unhandled content-type:application/octet-stream][unhandled content-type:application/octet-stream][unhandled content-type:application/octet-stream][unhandled content-type:application/octet-stream][unhandled content-type:application/octet-stream]
 \ /
  Last update: 2005-03-22 14:04    [W:0.190 / U:4.672 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site