Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp5094630ybi; Tue, 28 May 2019 07:27:55 -0700 (PDT) X-Google-Smtp-Source: APXvYqz0Re25D209glOwqfj1jJmAamAbwYycTb20qtm6H7g1w8vXhtRExNc/zRGcYaIqT37i2eUn X-Received: by 2002:a17:902:b58f:: with SMTP id a15mr91575817pls.201.1559053675233; Tue, 28 May 2019 07:27:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1559053675; cv=none; d=google.com; s=arc-20160816; b=dMXfdKjNlSWAvwEEKMpp7aK6e5SLKUugb81oCpYBKvIZ50CK5EOJ7JCMN0X48MaNWs vXPyILGfVtUiG2Je23dE/FUP0zD+X2suWE6lvBGpI2HoMBJo0YgCOUD8MvKpnmf5n4+V x2JqeDw68siV/Hzb0rqgX4jm0DoEuG2POmPIQrH7SxGh1EA0wQ6aNaBPaecPeR9o+Cy3 0FgV2pqlEd8mppapp3hkS+jPeANJFZvYRRxbSsEfxnsWdqZ/NTquujSBLTq1q/xUcAF+ Gt4xY2Ki+6hYgCZxKQhVF+tojYyO/UWAVsRWCHqqFUuNKRt//fN0RUT4OvLd8Fs5FEI3 8h1w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=DwBEjDicjoUzmmrSeAhWpEjkO/V2KDhNyhleGQkKM5Y=; b=DNRuv5F2ALbUbxtHSN1B6uVuSZVJXLQLkOWBczNyN+aCBvoV05CiBC9Lx9/FUpo5g2 KJOAQHUGLAdvcEqY48jtVMmQ8k9IqCs/eCPEFa8RlJLhqRv+frsiZhdLvbICSPLncplp /Rv0EIJRESoukBJB7MFQXuNdhH+BQS8P8ZaxvRTlpd/s63c6OxNdTeltygnuZKT/T+jF 0pzjLY9ITDYh4dQ74Og10tnQKeP6ow1bCJGLvYxb0ZG93Bt7u8Ds9P/r8BYx9YWqJmXU kNtdIHNfT5Ak7HJeRa75TvOq666epTE/yPWfYByVLFj4BdCoRaO+ac0TdKAYyP3AF6kr QfKA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 90si22840248pla.323.2019.05.28.07.27.40; Tue, 28 May 2019 07:27:55 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727430AbfE1Mpg (ORCPT + 99 others); Tue, 28 May 2019 08:45:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:45024 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727195AbfE1Mpg (ORCPT ); Tue, 28 May 2019 08:45:36 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E964D6CFA5; Tue, 28 May 2019 12:45:34 +0000 (UTC) Received: from dhcp-128-65.nay.redhat.com (ovpn-12-84.pek2.redhat.com [10.72.12.84]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 4ED87194A0; Tue, 28 May 2019 12:45:30 +0000 (UTC) Date: Tue, 28 May 2019 20:45:26 +0800 From: Dave Young To: Kairui Song Cc: linux-kernel@vger.kernel.org, Rahul Lakkireddy , "David S . Miller" , Eric Biederman , Alexey Dobriyan , Andrew Morton , "kexec@lists.infradead.org" , Bhupesh Sharma , Baoquan He Subject: Re: [PATCH v4] vmcore: Add a kernel parameter novmcoredd Message-ID: <20190528124526.GA11184@dhcp-128-65.nay.redhat.com> References: <20190528111856.7276-1-kasong@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190528111856.7276-1-kasong@redhat.com> User-Agent: Mutt/1.11.3 (2019-02-01) X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Tue, 28 May 2019 12:45:35 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/28/19 at 07:18pm, Kairui Song wrote: > Since commit 2724273e8fd0 ("vmcore: add API to collect hardware dump in > second kernel"), drivers is allowed to add device related dump data to > vmcore as they want by using the device dump API. This have a potential > issue, the data is stored in memory, drivers may append too much data > and use too much memory. The vmcore is typically used in a kdump kernel > which runs in a pre-reserved small chunk of memory. So as a result it > will make kdump unusable at all due to OOM issues. > > So introduce new 'novmcoredd' command line option. User can disable > device dump to reduce memory usage. This is helpful if device dump is > using too much memory, disabling device dump could make sure a regular > vmcore without device dump data is still available. > > Signed-off-by: Kairui Song > > --- > Update from V3: > - Use novmcoredd instead of vmcore_device_dump. Use > vmcore_device_dump and make it off by default is confusing, > novmcoredd is a cleaner way to let user space be able to disable > device dump to save memory. > > Update from V2: > - Improve related docs > > Update from V1: > - Use bool parameter to turn it on/off instead of letting user give > the size limit. Size of device dump is hard to determine. > > Documentation/admin-guide/kernel-parameters.txt | 11 +++++++++++ > fs/proc/Kconfig | 3 ++- > fs/proc/vmcore.c | 8 ++++++++ > 3 files changed, 21 insertions(+), 1 deletion(-) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index 138f6664b2e2..1b900d262680 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -2872,6 +2872,17 @@ > /sys/module/printk/parameters/console_suspend) to > turn on/off it dynamically. > > + novmcoredd [KNL,KDUMP] > + Disable device dump. Device dump allows drivers to > + append dump data to vmcore so you can collect driver > + specified debug info. The drivers could append the > + data without any limit, and the data is stored in > + memory, this may bring a significant memory stress. > + Disable device dump can help save memory but driver > + debug data will be no longer available. > + Only available when CONFIG_PROC_VMCORE_DEVICE_DUMP > + is set. > + > noaliencache [MM, NUMA, SLAB] Disables the allocation of alien > caches in the slab allocator. Saves per-node memory, > but will impact performance. > diff --git a/fs/proc/Kconfig b/fs/proc/Kconfig > index 817c02b13b1d..62b19162d198 100644 > --- a/fs/proc/Kconfig > +++ b/fs/proc/Kconfig > @@ -57,7 +57,8 @@ config PROC_VMCORE_DEVICE_DUMP > snapshot. > > If you say Y here, the collected device dumps will be added > - as ELF notes to /proc/vmcore. > + as ELF notes to /proc/vmcore. You can still disabled device > + dump by command line option 'novmcoredd'. > > config PROC_SYSCTL > bool "Sysctl support (/proc/sys)" if EXPERT > diff --git a/fs/proc/vmcore.c b/fs/proc/vmcore.c > index 3fe90443c1bb..e815fd035fc0 100644 > --- a/fs/proc/vmcore.c > +++ b/fs/proc/vmcore.c > @@ -53,6 +53,9 @@ static struct proc_dir_entry *proc_vmcore; > /* Device Dump list and mutex to synchronize access to list */ > static LIST_HEAD(vmcoredd_list); > static DEFINE_MUTEX(vmcoredd_mutex); > + > +static bool vmcoredd_disabled; > +core_param(novmcoredd, vmcoredd_disabled, bool, 0); > #endif /* CONFIG_PROC_VMCORE_DEVICE_DUMP */ > > /* Device Dump Size */ > @@ -1451,6 +1454,11 @@ int vmcore_add_device_dump(struct vmcoredd_data *data) > size_t data_size; > int ret; > > + if (vmcoredd_disabled) { > + pr_err_once("Device dump is disabled\n"); > + return -EINVAL; > + } > + > if (!data || !strlen(data->dump_name) || > !data->vmcoredd_callback || !data->size) > return -EINVAL; > -- > 2.21.0 > Acked-by: Dave Young Thanks Dave