From: Dave Kleikamp <shaggy@linux.vnet.ibm.com>
To: Alexey Dobriyan <adobriyan@gmail.com>
Cc: akpm@osdl.org, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH] jfs: switch to seq_files
Date: Tue, 13 May 2008 09:46:14 -0500 [thread overview]
Message-ID: <1210689974.6234.0.camel@norville.austin.ibm.com> (raw)
In-Reply-To: <20080509162325.GC8802@martell.zuzino.mipt.ru>
I've merged this into the jfs git tree.
On Fri, 2008-05-09 at 20:23 +0400, Alexey Dobriyan wrote:
> Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Dave Kleikamp <shaggy@linux.vnet.ibm.com>
> ---
>
> fs/jfs/jfs_debug.c | 62 ++++++++++++++++++++-------------------------
> fs/jfs/jfs_debug.h | 10 +++----
> fs/jfs/jfs_logmgr.c | 35 +++++++++++--------------
> fs/jfs/jfs_metapage.c | 36 ++++++++++++--------------
> fs/jfs/jfs_txnmgr.c | 68 ++++++++++++++++++++++----------------------------
> fs/jfs/jfs_xtree.c | 36 ++++++++++++--------------
> 6 files changed, 114 insertions(+), 133 deletions(-)
>
> --- a/fs/jfs/jfs_debug.c
> +++ b/fs/jfs/jfs_debug.c
> @@ -21,6 +21,7 @@
> #include <linux/ctype.h>
> #include <linux/module.h>
> #include <linux/proc_fs.h>
> +#include <linux/seq_file.h>
> #include <asm/uaccess.h>
> #include "jfs_incore.h"
> #include "jfs_filsys.h"
> @@ -30,29 +31,19 @@
>
> static struct proc_dir_entry *base;
> #ifdef CONFIG_JFS_DEBUG
> -static int loglevel_read(char *page, char **start, off_t off,
> - int count, int *eof, void *data)
> +static int jfs_loglevel_proc_show(struct seq_file *m, void *v)
> {
> - int len;
> -
> - len = sprintf(page, "%d\n", jfsloglevel);
> -
> - len -= off;
> - *start = page + off;
> -
> - if (len > count)
> - len = count;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> + seq_printf(m, "%d\n", jfsloglevel);
> + return 0;
> +}
>
> - return len;
> +static int jfs_loglevel_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_loglevel_proc_show, NULL);
> }
>
> -static int loglevel_write(struct file *file, const char __user *buffer,
> - unsigned long count, void *data)
> +static ssize_t jfs_loglevel_proc_write(struct file *file,
> + const char __user *buffer, size_t count, loff_t *ppos)
> {
> char c;
>
> @@ -65,22 +56,30 @@ static int loglevel_write(struct file *file, const char __user *buffer,
> jfsloglevel = c - '0';
> return count;
> }
> +
> +static const struct file_operations jfs_loglevel_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_loglevel_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> + .write = jfs_loglevel_proc_write,
> +};
> #endif
>
> static struct {
> const char *name;
> - read_proc_t *read_fn;
> - write_proc_t *write_fn;
> + const struct file_operations *proc_fops;
> } Entries[] = {
> #ifdef CONFIG_JFS_STATISTICS
> - { "lmstats", jfs_lmstats_read, },
> - { "txstats", jfs_txstats_read, },
> - { "xtstat", jfs_xtstat_read, },
> - { "mpstat", jfs_mpstat_read, },
> + { "lmstats", &jfs_lmstats_proc_fops, },
> + { "txstats", &jfs_txstats_proc_fops, },
> + { "xtstat", &jfs_xtstat_proc_fops, },
> + { "mpstat", &jfs_mpstat_proc_fops, },
> #endif
> #ifdef CONFIG_JFS_DEBUG
> - { "TxAnchor", jfs_txanchor_read, },
> - { "loglevel", loglevel_read, loglevel_write }
> + { "TxAnchor", &jfs_txanchor_proc_fops, },
> + { "loglevel", &jfs_loglevel_proc_fops }
> #endif
> };
> #define NPROCENT ARRAY_SIZE(Entries)
> @@ -93,13 +92,8 @@ void jfs_proc_init(void)
> return;
> base->owner = THIS_MODULE;
>
> - for (i = 0; i < NPROCENT; i++) {
> - struct proc_dir_entry *p;
> - if ((p = create_proc_entry(Entries[i].name, 0, base))) {
> - p->read_proc = Entries[i].read_fn;
> - p->write_proc = Entries[i].write_fn;
> - }
> - }
> + for (i = 0; i < NPROCENT; i++)
> + proc_create(Entries[i].name, 0, base, Entries[i].proc_fops);
> }
>
> void jfs_proc_clean(void)
> --- a/fs/jfs/jfs_debug.h
> +++ b/fs/jfs/jfs_debug.h
> @@ -62,7 +62,7 @@ extern void jfs_proc_clean(void);
>
> extern int jfsloglevel;
>
> -extern int jfs_txanchor_read(char *, char **, off_t, int, int *, void *);
> +extern const struct file_operations jfs_txanchor_proc_fops;
>
> /* information message: e.g., configuration, major event */
> #define jfs_info(fmt, arg...) do { \
> @@ -105,10 +105,10 @@ extern int jfs_txanchor_read(char *, char **, off_t, int, int *, void *);
> * ----------
> */
> #ifdef CONFIG_JFS_STATISTICS
> -extern int jfs_lmstats_read(char *, char **, off_t, int, int *, void *);
> -extern int jfs_txstats_read(char *, char **, off_t, int, int *, void *);
> -extern int jfs_mpstat_read(char *, char **, off_t, int, int *, void *);
> -extern int jfs_xtstat_read(char *, char **, off_t, int, int *, void *);
> +extern const struct file_operations jfs_lmstats_proc_fops;
> +extern const struct file_operations jfs_txstats_proc_fops;
> +extern const struct file_operations jfs_mpstat_proc_fops;
> +extern const struct file_operations jfs_xtstat_proc_fops;
>
> #define INCREMENT(x) ((x)++)
> #define DECREMENT(x) ((x)--)
> --- a/fs/jfs/jfs_logmgr.c
> +++ b/fs/jfs/jfs_logmgr.c
> @@ -69,6 +69,7 @@
> #include <linux/freezer.h>
> #include <linux/delay.h>
> #include <linux/mutex.h>
> +#include <linux/seq_file.h>
> #include "jfs_incore.h"
> #include "jfs_filsys.h"
> #include "jfs_metapage.h"
> @@ -2503,13 +2504,9 @@ exit:
> }
>
> #ifdef CONFIG_JFS_STATISTICS
> -int jfs_lmstats_read(char *buffer, char **start, off_t offset, int length,
> - int *eof, void *data)
> +static int jfs_lmstats_proc_show(struct seq_file *m, void *v)
> {
> - int len = 0;
> - off_t begin;
> -
> - len += sprintf(buffer,
> + seq_printf(m,
> "JFS Logmgr stats\n"
> "================\n"
> "commits = %d\n"
> @@ -2522,19 +2519,19 @@ int jfs_lmstats_read(char *buffer, char **start, off_t offset, int length,
> lmStat.pagedone,
> lmStat.full_page,
> lmStat.partial_page);
> + return 0;
> +}
>
> - begin = offset;
> - *start = buffer + begin;
> - len -= begin;
> -
> - if (len > length)
> - len = length;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> -
> - return len;
> +static int jfs_lmstats_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_lmstats_proc_show, NULL);
> }
> +
> +const struct file_operations jfs_lmstats_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_lmstats_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> #endif /* CONFIG_JFS_STATISTICS */
> --- a/fs/jfs/jfs_metapage.c
> +++ b/fs/jfs/jfs_metapage.c
> @@ -19,10 +19,12 @@
>
> #include <linux/fs.h>
> #include <linux/mm.h>
> +#include <linux/module.h>
> #include <linux/bio.h>
> #include <linux/init.h>
> #include <linux/buffer_head.h>
> #include <linux/mempool.h>
> +#include <linux/seq_file.h>
> #include "jfs_incore.h"
> #include "jfs_superblock.h"
> #include "jfs_filsys.h"
> @@ -804,13 +806,9 @@ void __invalidate_metapages(struct inode *ip, s64 addr, int len)
> }
>
> #ifdef CONFIG_JFS_STATISTICS
> -int jfs_mpstat_read(char *buffer, char **start, off_t offset, int length,
> - int *eof, void *data)
> +static int jfs_mpstat_proc_show(struct seq_file *m, void *v)
> {
> - int len = 0;
> - off_t begin;
> -
> - len += sprintf(buffer,
> + seq_printf(m,
> "JFS Metapage statistics\n"
> "=======================\n"
> "page allocations = %d\n"
> @@ -819,19 +817,19 @@ int jfs_mpstat_read(char *buffer, char **start, off_t offset, int length,
> mpStat.pagealloc,
> mpStat.pagefree,
> mpStat.lockwait);
> + return 0;
> +}
>
> - begin = offset;
> - *start = buffer + begin;
> - len -= begin;
> -
> - if (len > length)
> - len = length;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> -
> - return len;
> +static int jfs_mpstat_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_mpstat_proc_show, NULL);
> }
> +
> +const struct file_operations jfs_mpstat_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_mpstat_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> #endif
> --- a/fs/jfs/jfs_txnmgr.c
> +++ b/fs/jfs/jfs_txnmgr.c
> @@ -49,6 +49,7 @@
> #include <linux/module.h>
> #include <linux/moduleparam.h>
> #include <linux/kthread.h>
> +#include <linux/seq_file.h>
> #include "jfs_incore.h"
> #include "jfs_inode.h"
> #include "jfs_filsys.h"
> @@ -3009,11 +3010,8 @@ int jfs_sync(void *arg)
> }
>
> #if defined(CONFIG_PROC_FS) && defined(CONFIG_JFS_DEBUG)
> -int jfs_txanchor_read(char *buffer, char **start, off_t offset, int length,
> - int *eof, void *data)
> +static int jfs_txanchor_proc_show(struct seq_file *m, void *v)
> {
> - int len = 0;
> - off_t begin;
> char *freewait;
> char *freelockwait;
> char *lowlockwait;
> @@ -3025,7 +3023,7 @@ int jfs_txanchor_read(char *buffer, char **start, off_t offset, int length,
> lowlockwait =
> waitqueue_active(&TxAnchor.lowlockwait) ? "active" : "empty";
>
> - len += sprintf(buffer,
> + seq_printf(m,
> "JFS TxAnchor\n"
> "============\n"
> "freetid = %d\n"
> @@ -3044,31 +3042,27 @@ int jfs_txanchor_read(char *buffer, char **start, off_t offset, int length,
> TxAnchor.tlocksInUse,
> jfs_tlocks_low,
> list_empty(&TxAnchor.unlock_queue) ? "" : "not ");
> + return 0;
> +}
>
> - begin = offset;
> - *start = buffer + begin;
> - len -= begin;
> -
> - if (len > length)
> - len = length;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> -
> - return len;
> +static int jfs_txanchor_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_txanchor_proc_show, NULL);
> }
> +
> +const struct file_operations jfs_txanchor_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_txanchor_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> #endif
>
> #if defined(CONFIG_PROC_FS) && defined(CONFIG_JFS_STATISTICS)
> -int jfs_txstats_read(char *buffer, char **start, off_t offset, int length,
> - int *eof, void *data)
> +static int jfs_txstats_proc_show(struct seq_file *m, void *v)
> {
> - int len = 0;
> - off_t begin;
> -
> - len += sprintf(buffer,
> + seq_printf(m,
> "JFS TxStats\n"
> "===========\n"
> "calls to txBegin = %d\n"
> @@ -3089,19 +3083,19 @@ int jfs_txstats_read(char *buffer, char **start, off_t offset, int length,
> TxStat.txBeginAnon_lockslow,
> TxStat.txLockAlloc,
> TxStat.txLockAlloc_freelock);
> + return 0;
> +}
>
> - begin = offset;
> - *start = buffer + begin;
> - len -= begin;
> -
> - if (len > length)
> - len = length;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> -
> - return len;
> +static int jfs_txstats_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_txstats_proc_show, NULL);
> }
> +
> +const struct file_operations jfs_txstats_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_txstats_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> #endif
> --- a/fs/jfs/jfs_xtree.c
> +++ b/fs/jfs/jfs_xtree.c
> @@ -20,7 +20,9 @@
> */
>
> #include <linux/fs.h>
> +#include <linux/module.h>
> #include <linux/quotaops.h>
> +#include <linux/seq_file.h>
> #include "jfs_incore.h"
> #include "jfs_filsys.h"
> #include "jfs_metapage.h"
> @@ -4134,13 +4136,9 @@ s64 xtTruncate_pmap(tid_t tid, struct inode *ip, s64 committed_size)
> }
>
> #ifdef CONFIG_JFS_STATISTICS
> -int jfs_xtstat_read(char *buffer, char **start, off_t offset, int length,
> - int *eof, void *data)
> +static int jfs_xtstat_proc_show(struct seq_file *m, void *v)
> {
> - int len = 0;
> - off_t begin;
> -
> - len += sprintf(buffer,
> + seq_printf(m,
> "JFS Xtree statistics\n"
> "====================\n"
> "searches = %d\n"
> @@ -4149,19 +4147,19 @@ int jfs_xtstat_read(char *buffer, char **start, off_t offset, int length,
> xtStat.search,
> xtStat.fastSearch,
> xtStat.split);
> + return 0;
> +}
>
> - begin = offset;
> - *start = buffer + begin;
> - len -= begin;
> -
> - if (len > length)
> - len = length;
> - else
> - *eof = 1;
> -
> - if (len < 0)
> - len = 0;
> -
> - return len;
> +static int jfs_xtstat_proc_open(struct inode *inode, struct file *file)
> +{
> + return single_open(file, jfs_xtstat_proc_show, NULL);
> }
> +
> +const struct file_operations jfs_xtstat_proc_fops = {
> + .owner = THIS_MODULE,
> + .open = jfs_xtstat_proc_open,
> + .read = seq_read,
> + .llseek = seq_lseek,
> + .release = single_release,
> +};
> #endif
--
David Kleikamp
IBM Linux Technology Center
prev parent reply other threads:[~2008-05-13 14:46 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-09 16:23 [PATCH] jfs: switch to seq_files Alexey Dobriyan
2008-05-09 15:56 ` Dave Kleikamp
2008-05-13 14:46 ` Dave Kleikamp [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1210689974.6234.0.camel@norville.austin.ibm.com \
--to=shaggy@linux.vnet.ibm.com \
--cc=adobriyan@gmail.com \
--cc=akpm@osdl.org \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).