don't pass superblock to hypfs_{mkdir,create*}

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
Al Viro 2013-07-19 18:05:21 +04:00
parent e334cf4fe8
commit a118cfdf99
4 changed files with 39 additions and 46 deletions

View File

@ -18,15 +18,12 @@
#define UPDATE_FILE_MODE 0220 #define UPDATE_FILE_MODE 0220
#define DIR_MODE 0550 #define DIR_MODE 0550
extern struct dentry *hypfs_mkdir(struct super_block *sb, struct dentry *parent, extern struct dentry *hypfs_mkdir(struct dentry *parent, const char *name);
const char *name);
extern struct dentry *hypfs_create_u64(struct super_block *sb, extern struct dentry *hypfs_create_u64(struct dentry *dir, const char *name,
struct dentry *dir, const char *name,
__u64 value); __u64 value);
extern struct dentry *hypfs_create_str(struct super_block *sb, extern struct dentry *hypfs_create_str(struct dentry *dir, const char *name,
struct dentry *dir, const char *name,
char *string); char *string);
/* LPAR Hypervisor */ /* LPAR Hypervisor */

View File

@ -631,25 +631,25 @@ static int hypfs_create_cpu_files(struct dentry *cpus_dir, void *cpu_info)
snprintf(buffer, TMP_SIZE, "%d", cpu_info__cpu_addr(diag204_info_type, snprintf(buffer, TMP_SIZE, "%d", cpu_info__cpu_addr(diag204_info_type,
cpu_info)); cpu_info));
cpu_dir = hypfs_mkdir(cpus_dir->d_sb, cpus_dir, buffer); cpu_dir = hypfs_mkdir(cpus_dir, buffer);
rc = hypfs_create_u64(cpu_dir->d_sb, cpu_dir, "mgmtime", rc = hypfs_create_u64(cpu_dir, "mgmtime",
cpu_info__acc_time(diag204_info_type, cpu_info) - cpu_info__acc_time(diag204_info_type, cpu_info) -
cpu_info__lp_time(diag204_info_type, cpu_info)); cpu_info__lp_time(diag204_info_type, cpu_info));
if (IS_ERR(rc)) if (IS_ERR(rc))
return PTR_ERR(rc); return PTR_ERR(rc);
rc = hypfs_create_u64(cpu_dir->d_sb, cpu_dir, "cputime", rc = hypfs_create_u64(cpu_dir, "cputime",
cpu_info__lp_time(diag204_info_type, cpu_info)); cpu_info__lp_time(diag204_info_type, cpu_info));
if (IS_ERR(rc)) if (IS_ERR(rc))
return PTR_ERR(rc); return PTR_ERR(rc);
if (diag204_info_type == INFO_EXT) { if (diag204_info_type == INFO_EXT) {
rc = hypfs_create_u64(cpu_dir->d_sb, cpu_dir, "onlinetime", rc = hypfs_create_u64(cpu_dir, "onlinetime",
cpu_info__online_time(diag204_info_type, cpu_info__online_time(diag204_info_type,
cpu_info)); cpu_info));
if (IS_ERR(rc)) if (IS_ERR(rc))
return PTR_ERR(rc); return PTR_ERR(rc);
} }
diag224_idx2name(cpu_info__ctidx(diag204_info_type, cpu_info), buffer); diag224_idx2name(cpu_info__ctidx(diag204_info_type, cpu_info), buffer);
rc = hypfs_create_str(cpu_dir->d_sb, cpu_dir, "type", buffer); rc = hypfs_create_str(cpu_dir, "type", buffer);
return PTR_RET(rc); return PTR_RET(rc);
} }
@ -663,10 +663,10 @@ static void *hypfs_create_lpar_files(struct dentry *systems_dir, void *part_hdr)
part_hdr__part_name(diag204_info_type, part_hdr, lpar_name); part_hdr__part_name(diag204_info_type, part_hdr, lpar_name);
lpar_name[LPAR_NAME_LEN] = 0; lpar_name[LPAR_NAME_LEN] = 0;
lpar_dir = hypfs_mkdir(systems_dir->d_sb, systems_dir, lpar_name); lpar_dir = hypfs_mkdir(systems_dir, lpar_name);
if (IS_ERR(lpar_dir)) if (IS_ERR(lpar_dir))
return lpar_dir; return lpar_dir;
cpus_dir = hypfs_mkdir(lpar_dir->d_sb, lpar_dir, "cpus"); cpus_dir = hypfs_mkdir(lpar_dir, "cpus");
if (IS_ERR(cpus_dir)) if (IS_ERR(cpus_dir))
return cpus_dir; return cpus_dir;
cpu_info = part_hdr + part_hdr__size(diag204_info_type); cpu_info = part_hdr + part_hdr__size(diag204_info_type);
@ -688,15 +688,15 @@ static int hypfs_create_phys_cpu_files(struct dentry *cpus_dir, void *cpu_info)
snprintf(buffer, TMP_SIZE, "%i", phys_cpu__cpu_addr(diag204_info_type, snprintf(buffer, TMP_SIZE, "%i", phys_cpu__cpu_addr(diag204_info_type,
cpu_info)); cpu_info));
cpu_dir = hypfs_mkdir(cpus_dir->d_sb, cpus_dir, buffer); cpu_dir = hypfs_mkdir(cpus_dir, buffer);
if (IS_ERR(cpu_dir)) if (IS_ERR(cpu_dir))
return PTR_ERR(cpu_dir); return PTR_ERR(cpu_dir);
rc = hypfs_create_u64(cpu_dir->d_sb, cpu_dir, "mgmtime", rc = hypfs_create_u64(cpu_dir, "mgmtime",
phys_cpu__mgm_time(diag204_info_type, cpu_info)); phys_cpu__mgm_time(diag204_info_type, cpu_info));
if (IS_ERR(rc)) if (IS_ERR(rc))
return PTR_ERR(rc); return PTR_ERR(rc);
diag224_idx2name(phys_cpu__ctidx(diag204_info_type, cpu_info), buffer); diag224_idx2name(phys_cpu__ctidx(diag204_info_type, cpu_info), buffer);
rc = hypfs_create_str(cpu_dir->d_sb, cpu_dir, "type", buffer); rc = hypfs_create_str(cpu_dir, "type", buffer);
return PTR_RET(rc); return PTR_RET(rc);
} }
@ -706,7 +706,7 @@ static void *hypfs_create_phys_files(struct dentry *parent_dir, void *phys_hdr)
void *cpu_info; void *cpu_info;
struct dentry *cpus_dir; struct dentry *cpus_dir;
cpus_dir = hypfs_mkdir(parent_dir->d_sb, parent_dir, "cpus"); cpus_dir = hypfs_mkdir(parent_dir, "cpus");
if (IS_ERR(cpus_dir)) if (IS_ERR(cpus_dir))
return cpus_dir; return cpus_dir;
cpu_info = phys_hdr + phys_hdr__size(diag204_info_type); cpu_info = phys_hdr + phys_hdr__size(diag204_info_type);
@ -731,7 +731,7 @@ int hypfs_diag_create_files(struct dentry *root)
if (IS_ERR(buffer)) if (IS_ERR(buffer))
return PTR_ERR(buffer); return PTR_ERR(buffer);
systems_dir = hypfs_mkdir(root->d_sb, root, "systems"); systems_dir = hypfs_mkdir(root, "systems");
if (IS_ERR(systems_dir)) { if (IS_ERR(systems_dir)) {
rc = PTR_ERR(systems_dir); rc = PTR_ERR(systems_dir);
goto err_out; goto err_out;
@ -752,12 +752,12 @@ int hypfs_diag_create_files(struct dentry *root)
goto err_out; goto err_out;
} }
} }
hyp_dir = hypfs_mkdir(root->d_sb, root, "hyp"); hyp_dir = hypfs_mkdir(root, "hyp");
if (IS_ERR(hyp_dir)) { if (IS_ERR(hyp_dir)) {
rc = PTR_ERR(hyp_dir); rc = PTR_ERR(hyp_dir);
goto err_out; goto err_out;
} }
ptr = hypfs_create_str(root->d_sb, hyp_dir, "type", "LPAR Hypervisor"); ptr = hypfs_create_str(hyp_dir, "type", "LPAR Hypervisor");
if (IS_ERR(ptr)) { if (IS_ERR(ptr)) {
rc = PTR_ERR(ptr); rc = PTR_ERR(ptr);
goto err_out; goto err_out;

View File

@ -110,7 +110,7 @@ static void diag2fc_free(const void *data)
#define ATTRIBUTE(dir, name, member) \ #define ATTRIBUTE(dir, name, member) \
do { \ do { \
void *rc; \ void *rc; \
rc = hypfs_create_u64(dir->d_sb, dir, name, member); \ rc = hypfs_create_u64(dir, name, member); \
if (IS_ERR(rc)) \ if (IS_ERR(rc)) \
return PTR_ERR(rc); \ return PTR_ERR(rc); \
} while(0) } while(0)
@ -129,13 +129,13 @@ static int hpyfs_vm_create_guest(struct dentry *systems_dir,
memcpy(guest_name, data->guest_name, NAME_LEN); memcpy(guest_name, data->guest_name, NAME_LEN);
EBCASC(guest_name, NAME_LEN); EBCASC(guest_name, NAME_LEN);
strim(guest_name); strim(guest_name);
guest_dir = hypfs_mkdir(systems_dir->d_sb, systems_dir, guest_name); guest_dir = hypfs_mkdir(systems_dir, guest_name);
if (IS_ERR(guest_dir)) if (IS_ERR(guest_dir))
return PTR_ERR(guest_dir); return PTR_ERR(guest_dir);
ATTRIBUTE(guest_dir, "onlinetime_us", data->el_time); ATTRIBUTE(guest_dir, "onlinetime_us", data->el_time);
/* logical cpu information */ /* logical cpu information */
cpus_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "cpus"); cpus_dir = hypfs_mkdir(guest_dir, "cpus");
if (IS_ERR(cpus_dir)) if (IS_ERR(cpus_dir))
return PTR_ERR(cpus_dir); return PTR_ERR(cpus_dir);
ATTRIBUTE(cpus_dir, "cputime_us", data->used_cpu); ATTRIBUTE(cpus_dir, "cputime_us", data->used_cpu);
@ -147,7 +147,7 @@ static int hpyfs_vm_create_guest(struct dentry *systems_dir,
ATTRIBUTE(cpus_dir, "weight_cur", data->cpu_shares); ATTRIBUTE(cpus_dir, "weight_cur", data->cpu_shares);
/* memory information */ /* memory information */
mem_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "mem"); mem_dir = hypfs_mkdir(guest_dir, "mem");
if (IS_ERR(mem_dir)) if (IS_ERR(mem_dir))
return PTR_ERR(mem_dir); return PTR_ERR(mem_dir);
ATTRIBUTE(mem_dir, "min_KiB", data->mem_min_kb); ATTRIBUTE(mem_dir, "min_KiB", data->mem_min_kb);
@ -156,7 +156,7 @@ static int hpyfs_vm_create_guest(struct dentry *systems_dir,
ATTRIBUTE(mem_dir, "share_KiB", data->mem_share_kb); ATTRIBUTE(mem_dir, "share_KiB", data->mem_share_kb);
/* samples */ /* samples */
samples_dir = hypfs_mkdir(guest_dir->d_sb, guest_dir, "samples"); samples_dir = hypfs_mkdir(guest_dir, "samples");
if (IS_ERR(samples_dir)) if (IS_ERR(samples_dir))
return PTR_ERR(samples_dir); return PTR_ERR(samples_dir);
ATTRIBUTE(samples_dir, "cpu_using", data->cpu_use_samp); ATTRIBUTE(samples_dir, "cpu_using", data->cpu_use_samp);
@ -180,31 +180,31 @@ int hypfs_vm_create_files(struct dentry *root)
return PTR_ERR(data); return PTR_ERR(data);
/* Hpervisor Info */ /* Hpervisor Info */
dir = hypfs_mkdir(root->d_sb, root, "hyp"); dir = hypfs_mkdir(root, "hyp");
if (IS_ERR(dir)) { if (IS_ERR(dir)) {
rc = PTR_ERR(dir); rc = PTR_ERR(dir);
goto failed; goto failed;
} }
file = hypfs_create_str(root->d_sb, dir, "type", "z/VM Hypervisor"); file = hypfs_create_str(dir, "type", "z/VM Hypervisor");
if (IS_ERR(file)) { if (IS_ERR(file)) {
rc = PTR_ERR(file); rc = PTR_ERR(file);
goto failed; goto failed;
} }
/* physical cpus */ /* physical cpus */
dir = hypfs_mkdir(root->d_sb, root, "cpus"); dir = hypfs_mkdir(root, "cpus");
if (IS_ERR(dir)) { if (IS_ERR(dir)) {
rc = PTR_ERR(dir); rc = PTR_ERR(dir);
goto failed; goto failed;
} }
file = hypfs_create_u64(root->d_sb, dir, "count", data->lcpus); file = hypfs_create_u64(dir, "count", data->lcpus);
if (IS_ERR(file)) { if (IS_ERR(file)) {
rc = PTR_ERR(file); rc = PTR_ERR(file);
goto failed; goto failed;
} }
/* guests */ /* guests */
dir = hypfs_mkdir(root->d_sb, root, "systems"); dir = hypfs_mkdir(root, "systems");
if (IS_ERR(dir)) { if (IS_ERR(dir)) {
rc = PTR_ERR(dir); rc = PTR_ERR(dir);
goto failed; goto failed;

View File

@ -28,8 +28,7 @@
#define HYPFS_MAGIC 0x687970 /* ASCII 'hyp' */ #define HYPFS_MAGIC 0x687970 /* ASCII 'hyp' */
#define TMP_SIZE 64 /* size of temporary buffers */ #define TMP_SIZE 64 /* size of temporary buffers */
static struct dentry *hypfs_create_update_file(struct super_block *sb, static struct dentry *hypfs_create_update_file(struct dentry *dir);
struct dentry *dir);
struct hypfs_sb_info { struct hypfs_sb_info {
kuid_t uid; /* uid used for files and dirs */ kuid_t uid; /* uid used for files and dirs */
@ -307,7 +306,7 @@ static int hypfs_fill_super(struct super_block *sb, void *data, int silent)
rc = hypfs_diag_create_files(root_dentry); rc = hypfs_diag_create_files(root_dentry);
if (rc) if (rc)
return rc; return rc;
sbi->update_file = hypfs_create_update_file(sb, root_dentry); sbi->update_file = hypfs_create_update_file(root_dentry);
if (IS_ERR(sbi->update_file)) if (IS_ERR(sbi->update_file))
return PTR_ERR(sbi->update_file); return PTR_ERR(sbi->update_file);
hypfs_update_update(sb); hypfs_update_update(sb);
@ -334,8 +333,7 @@ static void hypfs_kill_super(struct super_block *sb)
kill_litter_super(sb); kill_litter_super(sb);
} }
static struct dentry *hypfs_create_file(struct super_block *sb, static struct dentry *hypfs_create_file(struct dentry *parent, const char *name,
struct dentry *parent, const char *name,
char *data, umode_t mode) char *data, umode_t mode)
{ {
struct dentry *dentry; struct dentry *dentry;
@ -347,7 +345,7 @@ static struct dentry *hypfs_create_file(struct super_block *sb,
dentry = ERR_PTR(-ENOMEM); dentry = ERR_PTR(-ENOMEM);
goto fail; goto fail;
} }
inode = hypfs_make_inode(sb, mode); inode = hypfs_make_inode(parent->d_sb, mode);
if (!inode) { if (!inode) {
dput(dentry); dput(dentry);
dentry = ERR_PTR(-ENOMEM); dentry = ERR_PTR(-ENOMEM);
@ -373,24 +371,22 @@ fail:
return dentry; return dentry;
} }
struct dentry *hypfs_mkdir(struct super_block *sb, struct dentry *parent, struct dentry *hypfs_mkdir(struct dentry *parent, const char *name)
const char *name)
{ {
struct dentry *dentry; struct dentry *dentry;
dentry = hypfs_create_file(sb, parent, name, NULL, S_IFDIR | DIR_MODE); dentry = hypfs_create_file(parent, name, NULL, S_IFDIR | DIR_MODE);
if (IS_ERR(dentry)) if (IS_ERR(dentry))
return dentry; return dentry;
hypfs_add_dentry(dentry); hypfs_add_dentry(dentry);
return dentry; return dentry;
} }
static struct dentry *hypfs_create_update_file(struct super_block *sb, static struct dentry *hypfs_create_update_file(struct dentry *dir)
struct dentry *dir)
{ {
struct dentry *dentry; struct dentry *dentry;
dentry = hypfs_create_file(sb, dir, "update", NULL, dentry = hypfs_create_file(dir, "update", NULL,
S_IFREG | UPDATE_FILE_MODE); S_IFREG | UPDATE_FILE_MODE);
/* /*
* We do not put the update file on the 'delete' list with * We do not put the update file on the 'delete' list with
@ -400,7 +396,7 @@ static struct dentry *hypfs_create_update_file(struct super_block *sb,
return dentry; return dentry;
} }
struct dentry *hypfs_create_u64(struct super_block *sb, struct dentry *dir, struct dentry *hypfs_create_u64(struct dentry *dir,
const char *name, __u64 value) const char *name, __u64 value)
{ {
char *buffer; char *buffer;
@ -412,7 +408,7 @@ struct dentry *hypfs_create_u64(struct super_block *sb, struct dentry *dir,
if (!buffer) if (!buffer)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
dentry = dentry =
hypfs_create_file(sb, dir, name, buffer, S_IFREG | REG_FILE_MODE); hypfs_create_file(dir, name, buffer, S_IFREG | REG_FILE_MODE);
if (IS_ERR(dentry)) { if (IS_ERR(dentry)) {
kfree(buffer); kfree(buffer);
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
@ -421,7 +417,7 @@ struct dentry *hypfs_create_u64(struct super_block *sb, struct dentry *dir,
return dentry; return dentry;
} }
struct dentry *hypfs_create_str(struct super_block *sb, struct dentry *dir, struct dentry *hypfs_create_str(struct dentry *dir,
const char *name, char *string) const char *name, char *string)
{ {
char *buffer; char *buffer;
@ -432,7 +428,7 @@ struct dentry *hypfs_create_str(struct super_block *sb, struct dentry *dir,
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
sprintf(buffer, "%s\n", string); sprintf(buffer, "%s\n", string);
dentry = dentry =
hypfs_create_file(sb, dir, name, buffer, S_IFREG | REG_FILE_MODE); hypfs_create_file(dir, name, buffer, S_IFREG | REG_FILE_MODE);
if (IS_ERR(dentry)) { if (IS_ERR(dentry)) {
kfree(buffer); kfree(buffer);
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);