Re: [patch 03/13] mutex subsystem, add include/asm-i386/mutex.h

[Ingo Molnar]

* Chuck Ebbert <76306.1226@xxxxxxxxxxxxxx> wrote:

> In-Reply-To: <20051229210336.GD665@xxxxxxx>
> 
> On Thu, 29 Dec 2005 at 22:03:36 +0100, Ingo Molnar wrote:
> 
> > +#define __mutex_fastpath_lock(count, fn_name)                                \
> > +do {                                                                 \
> > +     /* type-check the function too: */                              \
> > +     void fastcall (*__tmp)(atomic_t *) = fn_name;                   \
> > +     unsigned int dummy;                                             \
> > +                                                                     \
> > +     (void)__tmp;                                                    \
> > +     typecheck(atomic_t *, count);                                   \
> 
>  The function type checking is ugly.  Wouldn't this be better?
> 
> Signed-off-by: Chuck Ebbert <76306.1226@xxxxxxxxxxxxxx>

thanks, applied.

	Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/