On Wed, 2007-07-11 at 12:33 +0200, Christoph Hellwig wrote: > > mm_context_id_t isn't actually used anywhere but in te mm_context_t > definition. So if you kill it you have two common fields and a bunch > of additional ones for PPC64 leading to a defintion like: > > typedef struct { > unsigned long id; > > #ifdef CONFIG_PPC64 > u16 user_psize; /* page size index */ > > #ifdef CONFIG_PPC_MM_SLICES > u64 low_slices_psize; /* SLB page size encodings */ > u64 high_slices_psize; /* 4 bits per slice for now */ > #else > u16 sllp; /* SLB page size encoding */ > #endif > #endif > > unsigned long vdso_base; > } mm_context_t;
Yes. I did mostly the same thing in version 2 of the patch, just using mm_context_id_t instead. Paul said the ifdefs made his eyes hurt. josh _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev