lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-id: <1356486933.16178.144.camel@kjgkr>
Date: Wed, 26 Dec 2012 10:55:33 +0900
From: Jaegeuk Kim <jaegeuk.kim@...sung.com>
To: Namjae Jeon <linkinjeon@...il.com>
Cc: linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-f2fs-devel@...ts.sourceforge.net,
Namjae Jeon <namjae.jeon@...sung.com>,
Amit Sahrawat <a.sahrawat@...sung.com>
Subject: Re: [PATCH 1/5] f2fs: Introduce some information prints in the mount
path
Hi,
Could you make them follow the file system convention?
Something like "F2FS: blah blah~".
Otherwise, how about adding a debugging function to express the prefix?
Thanks,
2012-12-22 (토), 12:08 +0900, Namjae Jeon:
> From: Namjae Jeon <namjae.jeon@...sung.com>
>
> Added few informative prints in the mount path, to convey proper error
> in case of mount failure.
>
> Signed-off-by: Namjae Jeon <namjae.jeon@...sung.com>
> Signed-off-by: Amit Sahrawat <a.sahrawat@...sung.com>
> ---
> fs/f2fs/super.c | 46 ++++++++++++++++++++++++++++++++++++----------
> 1 file changed, 36 insertions(+), 10 deletions(-)
>
> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> index cf0ffb8..86f8549 100644
> --- a/fs/f2fs/super.c
> +++ b/fs/f2fs/super.c
> @@ -342,19 +342,29 @@ static int sanity_check_raw_super(struct f2fs_super_block *raw_super)
> {
> unsigned int blocksize;
>
> - if (F2FS_SUPER_MAGIC != le32_to_cpu(raw_super->magic))
> + if (F2FS_SUPER_MAGIC != le32_to_cpu(raw_super->magic)) {
> + pr_info("Magic Mismatch, valid(0x%x) - read(0x%x)\n",
> + F2FS_SUPER_MAGIC, le32_to_cpu(raw_super->magic));
> return 1;
> + }
>
> /* Currently, support only 4KB block size */
> blocksize = 1 << le32_to_cpu(raw_super->log_blocksize);
> - if (blocksize != PAGE_CACHE_SIZE)
> + if (blocksize != PAGE_CACHE_SIZE) {
> + pr_info("Not valid blocksize (%u), supports only 4KB\n",
> + blocksize);
> return 1;
> + }
> if (le32_to_cpu(raw_super->log_sectorsize) !=
> - F2FS_LOG_SECTOR_SIZE)
> + F2FS_LOG_SECTOR_SIZE) {
> + pr_info("Not valid log sectorsize\n");
> return 1;
> + }
> if (le32_to_cpu(raw_super->log_sectors_per_block) !=
> - F2FS_LOG_SECTORS_PER_BLOCK)
> + F2FS_LOG_SECTORS_PER_BLOCK) {
> + pr_info("Not valid log sectors per block\n");
> return 1;
> + }
> return 0;
> }
>
> @@ -415,13 +425,16 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent)
> return -ENOMEM;
>
> /* set a temporary block size */
> - if (!sb_set_blocksize(sb, F2FS_BLKSIZE))
> + if (!sb_set_blocksize(sb, F2FS_BLKSIZE)) {
> + pr_err("unable to set blocksize\n");
> goto free_sbi;
> + }
>
> /* read f2fs raw super block */
> raw_super_buf = sb_bread(sb, 0);
> if (!raw_super_buf) {
> err = -EIO;
> + pr_err("unable to read superblock\n");
> goto free_sbi;
> }
> raw_super = (struct f2fs_super_block *)
> @@ -443,8 +456,10 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent)
> goto free_sb_buf;
>
> /* sanity checking of raw super */
> - if (sanity_check_raw_super(raw_super))
> + if (sanity_check_raw_super(raw_super)) {
> + pr_err("Not a valid F2FS filesystem\n");
> goto free_sb_buf;
> + }
>
> sb->s_maxbytes = max_file_size(le32_to_cpu(raw_super->log_blocksize));
> sb->s_max_links = F2FS_LINK_MAX;
> @@ -478,18 +493,23 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent)
> /* get an inode for meta space */
> sbi->meta_inode = f2fs_iget(sb, F2FS_META_INO(sbi));
> if (IS_ERR(sbi->meta_inode)) {
> + pr_err("Failed to read F2FS meta data inode\n");
> err = PTR_ERR(sbi->meta_inode);
> goto free_sb_buf;
> }
>
> err = get_valid_checkpoint(sbi);
> - if (err)
> + if (err) {
> + pr_err("Failed to get valid F2FS checkpoint\n");
> goto free_meta_inode;
> + }
>
> /* sanity checking of checkpoint */
> err = -EINVAL;
> - if (sanity_check_ckpt(raw_super, sbi->ckpt))
> + if (sanity_check_ckpt(raw_super, sbi->ckpt)) {
> + pr_err("Not a valid F2FS checkpoint\n");
> goto free_cp;
> + }
>
> sbi->total_valid_node_count =
> le32_to_cpu(sbi->ckpt->valid_node_count);
> @@ -511,17 +531,22 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent)
>
> /* setup f2fs internal modules */
> err = build_segment_manager(sbi);
> - if (err)
> + if (err) {
> + pr_err("Failed to initialize F2FS segment manager\n");
> goto free_sm;
> + }
> err = build_node_manager(sbi);
> - if (err)
> + if (err) {
> + pr_err("Failed to initialize F2FS node manager\n");
> goto free_nm;
> + }
>
> build_gc_manager(sbi);
>
> /* get an inode for node space */
> sbi->node_inode = f2fs_iget(sb, F2FS_NODE_INO(sbi));
> if (IS_ERR(sbi->node_inode)) {
> + pr_err("Failed to read node inode\n");
> err = PTR_ERR(sbi->node_inode);
> goto free_nm;
> }
> @@ -534,6 +559,7 @@ static int f2fs_fill_super(struct super_block *sb, void *data, int silent)
> /* read root inode and dentry */
> root = f2fs_iget(sb, F2FS_ROOT_INO(sbi));
> if (IS_ERR(root)) {
> + pr_err("Failed to read root inode\n");
> err = PTR_ERR(root);
> goto free_node_inode;
> }
--
Jaegeuk Kim
Samsung
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists