diff options
Diffstat (limited to 'fs/hpfs/super.c')
| -rw-r--r-- | fs/hpfs/super.c | 118 | 
1 files changed, 51 insertions, 67 deletions
diff --git a/fs/hpfs/super.c b/fs/hpfs/super.c index c89b4080858..98580a3b500 100644 --- a/fs/hpfs/super.c +++ b/fs/hpfs/super.c @@ -18,15 +18,16 @@  /* Mark the filesystem dirty, so that chkdsk checks it when os/2 booted */ -static void mark_dirty(struct super_block *s) +static void mark_dirty(struct super_block *s, int remount)  { -	if (hpfs_sb(s)->sb_chkdsk && !(s->s_flags & MS_RDONLY)) { +	if (hpfs_sb(s)->sb_chkdsk && (remount || !(s->s_flags & MS_RDONLY))) {  		struct buffer_head *bh;  		struct hpfs_spare_block *sb;  		if ((sb = hpfs_map_sector(s, 17, &bh, 0))) {  			sb->dirty = 1;  			sb->old_wrote = 0;  			mark_buffer_dirty(bh); +			sync_dirty_buffer(bh);  			brelse(bh);  		}  	} @@ -40,10 +41,12 @@ static void unmark_dirty(struct super_block *s)  	struct buffer_head *bh;  	struct hpfs_spare_block *sb;  	if (s->s_flags & MS_RDONLY) return; +	sync_blockdev(s->s_bdev);  	if ((sb = hpfs_map_sector(s, 17, &bh, 0))) {  		sb->dirty = hpfs_sb(s)->sb_chkdsk > 1 - hpfs_sb(s)->sb_was_error;  		sb->old_wrote = hpfs_sb(s)->sb_chkdsk >= 2 && !hpfs_sb(s)->sb_was_error;  		mark_buffer_dirty(bh); +		sync_dirty_buffer(bh);  		brelse(bh);  	}  } @@ -63,13 +66,13 @@ void hpfs_error(struct super_block *s, const char *fmt, ...)  	if (!hpfs_sb(s)->sb_was_error) {  		if (hpfs_sb(s)->sb_err == 2) {  			printk("; crashing the system because you wanted it\n"); -			mark_dirty(s); +			mark_dirty(s, 0);  			panic("HPFS panic");  		} else if (hpfs_sb(s)->sb_err == 1) {  			if (s->s_flags & MS_RDONLY) printk("; already mounted read-only\n");  			else {  				printk("; remounting read-only\n"); -				mark_dirty(s); +				mark_dirty(s, 0);  				s->s_flags |= MS_RDONLY;  			}  		} else if (s->s_flags & MS_RDONLY) printk("; going on - but anything won't be destroyed because it's read-only\n"); @@ -102,9 +105,12 @@ static void hpfs_put_super(struct super_block *s)  {  	struct hpfs_sb_info *sbi = hpfs_sb(s); +	hpfs_lock(s); +	unmark_dirty(s); +	hpfs_unlock(s); +  	kfree(sbi->sb_cp_table);  	kfree(sbi->sb_bmp_dir); -	unmark_dirty(s);  	s->s_fs_info = NULL;  	kfree(sbi);  } @@ -129,7 +135,7 @@ static unsigned count_bitmaps(struct super_block *s)  	n_bands = (hpfs_sb(s)->sb_fs_size + 0x3fff) >> 14;  	count = 0;  	for (n = 0; n < n_bands; n++) -		count += hpfs_count_one_bitmap(s, hpfs_sb(s)->sb_bmp_dir[n]); +		count += hpfs_count_one_bitmap(s, le32_to_cpu(hpfs_sb(s)->sb_bmp_dir[n]));  	return count;  } @@ -188,8 +194,6 @@ static void init_once(void *foo)  {  	struct hpfs_inode_info *ei = (struct hpfs_inode_info *) foo; -	mutex_init(&ei->i_mutex); -	mutex_init(&ei->i_parent_mutex);  	inode_init_once(&ei->vfs_inode);  } @@ -218,7 +222,6 @@ static void destroy_inodecache(void)  enum {  	Opt_help, Opt_uid, Opt_gid, Opt_umask, Opt_case_lower, Opt_case_asis, -	Opt_conv_binary, Opt_conv_text, Opt_conv_auto,  	Opt_check_none, Opt_check_normal, Opt_check_strict,  	Opt_err_cont, Opt_err_ro, Opt_err_panic,  	Opt_eas_no, Opt_eas_ro, Opt_eas_rw, @@ -233,9 +236,6 @@ static const match_table_t tokens = {  	{Opt_umask, "umask=%o"},  	{Opt_case_lower, "case=lower"},  	{Opt_case_asis, "case=asis"}, -	{Opt_conv_binary, "conv=binary"}, -	{Opt_conv_text, "conv=text"}, -	{Opt_conv_auto, "conv=auto"},  	{Opt_check_none, "check=none"},  	{Opt_check_normal, "check=normal"},  	{Opt_check_strict, "check=strict"}, @@ -253,7 +253,7 @@ static const match_table_t tokens = {  };  static int parse_opts(char *opts, uid_t *uid, gid_t *gid, umode_t *umask, -		      int *lowercase, int *conv, int *eas, int *chk, int *errs, +		      int *lowercase, int *eas, int *chk, int *errs,  		      int *chkdsk, int *timeshift)  {  	char *p; @@ -295,15 +295,6 @@ static int parse_opts(char *opts, uid_t *uid, gid_t *gid, umode_t *umask,  		case Opt_case_asis:  			*lowercase = 0;  			break; -		case Opt_conv_binary: -			*conv = CONV_BINARY; -			break; -		case Opt_conv_text: -			*conv = CONV_TEXT; -			break; -		case Opt_conv_auto: -			*conv = CONV_AUTO; -			break;  		case Opt_check_none:  			*chk = 0;  			break; @@ -370,9 +361,6 @@ HPFS filesystem options:\n\        umask=xxx         set mode of files that don't have mode specified in eas\n\        case=lower        lowercase all files\n\        case=asis         do not lowercase files (default)\n\ -      conv=binary       do not convert CR/LF -> LF (default)\n\ -      conv=auto         convert only files with known text extensions\n\ -      conv=text         convert all files\n\        check=none        no fs checks - kernel may crash on corrupted filesystem\n\        check=normal      do some checks - it should not crash (default)\n\        check=strict      do extra time-consuming checks, used for debugging\n\ @@ -394,7 +382,7 @@ static int hpfs_remount_fs(struct super_block *s, int *flags, char *data)  	uid_t uid;  	gid_t gid;  	umode_t umask; -	int lowercase, conv, eas, chk, errs, chkdsk, timeshift; +	int lowercase, eas, chk, errs, chkdsk, timeshift;  	int o;  	struct hpfs_sb_info *sbi = hpfs_sb(s);  	char *new_opts = kstrdup(data, GFP_KERNEL); @@ -405,11 +393,11 @@ static int hpfs_remount_fs(struct super_block *s, int *flags, char *data)  	lock_super(s);  	uid = sbi->sb_uid; gid = sbi->sb_gid;  	umask = 0777 & ~sbi->sb_mode; -	lowercase = sbi->sb_lowercase; conv = sbi->sb_conv; +	lowercase = sbi->sb_lowercase;  	eas = sbi->sb_eas; chk = sbi->sb_chk; chkdsk = sbi->sb_chkdsk;  	errs = sbi->sb_err; timeshift = sbi->sb_timeshift; -	if (!(o = parse_opts(data, &uid, &gid, &umask, &lowercase, &conv, +	if (!(o = parse_opts(data, &uid, &gid, &umask, &lowercase,  	    &eas, &chk, &errs, &chkdsk, ×hift))) {  		printk("HPFS: bad mount options.\n");  		goto out_err; @@ -427,11 +415,11 @@ static int hpfs_remount_fs(struct super_block *s, int *flags, char *data)  	sbi->sb_uid = uid; sbi->sb_gid = gid;  	sbi->sb_mode = 0777 & ~umask; -	sbi->sb_lowercase = lowercase; sbi->sb_conv = conv; +	sbi->sb_lowercase = lowercase;  	sbi->sb_eas = eas; sbi->sb_chk = chk; sbi->sb_chkdsk = chkdsk;  	sbi->sb_err = errs; sbi->sb_timeshift = timeshift; -	if (!(*flags & MS_RDONLY)) mark_dirty(s); +	if (!(*flags & MS_RDONLY)) mark_dirty(s, 1);  	replace_mount_options(s, new_opts); @@ -471,7 +459,7 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	uid_t uid;  	gid_t gid;  	umode_t umask; -	int lowercase, conv, eas, chk, errs, chkdsk, timeshift; +	int lowercase, eas, chk, errs, chkdsk, timeshift;  	dnode_secno root_dno;  	struct hpfs_dirent *de = NULL; @@ -479,11 +467,6 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	int o; -	if (num_possible_cpus() > 1) { -		printk(KERN_ERR "HPFS is not SMP safe\n"); -		return -EINVAL; -	} -  	save_mount_options(s, options);  	sbi = kzalloc(sizeof(*sbi), GFP_KERNEL); @@ -495,20 +478,20 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	sbi->sb_bmp_dir = NULL;  	sbi->sb_cp_table = NULL; -	mutex_init(&sbi->hpfs_creation_de); +	mutex_init(&sbi->hpfs_mutex); +	hpfs_lock(s);  	uid = current_uid();  	gid = current_gid();  	umask = current_umask();  	lowercase = 0; -	conv = CONV_BINARY;  	eas = 2;  	chk = 1;  	errs = 1;  	chkdsk = 1;  	timeshift = 0; -	if (!(o = parse_opts(options, &uid, &gid, &umask, &lowercase, &conv, +	if (!(o = parse_opts(options, &uid, &gid, &umask, &lowercase,  	    &eas, &chk, &errs, &chkdsk, ×hift))) {  		printk("HPFS: bad mount options.\n");  		goto bail0; @@ -526,9 +509,9 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	if (!(spareblock = hpfs_map_sector(s, 17, &bh2, 0))) goto bail3;  	/* Check magics */ -	if (/*bootblock->magic != BB_MAGIC -	    ||*/ superblock->magic != SB_MAGIC -	    || spareblock->magic != SP_MAGIC) { +	if (/*le16_to_cpu(bootblock->magic) != BB_MAGIC +	    ||*/ le32_to_cpu(superblock->magic) != SB_MAGIC +	    || le32_to_cpu(spareblock->magic) != SP_MAGIC) {  		if (!silent) printk("HPFS: Bad magic ... probably not HPFS\n");  		goto bail4;  	} @@ -549,19 +532,18 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	s->s_op = &hpfs_sops;  	s->s_d_op = &hpfs_dentry_operations; -	sbi->sb_root = superblock->root; -	sbi->sb_fs_size = superblock->n_sectors; -	sbi->sb_bitmaps = superblock->bitmaps; -	sbi->sb_dirband_start = superblock->dir_band_start; -	sbi->sb_dirband_size = superblock->n_dir_band; -	sbi->sb_dmap = superblock->dir_band_bitmap; +	sbi->sb_root = le32_to_cpu(superblock->root); +	sbi->sb_fs_size = le32_to_cpu(superblock->n_sectors); +	sbi->sb_bitmaps = le32_to_cpu(superblock->bitmaps); +	sbi->sb_dirband_start = le32_to_cpu(superblock->dir_band_start); +	sbi->sb_dirband_size = le32_to_cpu(superblock->n_dir_band); +	sbi->sb_dmap = le32_to_cpu(superblock->dir_band_bitmap);  	sbi->sb_uid = uid;  	sbi->sb_gid = gid;  	sbi->sb_mode = 0777 & ~umask;  	sbi->sb_n_free = -1;  	sbi->sb_n_free_dnodes = -1;  	sbi->sb_lowercase = lowercase; -	sbi->sb_conv = conv;  	sbi->sb_eas = eas;  	sbi->sb_chk = chk;  	sbi->sb_chkdsk = chkdsk; @@ -573,7 +555,7 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	sbi->sb_max_fwd_alloc = 0xffffff;  	/* Load bitmap directory */ -	if (!(sbi->sb_bmp_dir = hpfs_load_bitmap_directory(s, superblock->bitmaps))) +	if (!(sbi->sb_bmp_dir = hpfs_load_bitmap_directory(s, le32_to_cpu(superblock->bitmaps))))  		goto bail4;  	/* Check for general fs errors*/ @@ -591,20 +573,20 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  		mark_buffer_dirty(bh2);  	} -	if (spareblock->hotfixes_used || spareblock->n_spares_used) { +	if (le32_to_cpu(spareblock->hotfixes_used) || le32_to_cpu(spareblock->n_spares_used)) {  		if (errs >= 2) {  			printk("HPFS: Hotfixes not supported here, try chkdsk\n"); -			mark_dirty(s); +			mark_dirty(s, 0);  			goto bail4;  		}  		hpfs_error(s, "hotfixes not supported here, try chkdsk");  		if (errs == 0) printk("HPFS: Proceeding, but your filesystem will be probably corrupted by this driver...\n");  		else printk("HPFS: This driver may read bad files or crash when operating on disk with hotfixes.\n");  	} -	if (spareblock->n_dnode_spares != spareblock->n_dnode_spares_free) { +	if (le32_to_cpu(spareblock->n_dnode_spares) != le32_to_cpu(spareblock->n_dnode_spares_free)) {  		if (errs >= 2) {  			printk("HPFS: Spare dnodes used, try chkdsk\n"); -			mark_dirty(s); +			mark_dirty(s, 0);  			goto bail4;  		}  		hpfs_error(s, "warning: spare dnodes used, try chkdsk"); @@ -612,26 +594,26 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	}  	if (chk) {  		unsigned a; -		if (superblock->dir_band_end - superblock->dir_band_start + 1 != superblock->n_dir_band || -		    superblock->dir_band_end < superblock->dir_band_start || superblock->n_dir_band > 0x4000) { +		if (le32_to_cpu(superblock->dir_band_end) - le32_to_cpu(superblock->dir_band_start) + 1 != le32_to_cpu(superblock->n_dir_band) || +		    le32_to_cpu(superblock->dir_band_end) < le32_to_cpu(superblock->dir_band_start) || le32_to_cpu(superblock->n_dir_band) > 0x4000) {  			hpfs_error(s, "dir band size mismatch: dir_band_start==%08x, dir_band_end==%08x, n_dir_band==%08x", -				superblock->dir_band_start, superblock->dir_band_end, superblock->n_dir_band); +				le32_to_cpu(superblock->dir_band_start), le32_to_cpu(superblock->dir_band_end), le32_to_cpu(superblock->n_dir_band));  			goto bail4;  		}  		a = sbi->sb_dirband_size;  		sbi->sb_dirband_size = 0; -		if (hpfs_chk_sectors(s, superblock->dir_band_start, superblock->n_dir_band, "dir_band") || -		    hpfs_chk_sectors(s, superblock->dir_band_bitmap, 4, "dir_band_bitmap") || -		    hpfs_chk_sectors(s, superblock->bitmaps, 4, "bitmaps")) { -			mark_dirty(s); +		if (hpfs_chk_sectors(s, le32_to_cpu(superblock->dir_band_start), le32_to_cpu(superblock->n_dir_band), "dir_band") || +		    hpfs_chk_sectors(s, le32_to_cpu(superblock->dir_band_bitmap), 4, "dir_band_bitmap") || +		    hpfs_chk_sectors(s, le32_to_cpu(superblock->bitmaps), 4, "bitmaps")) { +			mark_dirty(s, 0);  			goto bail4;  		}  		sbi->sb_dirband_size = a;  	} else printk("HPFS: You really don't want any checks? You are crazy...\n");  	/* Load code page table */ -	if (spareblock->n_code_pages) -		if (!(sbi->sb_cp_table = hpfs_load_code_page(s, spareblock->code_page_dir))) +	if (le32_to_cpu(spareblock->n_code_pages)) +		if (!(sbi->sb_cp_table = hpfs_load_code_page(s, le32_to_cpu(spareblock->code_page_dir))))  			printk("HPFS: Warning: code page support is disabled\n");  	brelse(bh2); @@ -660,13 +642,13 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  	if (!de)  		hpfs_error(s, "unable to find root dir");  	else { -		root->i_atime.tv_sec = local_to_gmt(s, de->read_date); +		root->i_atime.tv_sec = local_to_gmt(s, le32_to_cpu(de->read_date));  		root->i_atime.tv_nsec = 0; -		root->i_mtime.tv_sec = local_to_gmt(s, de->write_date); +		root->i_mtime.tv_sec = local_to_gmt(s, le32_to_cpu(de->write_date));  		root->i_mtime.tv_nsec = 0; -		root->i_ctime.tv_sec = local_to_gmt(s, de->creation_date); +		root->i_ctime.tv_sec = local_to_gmt(s, le32_to_cpu(de->creation_date));  		root->i_ctime.tv_nsec = 0; -		hpfs_i(root)->i_ea_size = de->ea_size; +		hpfs_i(root)->i_ea_size = le16_to_cpu(de->ea_size);  		hpfs_i(root)->i_parent_dir = root->i_ino;  		if (root->i_size == -1)  			root->i_size = 2048; @@ -674,6 +656,7 @@ static int hpfs_fill_super(struct super_block *s, void *options, int silent)  			root->i_blocks = 5;  		hpfs_brelse4(&qbh);  	} +	hpfs_unlock(s);  	return 0;  bail4:	brelse(bh2); @@ -681,6 +664,7 @@ bail3:	brelse(bh1);  bail2:	brelse(bh0);  bail1:  bail0: +	hpfs_unlock(s);  	kfree(sbi->sb_bmp_dir);  	kfree(sbi->sb_cp_table);  	s->s_fs_info = NULL;  |