Re: [RFC][PATCH] slub: Check for page NULL before doing the node_matchcheck

From: Christoph Lameter
Date: Thu Jan 17 2013 - 16:56:17 EST


on Thu, 17 Jan 2013, Christoph Lameter wrote:

> Ditto which leaves us with:
>
> Index: linux/mm/slub.c
> ===================================================================
> --- linux.orig/mm/slub.c 2013-01-17 15:49:57.417491975 -0600
> +++ linux/mm/slub.c 2013-01-17 15:50:49.010287150 -0600
> @@ -1993,8 +1993,9 @@ static inline void flush_slab(struct kme
> deactivate_slab(s, c->page, c->freelist);
>
> c->tid = next_tid(c->tid);
> - c->page = NULL;
> c->freelist = NULL;
> + barrier();
> + c->page = NULL;
> }

But the larger question is why is flush_slab() called with interrupts
enabled?

RT?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/