Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752515AbdLGUPa (ORCPT ); Thu, 7 Dec 2017 15:15:30 -0500 Received: from resqmta-ch2-05v.sys.comcast.net ([69.252.207.37]:47244 "EHLO resqmta-ch2-05v.sys.comcast.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750978AbdLGUP3 (ORCPT ); Thu, 7 Dec 2017 15:15:29 -0500 Date: Thu, 7 Dec 2017 14:15:27 -0600 (CST) From: Christopher Lameter X-X-Sender: cl@nuc-kabylake To: Kees Cook cc: Geert Uytterhoeven , Pekka Enberg , David Rientjes , Joonsoo Kim , Andrew Morton , "Tobin C . Harding" , Linux-MM , LKML , Linus Torvalds Subject: Re: [PATCH] mm/slab: Do not hash pointers when debugging slab In-Reply-To: Message-ID: References: <1512641861-5113-1-git-send-email-geert+renesas@glider.be> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-CMAE-Envelope: MS4wfCUu6D+p2Xt/kbH3NNAzQQWWhbzc3FrxDCTDlUFfds3HazFxEXMezVBAwcrXJws/MjEhFCirwCQjKlm4VBow518VDpz/0/YREi3tVxpEo5M1DJqpMtzU AypXxmrp/iPBTuQeusazK9G0R4nUbqU2A+Udx+iKj0C5FLYCPo8gQr3PkssA4y5KKOnpyDLWg/emFOY//rZjFQIE187851WRVzDP9akYW8kDEZ4JPSzJ2y27 0HHREPuRTk/YI6MTCk6AiFsG2MqYcFFOQGf8eS0NIKddDuiUDpQJvZ74u8AP4ARJR9f2k5+7fT0vZHhQ4b+CkUR3E01xw2sTZeZaNpbzF8YDXIuxGHI8muQJ 3Xxkyo03cLYjmfLijd4bYYlmJILOaAziX5BvVO26p8TTzoIfgMeqmGEYW3PHQkaQGPD0vlmyC214W/8Mc8KszMBpRNlo+ZnCY5+MIN3hqYHOnDNy8sk= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 545 Lines: 13 On Thu, 7 Dec 2017, Kees Cook wrote: > > These SLAB config options are only used for testing so this is ok. > > Most systems use SLUB so I can't say how common CONFIG_DEBUG_SLAB is. > (Though, FWIW with SLUB, CONFIG_SLUB_DEBUG is very common.) CONFIG_SLUB_DEBUG is on by default because it compiles into the kernel the runtime configurable debugging framework. It does not activate any debugging. CONFIG_SLUB_DEBUG_ON is the equivalent to CONFIG_SLAB_DEBUG. The kernel will boot with debugging on without any extra kernel options with these.