Messages in this thread | | | From | Rusty Russell <> | Subject | Re: [PATCH 7/24] consolidate msr.h | Date | Tue, 20 Nov 2007 21:43:12 +1100 |
| |
On Tuesday 20 November 2007 17:16:45 Steven Rostedt wrote: > On Tue, 20 Nov 2007, Ingo Molnar wrote: > > i dont think there's ever any true need (and good cause) to force > > integer type casts like that at the callee site. > > Unless you mean we should do something like this: > > static inline void __wrmsrl(unsigned int msr, unsigned long long val); > #define wrmsr(msr, val) __wrmsrl(msr, (unsigned long long)var)
Heh:
union ptr_or_val { void *ptr; unsigned long long val; };
static inline void __wrmsrl(unsigned int msr, union ptr_or_val pv); #define wrmsr(msr, v) __wrmsrl(msr, (union ptr_or_val)v)
Ok, maybe not... Rusty. - 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/
| |