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

* Chuck Ebbert <[email protected]> wrote:

> In-Reply-To: <[email protected]>
> 
> 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 <[email protected]>

thanks, applied.

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

---

• Prev by Date: Re: mtrr: 0xe4000000,0x4000000 overlaps existing 0xe4000000,0x800000
• Next by Date: Panic in SCSI subsystem
• Previous by thread: Re: [PATCH] protect remove_proc_entry
• Next by thread: Re: [patch 03/13] mutex subsystem, add include/asm-i386/mutex.h
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]