From patchwork Wed Jun 11 19:01:13 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Borislav Petkov X-Patchwork-Id: 4338031 Return-Path: X-Original-To: patchwork-linux-acpi@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id C62D99F3BC for ; Wed, 11 Jun 2014 19:01:51 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id D130C20256 for ; Wed, 11 Jun 2014 19:01:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 887002025B for ; Wed, 11 Jun 2014 19:01:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752652AbaFKTBR (ORCPT ); Wed, 11 Jun 2014 15:01:17 -0400 Received: from mail.skyhub.de ([78.46.96.112]:37641 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752572AbaFKTBQ (ORCPT ); Wed, 11 Jun 2014 15:01:16 -0400 X-Virus-Scanned: Nedap ESD1 at mail.skyhub.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alien8.de; s=alien8; t=1402513275; bh=BUymCOTQ/xLwknNx+RauDBwI22Zezy3o/q8mUJgrw/8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:In-Reply-To; b=UALyFhD9TI/eFQOLG4jCq7RU7BiLvihoZC8qeH ngg8E+7Bvn7YnPpxUPArmu9YEy9jZO5rEJVj4wJLUydGmZddC55V3S4Zkkc0UAdlySO Z5tChyhUtPkeV59Qfd17ra9KwDQCpLauPTzgedZOczbjWC/ZNBzX0LIiOk8PoBouB0= Received: from mail.skyhub.de ([127.0.0.1]) by localhost (door.skyhub.de [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id Rr4Pz1PDZE5Z; Wed, 11 Jun 2014 21:01:14 +0200 (CEST) Received: from liondog.tnic (p5DDC4B69.dip0.t-ipconnect.de [93.220.75.105]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.skyhub.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id 08FBC1DA26E; Wed, 11 Jun 2014 21:01:14 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alien8.de; s=alien8; t=1402513274; bh=BUymCOTQ/xLwknNx+RauDBwI22Zezy3o/q8mUJgrw/8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:In-Reply-To; b=m4br1f4a06Xh0JMhlk8TrhfmkxXVWLQoELAIXu 2rbijoNkrC4vrcoK45SmRoDrroFFjXEyjshK+aEjiRhiYmLm0m0UyaJVdCSv2K/DOWJ Q1GHAejFFMtdyH/fKe0f+cx7m6PTCA8UzhFSL/1As17HMCt7n29YHYtBAheBa0ip5A= Received: by liondog.tnic (Postfix, from userid 1000) id 19B8E10105E; Wed, 11 Jun 2014 21:01:13 +0200 (CEST) Date: Wed, 11 Jun 2014 21:01:13 +0200 From: Borislav Petkov To: "Chen, Gong" Cc: tony.luck@intel.com, m.chehab@samsung.com, rostedt@goodmis.org, linux-acpi@vger.kernel.org, lkml Subject: Re: [PATCH 4/7 v2] RAS, debugfs: Add debugfs interface for RAS subsystem Message-ID: <20140611190113.GD14923@pd.tnic> References: <1402475691-30045-1-git-send-email-gong.chen@linux.intel.com> <1402475691-30045-5-git-send-email-gong.chen@linux.intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1402475691-30045-5-git-send-email-gong.chen@linux.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-acpi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Wed, Jun 11, 2014 at 04:34:48AM -0400, Chen, Gong wrote: > Implement a new debugfs interface for RAS susbsystem. > A file named daemon_active is added there accordingly. > This file is used to track if user space daemon enables > perf/trace interface or not. One can track which daemon > opens it via "lsof /path/to/debugfs/ras/daemon_active". > > v2 -> v1: Change file access mode from 0444 to 0400. > > Signed-off-by: Chen, Gong > --- > drivers/ras/Makefile | 2 +- > drivers/ras/debugfs.c | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++ > drivers/ras/ras.c | 14 +++++++++++++ > include/linux/ras.h | 15 ++++++++++++++ > 4 files changed, 87 insertions(+), 1 deletion(-) > create mode 100644 drivers/ras/debugfs.c > create mode 100644 include/linux/ras.h > > diff --git a/drivers/ras/Makefile b/drivers/ras/Makefile > index 223e806..d7f7334 100644 > --- a/drivers/ras/Makefile > +++ b/drivers/ras/Makefile > @@ -1 +1 @@ > -obj-$(CONFIG_RAS) += ras.o > +obj-$(CONFIG_RAS) += ras.o debugfs.o > diff --git a/drivers/ras/debugfs.c b/drivers/ras/debugfs.c > new file mode 100644 > index 0000000..d0bc389 > --- /dev/null > +++ b/drivers/ras/debugfs.c > @@ -0,0 +1,57 @@ > +#include > + > +struct dentry *ras_debugfs_dir; > +EXPORT_SYMBOL_GPL(ras_debugfs_dir); No need to export this. Revised version below: --- From: "Chen, Gong" Implement a new debugfs interface for RAS susbsystem. A file named daemon_active is added there accordingly. This file is used to track if user space daemon accesses perf/trace interface or not. One can track which daemon opens it via "lsof /path/to/debugfs/ras/daemon_active". Signed-off-by: Chen, Gong Link: http://lkml.kernel.org/r/1402475691-30045-5-git-send-email-gong.chen@linux.intel.com Signed-off-by: Borislav Petkov --- drivers/ras/Makefile | 2 +- drivers/ras/debugfs.c | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/ras/ras.c | 14 +++++++++++++ include/linux/ras.h | 15 ++++++++++++++ 4 files changed, 87 insertions(+), 1 deletion(-) create mode 100644 drivers/ras/debugfs.c create mode 100644 include/linux/ras.h Index: linux/drivers/ras/Makefile =================================================================== --- linux.orig/drivers/ras/Makefile 2014-06-11 17:54:21.738397566 +0200 +++ linux/drivers/ras/Makefile 2014-06-11 17:54:21.726397566 +0200 @@ -1 +1 @@ -obj-$(CONFIG_RAS) += ras.o +obj-$(CONFIG_RAS) += ras.o debugfs.o Index: linux/drivers/ras/debugfs.c =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/drivers/ras/debugfs.c 2014-06-11 17:58:47.214393178 +0200 @@ -0,0 +1,56 @@ +#include + +static struct dentry *ras_debugfs_dir; + +static atomic_t trace_count = ATOMIC_INIT(0); + +int ras_userspace_consumers(void) +{ + return atomic_read(&trace_count); +} +EXPORT_SYMBOL_GPL(ras_userspace_consumers); + +static int trace_show(struct seq_file *m, void *v) +{ + return atomic_read(&trace_count); +} + +static int trace_open(struct inode *inode, struct file *file) +{ + atomic_inc(&trace_count); + return single_open(file, trace_show, NULL); +} + +static int trace_release(struct inode *inode, struct file *file) +{ + atomic_dec(&trace_count); + return single_release(inode, file); +} + +static const struct file_operations trace_fops = { + .open = trace_open, + .read = seq_read, + .llseek = seq_lseek, + .release = trace_release, +}; + +int __init ras_add_daemon_trace(void) +{ + struct dentry *fentry; + + if (!ras_debugfs_dir) + return -ENOENT; + + fentry = debugfs_create_file("daemon_active", S_IRUSR, ras_debugfs_dir, + NULL, &trace_fops); + if (!fentry) + return -ENODEV; + + return 0; + +} + +void __init ras_debugfs_init(void) +{ + ras_debugfs_dir = debugfs_create_dir("ras", NULL); +} Index: linux/drivers/ras/ras.c =================================================================== --- linux.orig/drivers/ras/ras.c 2014-06-11 17:54:21.738397566 +0200 +++ linux/drivers/ras/ras.c 2014-06-11 17:54:21.730397566 +0200 @@ -5,8 +5,22 @@ * Chen, Gong */ +#include +#include + #define CREATE_TRACE_POINTS #define TRACE_INCLUDE_PATH ../../include/ras #include +static int __init ras_init(void) +{ + int rc = 0; + + ras_debugfs_init(); + rc = ras_add_daemon_trace(); + + return rc; +} +subsys_initcall(ras_init); + EXPORT_TRACEPOINT_SYMBOL_GPL(mc_event); Index: linux/include/linux/ras.h =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/include/linux/ras.h 2014-06-11 17:58:43.350393242 +0200 @@ -0,0 +1,14 @@ +#ifndef __RAS_H__ +#define __RAS_H__ + +#ifdef CONFIG_DEBUG_FS +int ras_userspace_consumers(void); +void ras_debugfs_init(void); +int ras_add_daemon_trace(void); +#else +static inline int ras_userspace_consumers(void) { return 0; } +static inline void ras_debugfs_init(void) { return; } +static inline int ras_add_daemon_trace(void) { return 0; } +#endif + +#endif