On Tuesday 09 May 2006 09:21, Pavel Machek wrote: > Hi! > > > +static inline void load_TLS(struct thread_struct *t, unsigned int cpu) > > +{ > > +#define C(i) get_cpu_gdt_table(cpu)[GDT_ENTRY_TLS_MIN + i] = t->tls_array[i] > > + C(0); C(1); C(2); > > +#undef C > > +} > > Why not use for loop here? gcc should be able to optimize it... I don't think you can really blame the Xen people for that code - it's just the old code moved. -Andi