Received: by 10.213.65.68 with SMTP id h4csp291703imn; Mon, 12 Mar 2018 14:02:26 -0700 (PDT) X-Google-Smtp-Source: AG47ELvROrLfGqcO+9nzTcoaSqFD231uC0gMXFt99OIYczov15T6ZKSzstpL/TJQVBAP8OYxsyHD X-Received: by 10.101.96.142 with SMTP id t14mr7607316pgu.58.1520888546258; Mon, 12 Mar 2018 14:02:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1520888546; cv=none; d=google.com; s=arc-20160816; b=MQX/C3i8kF1jN7ovnXrGaMFAIYXe6fied7leWWCEovUJwyuszuG7+4bw6/mAOXHsc1 0dufMfJ3HykuGU3xxWsxJqmrLjTobTilykMdQYN+1amAWEKGHt+p9WRlFY3iw9FmNw0U X1ts5MKzPbUrxnI1pezi0fE435Xj3fw3otD63L8gKNrKQbZWF+CUCFa/bX9gH0GlDkWH a2R6rxJjjl2ooy+vnSKXVsGAqRopHDC197dxVOQt6IrpWag1fFWJDcLtY5dboBPDrq+L GE2krToXrf/hyVHJsKRt5COG/RRUYDRFNRLvj9KLBfkmzCI8r0/yiWxkb2crlI6HR45l Oh0g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :arc-authentication-results; bh=esK7mMCbXsB5PlQYSmUV3gM3eONYjQKzaGJMjKSOm10=; b=IsFHyX6rVcegwsRU6Smp5PmwjC/qnKqJzEoX7/hi8Y1VHdtZ40wRr8JriNVNDocZ4u bNTvQ1NMIn8Kx466ZsKNFN0Z6dAkdq+yO6jsdaryeZ5FKMzJx8NLuELAZdevpAmVmEKj 3SJyQLRA8arFuIVteUrqa4jHTN837MiqIc8Vs/xnNQuNuDpVzUjpr8AC82oBSSv+9LPB igHRSO3UzG+kjV08XimW7hD7g9Oraw0zxSjOrzi4d+faJY3q+FUskcUK5aB5bnCbG90o txIt7ZyLEye+5my6iXYEKXjBGhW0fkLPQSIpw76wOpbebKCEd1voR3Wet2LGf20dMrxZ pl7g== 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t10si5641676pgc.18.2018.03.12.14.02.09; Mon, 12 Mar 2018 14:02:26 -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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932496AbeCLVAa (ORCPT + 99 others); Mon, 12 Mar 2018 17:00:30 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33628 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932302AbeCLVA3 (ORCPT ); Mon, 12 Mar 2018 17:00:29 -0400 Received: from akpm3.svl.corp.google.com (unknown [104.133.9.71]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id 7AD1AEA3; Mon, 12 Mar 2018 21:00:28 +0000 (UTC) Date: Mon, 12 Mar 2018 14:00:27 -0700 From: Andrew Morton To: Waiman Long Cc: "Luis R. Rodriguez" , Kees Cook , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, Al Viro , Matthew Wilcox Subject: Re: [PATCH v4 3/6] sysctl: Warn when a clamped sysctl parameter is set out of range Message-Id: <20180312140027.97db31d5dfc5c5f52a7c19a3@linux-foundation.org> In-Reply-To: <1520885744-1546-4-git-send-email-longman@redhat.com> References: <1520885744-1546-1-git-send-email-longman@redhat.com> <1520885744-1546-4-git-send-email-longman@redhat.com> X-Mailer: Sylpheed 3.6.0 (GTK+ 2.24.31; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 12 Mar 2018 16:15:41 -0400 Waiman Long wrote: > Even with clamped sysctl parameters, it is still not that straight > forward to figure out the exact range of those parameters. One may > try to write extreme parameter values to see if they get clamped. > To make it easier, a warning with the expected range will now be > printed into the kernel ring buffer when a clamped sysctl parameter > receives an out of range value. > > The pr_warn_ratelimited() macro is used to limit the number of warning > messages that can be printed within a given period of time. > > ... > > +#ifdef pr_fmt > +#undef pr_fmt > +#endif > +#define pr_fmt(fmt) "sysctl: " fmt Why is it necessary to undefine pr_fmt? That's a somewhat unusual thing to do.