From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751596AbdFHS0I (ORCPT ); Thu, 8 Jun 2017 14:26:08 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33708 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751383AbdFHS0H (ORCPT ); Thu, 8 Jun 2017 14:26:07 -0400 Date: Thu, 8 Jun 2017 20:25:05 +0200 From: Greg KH To: Felix Schnizlein Cc: linux-kernel@vger.kernel.org, x86@vger.kernel.org, yanmin.zhang@intel.com, trenn@suse.com Subject: Re: [RFC PATCH 3/3] Add deprecation warning to cpuinfo proc Message-ID: <20170608182505.GD13995@kroah.com> References: <1496940975-9164-1-git-send-email-fschnizlein@suse.com> <1496940975-9164-4-git-send-email-fschnizlein@suse.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1496940975-9164-4-git-send-email-fschnizlein@suse.com> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jun 08, 2017 at 06:56:15PM +0200, Felix Schnizlein wrote: > Enable deprecation warning if the kernel was compiled with sysfs cpuinfo. > > Signed-off-by: Thomas Renninger > --- > fs/proc/cpuinfo.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/fs/proc/cpuinfo.c b/fs/proc/cpuinfo.c > index 06f4d31..044f56e 100644 > --- a/fs/proc/cpuinfo.c > +++ b/fs/proc/cpuinfo.c > @@ -6,6 +6,10 @@ > extern const struct seq_operations cpuinfo_op; > static int cpuinfo_open(struct inode *inode, struct file *file) > { > +#ifdef CONFIG_CPUINFO_SYSFS > + pr_info("Do not access deprecated file /proc/cpuinfo, " > + "opened by: (%s,%d)\n", current->comm, current->pid); Hah, that's not going to happen. Do you know just how many different programs open this file for no good reason? Well, they think it's a good reason... You are just going to spam syslog so hard, it's going to be annoying. This patch is not ok at all, sorry. It's not like we are ever going to get rid of /proc/cpuinfo so it's not a big deal. thanks, greg k-h