Re: slub-i386-support.patch

[Christoph Lameter <clameter@sgi.com>]

On Thu, 10 May 2007, William Lee Irwin III wrote:

> So now quicklist semantics vs. TLB flushing are the motive behind the
> odd flush_tlb_mm() affair. The real trick with it is that flushing
> must never occur until the TLB flush. Any change to the core quicklist
> code that retires pages back to the page allocator earlier (e.g. based
> on some limit) will break things badly.

Right.

> On Thu, May 10, 2007 at 09:03:39PM +0100, Hugh Dickins wrote:
> >> -struct kmem_cache *pgd_cache;
> >>  struct kmem_cache *pmd_cache;
> >>  
> >>  void __init pgtable_cache_init(void)
> >> @@ -776,12 +775,6 @@ void __init pgtable_cache_init(void)
> >>  			pgd_size = PAGE_SIZE;
> >>  		}
> >>  	}
> >> -	pgd_cache = kmem_cache_create("pgd",
> >> -				pgd_size,
> >> -				pgd_size,
> >> -				SLAB_PANIC,
> >> -				pgd_ctor,
> >> -				(!SHARED_KERNEL_PMD) ? pgd_dtor : NULL);
> >>  }
> 
> This is wrong; pgd's are smaller than PAGE_SIZE on PAE. Burning lowmem
> like this is very, very bad for such systems. pmd_cache is rather
> trivial to convert to quicklists, since all it does is zero pages. I
> still don't approve of even the !SHARED_KERNEL_PMD case using PAGE_SIZE
> -sized pgd's. Xen should really be fixed to avoid requiring guests to
> have recursive pagetables or whatever it's doing.

But the Xen guys need the full sized pgd?

> 
> >> @@ -205,8 +206,6 @@ void pmd_ctor(void *pmd, struct kmem_cac
> >>   * against pageattr.c; it is the unique case in which a valid change
> >>   * of kernel pagetables can't be lazily synchronized by vmalloc faults.
> >>   * vmalloc faults work because attached pagetables are never freed.
> >> - * The locking scheme was chosen on the basis of manfred's
> >> - * recommendations and having no core impact whatsoever.
> >>   * -- wli
> >>   */
> >>  DEFINE_SPINLOCK(pgd_lock);
> 
> This comment deletion is bogus, as the locking scheme has not changed.

The interaction with SLAB has changed with the conversion to quicklists. 
AFAIK Manfred's suggestions were due to the use of the slab allocator to manage 
pgds.

-
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/