Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/shaggy/jfs-2.6
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/shaggy/jfs-2.6: jfs: remove DIRENTSIZ JFS: diAlloc() should return -EIO rather than EIO JFS: skip bad iput() call in error path JFS: switch to seq_files JFS: 0 is not valid errno value so return NULL from jfs_lookup
This commit is contained in:
commit
e4e0fadcd9
|
@ -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;
|
||||
|
||||
return len;
|
||||
seq_printf(m, "%d\n", jfsloglevel);
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int loglevel_write(struct file *file, const char __user *buffer,
|
||||
unsigned long count, void *data)
|
||||
static int jfs_loglevel_proc_open(struct inode *inode, struct file *file)
|
||||
{
|
||||
return single_open(file, jfs_loglevel_proc_show, NULL);
|
||||
}
|
||||
|
||||
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)
|
||||
|
|
|
@ -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)--)
|
||||
|
|
|
@ -243,9 +243,6 @@ typedef union {
|
|||
#define JFS_REMOVE 3
|
||||
#define JFS_RENAME 4
|
||||
|
||||
#define DIRENTSIZ(namlen) \
|
||||
( (sizeof(struct dirent) - 2*(JFS_NAME_MAX+1) + 2*((namlen)+1) + 3) &~ 3 )
|
||||
|
||||
/*
|
||||
* Maximum file offset for directories.
|
||||
*/
|
||||
|
|
|
@ -1520,7 +1520,7 @@ int diAlloc(struct inode *pip, bool dir, struct inode *ip)
|
|||
jfs_error(ip->i_sb,
|
||||
"diAlloc: can't find free bit "
|
||||
"in wmap");
|
||||
return EIO;
|
||||
return -EIO;
|
||||
}
|
||||
|
||||
/* determine the inode number within the
|
||||
|
|
|
@ -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);
|
||||
|
||||
begin = offset;
|
||||
*start = buffer + begin;
|
||||
len -= begin;
|
||||
|
||||
if (len > length)
|
||||
len = length;
|
||||
else
|
||||
*eof = 1;
|
||||
|
||||
if (len < 0)
|
||||
len = 0;
|
||||
|
||||
return len;
|
||||
return 0;
|
||||
}
|
||||
|
||||
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 */
|
||||
|
|
|
@ -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);
|
||||
|
||||
begin = offset;
|
||||
*start = buffer + begin;
|
||||
len -= begin;
|
||||
|
||||
if (len > length)
|
||||
len = length;
|
||||
else
|
||||
*eof = 1;
|
||||
|
||||
if (len < 0)
|
||||
len = 0;
|
||||
|
||||
return len;
|
||||
return 0;
|
||||
}
|
||||
|
||||
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
|
||||
|
|
|
@ -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 ");
|
||||
|
||||
begin = offset;
|
||||
*start = buffer + begin;
|
||||
len -= begin;
|
||||
|
||||
if (len > length)
|
||||
len = length;
|
||||
else
|
||||
*eof = 1;
|
||||
|
||||
if (len < 0)
|
||||
len = 0;
|
||||
|
||||
return len;
|
||||
return 0;
|
||||
}
|
||||
|
||||
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);
|
||||
|
||||
begin = offset;
|
||||
*start = buffer + begin;
|
||||
len -= begin;
|
||||
|
||||
if (len > length)
|
||||
len = length;
|
||||
else
|
||||
*eof = 1;
|
||||
|
||||
if (len < 0)
|
||||
len = 0;
|
||||
|
||||
return len;
|
||||
return 0;
|
||||
}
|
||||
|
||||
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
|
||||
|
|
|
@ -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);
|
||||
|
||||
begin = offset;
|
||||
*start = buffer + begin;
|
||||
len -= begin;
|
||||
|
||||
if (len > length)
|
||||
len = length;
|
||||
else
|
||||
*eof = 1;
|
||||
|
||||
if (len < 0)
|
||||
len = 0;
|
||||
|
||||
return len;
|
||||
return 0;
|
||||
}
|
||||
|
||||
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
|
||||
|
|
|
@ -1455,7 +1455,7 @@ static struct dentry *jfs_lookup(struct inode *dip, struct dentry *dentry, struc
|
|||
free_UCSname(&key);
|
||||
if (rc == -ENOENT) {
|
||||
d_add(dentry, NULL);
|
||||
return ERR_PTR(0);
|
||||
return NULL;
|
||||
} else if (rc) {
|
||||
jfs_err("jfs_lookup: dtSearch returned %d", rc);
|
||||
return ERR_PTR(rc);
|
||||
|
|
|
@ -499,7 +499,7 @@ static int jfs_fill_super(struct super_block *sb, void *data, int silent)
|
|||
inode = jfs_iget(sb, ROOT_I);
|
||||
if (IS_ERR(inode)) {
|
||||
ret = PTR_ERR(inode);
|
||||
goto out_no_root;
|
||||
goto out_no_rw;
|
||||
}
|
||||
sb->s_root = d_alloc_root(inode);
|
||||
if (!sb->s_root)
|
||||
|
@ -521,9 +521,8 @@ static int jfs_fill_super(struct super_block *sb, void *data, int silent)
|
|||
return 0;
|
||||
|
||||
out_no_root:
|
||||
jfs_err("jfs_read_super: get root inode failed");
|
||||
if (inode)
|
||||
iput(inode);
|
||||
jfs_err("jfs_read_super: get root dentry failed");
|
||||
iput(inode);
|
||||
|
||||
out_no_rw:
|
||||
rc = jfs_umount(sb);
|
||||
|
|
Loading…
Reference in New Issue