From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753049AbeDTU4P (ORCPT ); Fri, 20 Apr 2018 16:56:15 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:54694 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751882AbeDTU4N (ORCPT ); Fri, 20 Apr 2018 16:56:13 -0400 Date: Fri, 20 Apr 2018 16:56:07 -0400 (EDT) From: Mikulas Patocka X-X-Sender: mpatocka@file01.intranet.prod.int.rdu2.redhat.com To: Michal Hocko cc: Matthew Wilcox , David Miller , Andrew Morton , linux-mm@kvack.org, eric.dumazet@gmail.com, edumazet@google.com, bhutchings@solarflare.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, mst@redhat.com, jasowang@redhat.com, virtualization@lists.linux-foundation.org, dm-devel@redhat.com, Vlastimil Babka Subject: Re: [PATCH] kvmalloc: always use vmalloc if CONFIG_DEBUG_VM In-Reply-To: <20180420134901.GB17484@dhcp22.suse.cz> Message-ID: References: <3e65977e-53cd-bf09-bc4b-0ce40e9091fe@gmail.com> <20180418.134651.2225112489265654270.davem@davemloft.net> <20180420130852.GC16083@dhcp22.suse.cz> <20180420134136.GD10788@bombadil.infradead.org> <20180420134901.GB17484@dhcp22.suse.cz> User-Agent: Alpine 2.02 (LRH 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 20 Apr 2018, Michal Hocko wrote: > On Fri 20-04-18 06:41:36, Matthew Wilcox wrote: > > On Fri, Apr 20, 2018 at 03:08:52PM +0200, Michal Hocko wrote: > > > > In order to detect these bugs reliably I submit this patch that changes > > > > kvmalloc to always use vmalloc if CONFIG_DEBUG_VM is turned on. > > > > > > No way. This is just wrong! First of all, you will explode most likely > > > on many allocations of small sizes. Second, CONFIG_DEBUG_VM tends to be > > > enabled quite often. > > > > I think it'll still suit Mikulas' debugging needs if we always use > > vmalloc for sizes above PAGE_SIZE? > > Even if that was the case then this doesn't sounds like CONFIG_DEBUG_VM > material. We do not want a completely different behavior when the config > > -- > Michal Hocko > SUSE Labs I'm not arguing that it must be turned on exactly by CONFIG_DEBUG_VM. It may be turned on some other option that is enabled in debug kernels (CONFIG_DEBUG_SG may be a better option, because you'll get meaningful stracktraces from DMA API then). > is enabled. If we really need some better fallback testing coverage > then the fault injection, as suggested by Vlastimil, sounds much more > reasonable to me People who test kernels will install the kernel-debug package, reboot to the debug kernel and run their testsuites. They won't turn on magic options in debugfs or use some hideous kernel commandline arguments. If the kvmalloc test isn't in the debug kernel, then the testing crew won't test it - that's it. Mikulas