From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932975AbbIDO0k (ORCPT ); Fri, 4 Sep 2015 10:26:40 -0400 Received: from mx1.redhat.com ([209.132.183.28]:53526 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932767AbbIDO0i (ORCPT ); Fri, 4 Sep 2015 10:26:38 -0400 Date: Fri, 4 Sep 2015 16:26:31 +0200 From: Jiri Olsa To: =?iso-8859-1?Q?Rapha=EBl?= Beamonte Cc: Jiri Olsa , Arnaldo Carvalho de Melo , lkml , David Ahern , Ingo Molnar , Namhyung Kim , Peter Zijlstra , Matt Fleming , Steven Rostedt Subject: Re: [PATCH 11/15] tools lib api: Add mount support for fs Message-ID: <20150904142631.GB658@krava.redhat.com> References: <1441180605-24737-1-git-send-email-jolsa@kernel.org> <1441180605-24737-12-git-send-email-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 04, 2015 at 10:14:28AM -0400, Raphaël Beamonte wrote: > 2015-09-02 3:56 GMT-04:00 Jiri Olsa : > > Adding name__mount (where name is in sysfs,procfs,debugfs,tracefs) > > interface that tries to mount the filesystem in case no mount is found. > > > > Link: http://lkml.kernel.org/n/tip-ja49vwfiq2qqkmoxx9yk26lm@git.kernel.org > > Signed-off-by: Jiri Olsa > > --- > > tools/lib/api/fs/fs.c | 44 +++++++++++++++++++++++++++++++++++++++----- > > tools/lib/api/fs/fs.h | 15 +++++++++++---- > > 2 files changed, 50 insertions(+), 9 deletions(-) > > Why automatic mounting the **fs filesystems? > Isn't it better to let the user decide when and where to mount it if > s/he forgot to do it before? Forcing a mount is also potentially > forcing to umount then mount when mounting has been forgotten before. well, I guess at some point we decided this was a good idea to make perf more user friendly AFAICS we do this only for debugfs and tracefs.. procfs and sysfs are mounted most of the time anyway we could track and unmount on exit, but I'd say let's have someone complaining about it first ;-) jirka > > > > diff --git a/tools/lib/api/fs/fs.c b/tools/lib/api/fs/fs.c > > index ef16d2a83a27..bc93baf33fff 100644 > > --- a/tools/lib/api/fs/fs.c > > +++ b/tools/lib/api/fs/fs.c > > @@ -9,6 +9,7 @@ > > #include > > #include > > #include > > +#include > > > > #include "debugfs.h" > > #include "fs.h" > > @@ -215,16 +216,49 @@ static const char *fs__mountpoint(int idx) > > return fs__get_mountpoint(fs); > > } > > > > -#define FS__MOUNTPOINT(name, idx) \ > > +static const char *mount_overload(struct fs *fs) > > +{ > > + size_t name_len = strlen(fs->name); > > + /* "PERF_" + name + "_ENVIRONMENT" + '\0' */ > > + char upper_name[5 + name_len + 12 + 1]; > > + > > + snprintf(upper_name, name_len, "PERF_%s_ENVIRONMENT", fs->name); > > + mem_toupper(upper_name, name_len); > > + > > + return getenv(upper_name) ?: *fs->mounts; > > +} > > + > > +static const char *fs__mount(int idx) > > +{ > > + struct fs *fs = &fs__entries[idx]; > > + const char *mountpoint; > > + > > + if (fs__mountpoint(idx)) > > + return (const char *)fs->path; > > + > > + mountpoint = mount_overload(fs); > > + > > + if (mount(NULL, mountpoint, fs->name, 0, NULL) < 0) > > + return NULL; > > + > > + return fs__check_mounts(fs) ? fs->path : NULL; > > +} > > + > > +#define FS(name, idx) \ > > const char *name##__mountpoint(void) \ > > { \ > > return fs__mountpoint(idx); \ > > +} \ > > + \ > > +const char *name##__mount(void) \ > > +{ \ > > + return fs__mount(idx); \ > > } > > > > -FS__MOUNTPOINT(sysfs, FS__SYSFS); > > -FS__MOUNTPOINT(procfs, FS__PROCFS); > > -FS__MOUNTPOINT(debugfs, FS__DEBUGFS); > > -FS__MOUNTPOINT(tracefs, FS__TRACEFS); > > +FS(sysfs, FS__SYSFS); > > +FS(procfs, FS__PROCFS); > > +FS(debugfs, FS__DEBUGFS); > > +FS(tracefs, FS__TRACEFS); > > > > int filename__read_int(const char *filename, int *value) > > { > > diff --git a/tools/lib/api/fs/fs.h b/tools/lib/api/fs/fs.h > > index 9013227ae0d1..a9627ea5e6ae 100644 > > --- a/tools/lib/api/fs/fs.h > > +++ b/tools/lib/api/fs/fs.h > > @@ -9,10 +9,17 @@ > > #define PATH_MAX 4096 > > #endif > > > > -const char *sysfs__mountpoint(void); > > -const char *procfs__mountpoint(void); > > -const char *debugfs__mountpoint(void); > > -const char *tracefs__mountpoint(void); > > +#define FS(name) \ > > + const char *name##__mountpoint(void); \ > > + const char *name##__mount(void); > > + > > +FS(sysfs) > > +FS(procfs) > > +FS(debugfs) > > +FS(tracefs) > > + > > +#undef FS > > + > > > > int filename__read_int(const char *filename, int *value); > > int sysctl__read_int(const char *sysctl, int *value); > > -- > > 2.4.3 > >