Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932661AbbGJOTX (ORCPT ); Fri, 10 Jul 2015 10:19:23 -0400 Received: from resqmta-ch2-08v.sys.comcast.net ([69.252.207.40]:54276 "EHLO resqmta-ch2-08v.sys.comcast.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754061AbbGJOTO (ORCPT ); Fri, 10 Jul 2015 10:19:14 -0400 Date: Fri, 10 Jul 2015 09:19:11 -0500 (CDT) From: Christoph Lameter X-X-Sender: cl@east.gentwo.org To: Thomas Gleixner cc: LKML , Pekka Enberg , David Rientjes , Joonsoo Kim , Andrew Morton , linux-mm@kvack.org, Sebastian Andrzej Siewior , Steven Rostedt , Peter Zijlstra Subject: Re: [patch] mm/slub: Move slab initialization into irq enabled region In-Reply-To: <20150710120259.836414367@linutronix.de> Message-ID: References: <20150710120259.836414367@linutronix.de> Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 792 Lines: 28 There is a duplicate check for page == NULL now. Subject: slub: allocate_slab: no need to check twice for page == NULL Remove the second check. Signed-off-by: Christoph Lameter Index: linux/mm/slub.c =================================================================== --- linux.orig/mm/slub.c +++ linux/mm/slub.c @@ -1396,9 +1396,6 @@ static struct page *allocate_slab(struct kmemcheck_mark_unallocated_pages(page, pages); } - if (!page) - goto out; - page->objects = oo_objects(oo); order = compound_order(page); -- 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/