nilfs2: introduce nilfs_prepare_super
This function checks validity of super block pointers. If first super block is invalid, it will swap the super blocks. The function should be called before any super block information updates. Caller must obtain nilfs->ns_sem. Signed-off-by: Jiro SEKIBA <jir@unicus.jp> Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
This commit is contained in:
parent
60f46b7efc
commit
d26493b6f0
|
@ -272,6 +272,7 @@ extern int nilfs_store_magic_and_option(struct super_block *,
|
||||||
struct nilfs_super_block *, char *);
|
struct nilfs_super_block *, char *);
|
||||||
extern void nilfs_set_log_cursor(struct nilfs_super_block *,
|
extern void nilfs_set_log_cursor(struct nilfs_super_block *,
|
||||||
struct the_nilfs *);
|
struct the_nilfs *);
|
||||||
|
extern struct nilfs_super_block **nilfs_prepare_super(struct nilfs_sb_info *);
|
||||||
extern int nilfs_commit_super(struct nilfs_sb_info *, int);
|
extern int nilfs_commit_super(struct nilfs_sb_info *, int);
|
||||||
extern int nilfs_cleanup_super(struct nilfs_sb_info *);
|
extern int nilfs_cleanup_super(struct nilfs_sb_info *);
|
||||||
extern int nilfs_attach_checkpoint(struct nilfs_sb_info *, __u64);
|
extern int nilfs_attach_checkpoint(struct nilfs_sb_info *, __u64);
|
||||||
|
|
|
@ -2408,6 +2408,7 @@ static int nilfs_segctor_construct(struct nilfs_sc_info *sci, int mode)
|
||||||
{
|
{
|
||||||
struct nilfs_sb_info *sbi = sci->sc_sbi;
|
struct nilfs_sb_info *sbi = sci->sc_sbi;
|
||||||
struct the_nilfs *nilfs = sbi->s_nilfs;
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
|
struct nilfs_super_block **sbp;
|
||||||
int err = 0;
|
int err = 0;
|
||||||
|
|
||||||
nilfs_segctor_accept(sci);
|
nilfs_segctor_accept(sci);
|
||||||
|
@ -2423,8 +2424,11 @@ static int nilfs_segctor_construct(struct nilfs_sc_info *sci, int mode)
|
||||||
if (test_bit(NILFS_SC_SUPER_ROOT, &sci->sc_flags) &&
|
if (test_bit(NILFS_SC_SUPER_ROOT, &sci->sc_flags) &&
|
||||||
nilfs_discontinued(nilfs)) {
|
nilfs_discontinued(nilfs)) {
|
||||||
down_write(&nilfs->ns_sem);
|
down_write(&nilfs->ns_sem);
|
||||||
err = nilfs_commit_super(
|
err = -EIO;
|
||||||
sbi, nilfs_altsb_need_update(nilfs));
|
sbp = nilfs_prepare_super(sbi);
|
||||||
|
if (likely(sbp))
|
||||||
|
err = nilfs_commit_super(
|
||||||
|
sbi, nilfs_altsb_need_update(nilfs));
|
||||||
up_write(&nilfs->ns_sem);
|
up_write(&nilfs->ns_sem);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -77,12 +77,16 @@ static int nilfs_remount(struct super_block *sb, int *flags, char *data);
|
||||||
static void nilfs_set_error(struct nilfs_sb_info *sbi)
|
static void nilfs_set_error(struct nilfs_sb_info *sbi)
|
||||||
{
|
{
|
||||||
struct the_nilfs *nilfs = sbi->s_nilfs;
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
|
struct nilfs_super_block **sbp;
|
||||||
|
|
||||||
down_write(&nilfs->ns_sem);
|
down_write(&nilfs->ns_sem);
|
||||||
if (!(nilfs->ns_mount_state & NILFS_ERROR_FS)) {
|
if (!(nilfs->ns_mount_state & NILFS_ERROR_FS)) {
|
||||||
nilfs->ns_mount_state |= NILFS_ERROR_FS;
|
nilfs->ns_mount_state |= NILFS_ERROR_FS;
|
||||||
nilfs->ns_sbp[0]->s_state |= cpu_to_le16(NILFS_ERROR_FS);
|
sbp = nilfs_prepare_super(sbi);
|
||||||
nilfs_commit_super(sbi, 1);
|
if (likely(sbp)) {
|
||||||
|
sbp[0]->s_state |= cpu_to_le16(NILFS_ERROR_FS);
|
||||||
|
nilfs_commit_super(sbi, 1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
up_write(&nilfs->ns_sem);
|
up_write(&nilfs->ns_sem);
|
||||||
}
|
}
|
||||||
|
@ -253,22 +257,32 @@ void nilfs_set_log_cursor(struct nilfs_super_block *sbp,
|
||||||
spin_unlock(&nilfs->ns_last_segment_lock);
|
spin_unlock(&nilfs->ns_last_segment_lock);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
struct nilfs_super_block **nilfs_prepare_super(struct nilfs_sb_info *sbi)
|
||||||
|
{
|
||||||
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
|
struct nilfs_super_block **sbp = nilfs->ns_sbp;
|
||||||
|
|
||||||
|
/* nilfs->ns_sem must be locked by the caller. */
|
||||||
|
if (sbp[0]->s_magic != cpu_to_le16(NILFS_SUPER_MAGIC)) {
|
||||||
|
if (sbp[1] &&
|
||||||
|
sbp[1]->s_magic == cpu_to_le16(NILFS_SUPER_MAGIC)) {
|
||||||
|
nilfs_swap_super_block(nilfs);
|
||||||
|
} else {
|
||||||
|
printk(KERN_CRIT "NILFS: superblock broke on dev %s\n",
|
||||||
|
sbi->s_super->s_id);
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return sbp;
|
||||||
|
}
|
||||||
|
|
||||||
int nilfs_commit_super(struct nilfs_sb_info *sbi, int dupsb)
|
int nilfs_commit_super(struct nilfs_sb_info *sbi, int dupsb)
|
||||||
{
|
{
|
||||||
struct the_nilfs *nilfs = sbi->s_nilfs;
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
struct nilfs_super_block **sbp = nilfs->ns_sbp;
|
struct nilfs_super_block **sbp = nilfs->ns_sbp;
|
||||||
time_t t;
|
time_t t;
|
||||||
|
|
||||||
/* nilfs->sem must be locked by the caller. */
|
/* nilfs->ns_sem must be locked by the caller. */
|
||||||
if (sbp[0]->s_magic != cpu_to_le16(NILFS_SUPER_MAGIC)) {
|
|
||||||
if (sbp[1] && sbp[1]->s_magic == cpu_to_le16(NILFS_SUPER_MAGIC))
|
|
||||||
nilfs_swap_super_block(nilfs);
|
|
||||||
else {
|
|
||||||
printk(KERN_CRIT "NILFS: superblock broke on dev %s\n",
|
|
||||||
sbi->s_super->s_id);
|
|
||||||
return -EIO;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
nilfs_set_log_cursor(sbp[0], nilfs);
|
nilfs_set_log_cursor(sbp[0], nilfs);
|
||||||
|
|
||||||
t = get_seconds();
|
t = get_seconds();
|
||||||
|
@ -296,11 +310,14 @@ int nilfs_commit_super(struct nilfs_sb_info *sbi, int dupsb)
|
||||||
*/
|
*/
|
||||||
int nilfs_cleanup_super(struct nilfs_sb_info *sbi)
|
int nilfs_cleanup_super(struct nilfs_sb_info *sbi)
|
||||||
{
|
{
|
||||||
struct nilfs_super_block **sbp = sbi->s_nilfs->ns_sbp;
|
struct nilfs_super_block **sbp;
|
||||||
int ret;
|
int ret = -EIO;
|
||||||
|
|
||||||
sbp[0]->s_state = cpu_to_le16(sbi->s_nilfs->ns_mount_state);
|
sbp = nilfs_prepare_super(sbi);
|
||||||
ret = nilfs_commit_super(sbi, 1);
|
if (sbp) {
|
||||||
|
sbp[0]->s_state = cpu_to_le16(sbi->s_nilfs->ns_mount_state);
|
||||||
|
ret = nilfs_commit_super(sbi, 1);
|
||||||
|
}
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -336,6 +353,7 @@ static int nilfs_sync_fs(struct super_block *sb, int wait)
|
||||||
{
|
{
|
||||||
struct nilfs_sb_info *sbi = NILFS_SB(sb);
|
struct nilfs_sb_info *sbi = NILFS_SB(sb);
|
||||||
struct the_nilfs *nilfs = sbi->s_nilfs;
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
|
struct nilfs_super_block **sbp;
|
||||||
int err = 0;
|
int err = 0;
|
||||||
|
|
||||||
/* This function is called when super block should be written back */
|
/* This function is called when super block should be written back */
|
||||||
|
@ -343,8 +361,11 @@ static int nilfs_sync_fs(struct super_block *sb, int wait)
|
||||||
err = nilfs_construct_segment(sb);
|
err = nilfs_construct_segment(sb);
|
||||||
|
|
||||||
down_write(&nilfs->ns_sem);
|
down_write(&nilfs->ns_sem);
|
||||||
if (nilfs_sb_dirty(nilfs))
|
if (nilfs_sb_dirty(nilfs)) {
|
||||||
nilfs_commit_super(sbi, 1);
|
sbp = nilfs_prepare_super(sbi);
|
||||||
|
if (likely(sbp))
|
||||||
|
nilfs_commit_super(sbi, 1);
|
||||||
|
}
|
||||||
up_write(&nilfs->ns_sem);
|
up_write(&nilfs->ns_sem);
|
||||||
|
|
||||||
return err;
|
return err;
|
||||||
|
@ -638,11 +659,18 @@ nilfs_set_default_options(struct nilfs_sb_info *sbi,
|
||||||
static int nilfs_setup_super(struct nilfs_sb_info *sbi)
|
static int nilfs_setup_super(struct nilfs_sb_info *sbi)
|
||||||
{
|
{
|
||||||
struct the_nilfs *nilfs = sbi->s_nilfs;
|
struct the_nilfs *nilfs = sbi->s_nilfs;
|
||||||
struct nilfs_super_block *sbp = nilfs->ns_sbp[0];
|
struct nilfs_super_block **sbp;
|
||||||
int max_mnt_count = le16_to_cpu(sbp->s_max_mnt_count);
|
int max_mnt_count;
|
||||||
int mnt_count = le16_to_cpu(sbp->s_mnt_count);
|
int mnt_count;
|
||||||
|
|
||||||
|
/* nilfs->ns_sem must be locked by the caller. */
|
||||||
|
sbp = nilfs_prepare_super(sbi);
|
||||||
|
if (!sbp)
|
||||||
|
return -EIO;
|
||||||
|
|
||||||
|
max_mnt_count = le16_to_cpu(sbp[0]->s_max_mnt_count);
|
||||||
|
mnt_count = le16_to_cpu(sbp[0]->s_mnt_count);
|
||||||
|
|
||||||
/* nilfs->sem must be locked by the caller. */
|
|
||||||
if (nilfs->ns_mount_state & NILFS_ERROR_FS) {
|
if (nilfs->ns_mount_state & NILFS_ERROR_FS) {
|
||||||
printk(KERN_WARNING
|
printk(KERN_WARNING
|
||||||
"NILFS warning: mounting fs with errors\n");
|
"NILFS warning: mounting fs with errors\n");
|
||||||
|
@ -653,11 +681,12 @@ static int nilfs_setup_super(struct nilfs_sb_info *sbi)
|
||||||
#endif
|
#endif
|
||||||
}
|
}
|
||||||
if (!max_mnt_count)
|
if (!max_mnt_count)
|
||||||
sbp->s_max_mnt_count = cpu_to_le16(NILFS_DFL_MAX_MNT_COUNT);
|
sbp[0]->s_max_mnt_count = cpu_to_le16(NILFS_DFL_MAX_MNT_COUNT);
|
||||||
|
|
||||||
sbp->s_mnt_count = cpu_to_le16(mnt_count + 1);
|
sbp[0]->s_mnt_count = cpu_to_le16(mnt_count + 1);
|
||||||
sbp->s_state = cpu_to_le16(le16_to_cpu(sbp->s_state) & ~NILFS_VALID_FS);
|
sbp[0]->s_state =
|
||||||
sbp->s_mtime = cpu_to_le64(get_seconds());
|
cpu_to_le16(le16_to_cpu(sbp[0]->s_state) & ~NILFS_VALID_FS);
|
||||||
|
sbp[0]->s_mtime = cpu_to_le64(get_seconds());
|
||||||
return nilfs_commit_super(sbi, 1);
|
return nilfs_commit_super(sbi, 1);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue