On 11/9/06, Kirill Korotaev <[EMAIL PROTECTED]> wrote:
> +#ifdef CONFIG_BEANCOUNTERS
> +#define BC_EXTRASIZE   sizeof(struct beancounter *)

Would much prefer you put all beancounter stuff into one #ifdef block
to avoid clutter.

> @@ -2579,14 +2635,14 @@ static struct slab *alloc_slabmgmt(struc
>         slabp->colouroff = colour_off;
>         slabp->s_mem = objp + colour_off;
>         slabp->nodeid = nodeid;
> +#ifdef CONFIG_BEANCOUNTERS
> +       if (cachep->flags & SLAB_BC)
> +               memset(slab_bc_ptrs(cachep, slabp), 0,
> +                               cachep->num * BC_EXTRASIZE);
> +#endif

No #ifdef within functions, please, but instead, make it an static
inline function.

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to