Re: [PATCH 8/8] ALSA: convert kcalloc to kzalloc

[Paulo Marques <pmarques@grupopie.com>]

Dmitry Torokhov wrote:
> On 8/5/05, Pekka Enberg <penberg@cs.helsinki.fi> wrote:
>
> > This patch converts kcalloc(1, ...) calls to use the new kzalloc() function.
>
> Hi,
>
> Have you seen the following in include/sound/core?
>
> ...
> #define kmalloc(size, flags) snd_hidden_kmalloc(size, flags)
> #define kcalloc(n, size, flags) snd_hidden_kcalloc(n, size, flags)
> #define kfree(obj) snd_hidden_kfree(obj)
Arghh... I've been bitten by this before, too.

Pekka, the sound subsystem uses its own allocation functions that are 
just defined to the standard kernel counterparts when 
CONFIG_SND_DEBUG_MEMORY is not defined, but that map to their own 
functions that add aditional debug information when that config is set.
To play well with the current structure you have to define you own 
snd_hidden_kzalloc(size, flags) and use the same scheme.
For my previous encounter with this problem search a thread named 
"replace snd_kmalloc_strdup by kstrdup". The patch there might useful to 
you.
I really hate this "#define kmalloc" hack. It makes the code really 
unreadble, because you expect a kmalloc to be just a kmalloc...
Couldn't we turn this into a generic kernel debugging option, so that it 
could be used for every kmalloc instead of just the ones from the sound 
system?
If I get this right, what this code does is to track kfree's on pointers 
that were not alloc'ed with kmalloc (using a magic number) and keep 
track of all the allocations to detect leaks.
We already have SLAB_DEBUG. We could add a list of allocations with a 
proc interface (or something) to give an histogram of kmalloc callers / 
number of allocations not yet freed.
This way, if after stopping everything related to sound there were still 
callers like "snd_xxxx" (through kallsyms) you would know there is a 
leak there.
What does CONFIG_SND_DEBUG_MEMORY provide that this more generic scheme 
does not?
--
Paulo Marques
Software Development Department - Grupo PIE, S.A.
Phone: +351 252 290600, Fax: +351 252 290601
Web: www.grupopie.com

It is a mistake to think you can solve any major problems
just with potatoes.
Douglas Adams
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/