[PATCH] Consolidate mm_context_t definition in mmu.h

Josh Boyer jwboyer at linux.vnet.ibm.com
Wed Jul 11 22:35:19 EST 2007


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




More information about the Linuxppc-dev mailing list