mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Nikolay Borisov <nborisov@suse.com>
To: Namjae Jeon <namjae.jeon@samsung.com>,
	linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Cc: gregkh@linuxfoundation.org, valdis.kletnieks@vt.edu, hch@lst.de,
	linkinjeon@gmail.com, Markus.Elfring@web.de,
	sj1557.seo@samsung.com
Subject: Re: [PATCH v2 07/13] exfat: add bitmap operations
Date: Wed, 20 Nov 2019 11:24:00 +0200	[thread overview]
Message-ID: <09920df0-ff2d-72f9-b82f-21e0ac5499ac@suse.com> (raw)
In-Reply-To: <20191119071107.1947-8-namjae.jeon@samsung.com>



On 19.11.19 г. 9:11 ч., Namjae Jeon wrote:
> This adds the implementation of bitmap operations for exfat.
> 
> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
> Signed-off-by: Sungjong Seo <sj1557.seo@samsung.com>
> ---
>  fs/exfat/balloc.c | 271 ++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 271 insertions(+)
>  create mode 100644 fs/exfat/balloc.c
> 
> diff --git a/fs/exfat/balloc.c b/fs/exfat/balloc.c
> new file mode 100644
> index 000000000000..930e0ea3dbfd
> --- /dev/null
> +++ b/fs/exfat/balloc.c
> @@ -0,0 +1,271 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + *  Copyright (C) 2012-2013 Samsung Electronics Co., Ltd.
> + */
> +
> +#include <linux/blkdev.h>
> +#include <linux/slab.h>
> +#include <linux/buffer_head.h>
> +
> +#include "exfat_raw.h"
> +#include "exfat_fs.h"
> +
> +static const unsigned char free_bit[] = {
> +	0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 4, 0, 1, 0, 2,/*  0 ~  19*/
> +	0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 5, 0, 1, 0, 2, 0, 1, 0, 3,/* 20 ~  39*/
> +	0, 1, 0, 2, 0, 1, 0, 4, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2,/* 40 ~  59*/
> +	0, 1, 0, 6, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 4,/* 60 ~  79*/
> +	0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 5, 0, 1, 0, 2,/* 80 ~  99*/
> +	0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 4, 0, 1, 0, 2, 0, 1, 0, 3,/*100 ~ 119*/
> +	0, 1, 0, 2, 0, 1, 0, 7, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2,/*120 ~ 139*/
> +	0, 1, 0, 4, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 5,/*140 ~ 159*/
> +	0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 4, 0, 1, 0, 2,/*160 ~ 179*/
> +	0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 6, 0, 1, 0, 2, 0, 1, 0, 3,/*180 ~ 199*/
> +	0, 1, 0, 2, 0, 1, 0, 4, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2,/*200 ~ 219*/
> +	0, 1, 0, 5, 0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0, 4,/*220 ~ 239*/
> +	0, 1, 0, 2, 0, 1, 0, 3, 0, 1, 0, 2, 0, 1, 0                /*240 ~ 254*/
> +};
> +
> +static const unsigned char used_bit[] = {
> +	0, 1, 1, 2, 1, 2, 2, 3, 1, 2, 2, 3, 2, 3, 3, 4, 1, 2, 2, 3,/*  0 ~  19*/
> +	2, 3, 3, 4, 2, 3, 3, 4, 3, 4, 4, 5, 1, 2, 2, 3, 2, 3, 3, 4,/* 20 ~  39*/
> +	2, 3, 3, 4, 3, 4, 4, 5, 2, 3, 3, 4, 3, 4, 4, 5, 3, 4, 4, 5,/* 40 ~  59*/
> +	4, 5, 5, 6, 1, 2, 2, 3, 2, 3, 3, 4, 2, 3, 3, 4, 3, 4, 4, 5,/* 60 ~  79*/
> +	2, 3, 3, 4, 3, 4, 4, 5, 3, 4, 4, 5, 4, 5, 5, 6, 2, 3, 3, 4,/* 80 ~  99*/
> +	3, 4, 4, 5, 3, 4, 4, 5, 4, 5, 5, 6, 3, 4, 4, 5, 4, 5, 5, 6,/*100 ~ 119*/
> +	4, 5, 5, 6, 5, 6, 6, 7, 1, 2, 2, 3, 2, 3, 3, 4, 2, 3, 3, 4,/*120 ~ 139*/
> +	3, 4, 4, 5, 2, 3, 3, 4, 3, 4, 4, 5, 3, 4, 4, 5, 4, 5, 5, 6,/*140 ~ 159*/
> +	2, 3, 3, 4, 3, 4, 4, 5, 3, 4, 4, 5, 4, 5, 5, 6, 3, 4, 4, 5,/*160 ~ 179*/
> +	4, 5, 5, 6, 4, 5, 5, 6, 5, 6, 6, 7, 2, 3, 3, 4, 3, 4, 4, 5,/*180 ~ 199*/
> +	3, 4, 4, 5, 4, 5, 5, 6, 3, 4, 4, 5, 4, 5, 5, 6, 4, 5, 5, 6,/*200 ~ 219*/
> +	5, 6, 6, 7, 3, 4, 4, 5, 4, 5, 5, 6, 4, 5, 5, 6, 5, 6, 6, 7,/*220 ~ 239*/
> +	4, 5, 5, 6, 5, 6, 6, 7, 5, 6, 6, 7, 6, 7, 7, 8             /*240 ~ 255*/
> +};
> +
> +/*
> + *  Allocation Bitmap Management Functions
> + */
> +static int exfat_allocate_bitmap(struct super_block *sb,
> +		struct exfat_dentry *ep)
> +{
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +	long long map_size;
> +	unsigned int i, need_map_size;
> +	sector_t sector;
> +
> +	sbi->map_clu = le32_to_cpu(ep->bitmap_start_clu);
> +	map_size = le64_to_cpu(ep->bitmap_size);
> +	need_map_size = (((sbi->num_clusters - BASE_CLUSTER) - 1) >> 3) + 1;
> +	if (need_map_size != map_size) {
> +		exfat_msg(sb, KERN_ERR,
> +				"bogus allocation bitmap size(need : %u, cur : %lld)",
> +				need_map_size, map_size);
> +		/*
> +		 * Only allowed when bogus allocation
> +		 * bitmap size is large
> +		 */
> +		if (need_map_size > map_size)
> +			return -EIO;
> +	}
> +	sbi->map_sectors = ((need_map_size - 1) >>
> +			(sb->s_blocksize_bits)) + 1;
> +	sbi->vol_amap = kmalloc_array(sbi->map_sectors,
> +				sizeof(struct buffer_head *), GFP_KERNEL);
> +	if (!sbi->vol_amap)
> +		return -ENOMEM;
> +
> +	sector = exfat_cluster_to_sector(sbi, sbi->map_clu);
> +	for (i = 0; i < sbi->map_sectors; i++) {
> +		sbi->vol_amap[i] = sb_bread(sb, sector + i);
> +		if (!sbi->vol_amap[i]) {
> +			/* release all buffers and free vol_amap */
> +			int j = 0;
> +
> +			while (j < i)
> +				brelse(sbi->vol_amap[j++]);
> +
> +			kfree(sbi->vol_amap);
> +			sbi->vol_amap = NULL;
> +			return -EIO;
> +		}
> +	}
> +
> +	sbi->pbr_bh = NULL;
> +	return 0;
> +}
> +
> +int exfat_load_bitmap(struct super_block *sb)
> +{
> +	unsigned int i, type;
> +	struct exfat_chain clu;
> +	struct exfat_dentry *ep = NULL;
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +	struct buffer_head *bh;
> +
> +	exfat_chain_set(&clu, sbi->root_dir, 0, 0x01);

magic constant for the 4th parameter
> +
> +	while (clu.dir != EOF_CLUSTER) {
> +		for (i = 0; i < sbi->dentries_per_clu; i++) {
> +			ep = exfat_get_dentry(sb, &clu, i, &bh, NULL);
> +			if (!ep)
> +				return -EIO;
> +
> +			type = exfat_get_entry_type(ep);
> +			if (type == TYPE_UNUSED)
> +				break;
> +			if (type != TYPE_BITMAP)
> +				continue;
> +			if (ep->bitmap_flags == 0x0) {
> +				int err;
> +
> +				err = exfat_allocate_bitmap(sb, ep);
> +				brelse(bh);
> +				return err;
> +			}
> +		}
> +
> +		if (exfat_get_next_cluster(sb, &clu.dir))
> +			return -EIO;
> +	}
> +
> +	return -EINVAL;
> +}
> +
> +void exfat_free_bitmap(struct super_block *sb)
> +{
> +	int i;
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +
> +	brelse(sbi->pbr_bh);
> +
> +	for (i = 0; i < sbi->map_sectors; i++)
> +		__brelse(sbi->vol_amap[i]);
> +
> +	kfree(sbi->vol_amap);
> +	sbi->vol_amap = NULL;
> +}
> +
> +/*
> + * If the value of "clu" is 0, it means cluster 2 which is the first cluster of
> + * the cluster heap.
> + */
> +int exfat_set_bitmap(struct inode *inode, unsigned int clu)
> +{
> +	int i, b;
> +	struct super_block *sb = inode->i_sb;
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +
> +	i = clu >> (sb->s_blocksize_bits + 3);
> +	b = clu & ((sb->s_blocksize << 3) - 1);
> +
> +	set_bit_le(b, sbi->vol_amap[i]->b_data);
> +	exfat_update_bh(sb, sbi->vol_amap[i], IS_DIRSYNC(inode));
> +
> +	return 0;
> +}
> +
> +/*
> + * If the value of "clu" is 0, it means cluster 2 which is the first cluster of
> + * the cluster heap.
> + */
> +void exfat_clear_bitmap(struct inode *inode, unsigned int clu)
> +{
> +	int i, b;
> +	struct super_block *sb = inode->i_sb;
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +	struct exfat_mount_options *opts = &sbi->options;
> +
> +	i = clu >> (sb->s_blocksize_bits + 3);
> +	b = clu & ((sb->s_blocksize << 3) - 1);
> +
> +	clear_bit_le(b, sbi->vol_amap[i]->b_data);
> +	exfat_update_bh(sb, sbi->vol_amap[i], IS_DIRSYNC(inode));
> +
> +	if (opts->discard) {
> +		int ret_discard;
> +
> +		ret_discard = sb_issue_discard(sb,
> +				exfat_cluster_to_sector(sbi, clu + 2),
> +				(1 << sbi->sect_per_clus_bits), GFP_NOFS, 0);
> +
> +		if (ret_discard == -EOPNOTSUPP) {
> +			exfat_msg(sb, KERN_ERR,
> +				"discard not supported by device, disabling");
> +			opts->discard = 0;
> +		}
> +	}
> +}
> +
> +/*
> + * If the value of "clu" is 0, it means cluster 2 which is the first cluster of
> + * the cluster heap.
> + */
> +unsigned int exfat_test_bitmap(struct super_block *sb, unsigned int clu)
> +{
> +	unsigned int i, map_i, map_b;
> +	unsigned int clu_base, clu_free;
> +	unsigned char k, clu_mask;
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +
> +	clu_base = (clu & ~(0x7)) + 2;
> +	clu_mask = (1 << (clu - clu_base + 2)) - 1;
> +
> +	map_i = clu >> (sb->s_blocksize_bits + 3);
> +	map_b = (clu >> 3) & (unsigned int)(sb->s_blocksize - 1);
> +
> +	for (i = 2; i < sbi->num_clusters; i += 8) {
> +		k = *(sbi->vol_amap[map_i]->b_data + map_b);
> +		if (clu_mask > 0) {
> +			k |= clu_mask;
> +			clu_mask = 0;
> +		}
> +		if (k < 0xFF) {
> +			clu_free = clu_base + free_bit[k];
> +			if (clu_free < sbi->num_clusters)
> +				return clu_free;
> +		}
> +		clu_base += 8;
> +
> +		if (++map_b >= sb->s_blocksize ||
> +		    clu_base >= sbi->num_clusters) {
> +			if (++map_i >= sbi->map_sectors) {
> +				clu_base = 2;
> +				map_i = 0;
> +			}
> +			map_b = 0;
> +		}
> +	}
> +
> +	return EOF_CLUSTER;
> +}
> +
> +int exfat_count_used_clusters(struct super_block *sb, unsigned int *ret_count)
> +{
> +	struct exfat_sb_info *sbi = EXFAT_SB(sb);
> +	unsigned int count = 0;
> +	unsigned int i, map_i = 0, map_b = 0;
> +	unsigned int total_clus = sbi->num_clusters - 2;
> +	unsigned int last_mask = total_clus & 7;
> +	unsigned char clu_bits;
> +	const unsigned char last_bit_mask[] = {0, 0b00000001, 0b00000011,
> +		0b00000111, 0b00001111, 0b00011111, 0b00111111, 0b01111111};
> +
> +	total_clus &= ~last_mask;
> +	for (i = 0; i < total_clus; i += 8) {
> +		clu_bits = *(sbi->vol_amap[map_i]->b_data + map_b);
> +		count += used_bit[clu_bits];
> +		if (++map_b >= (unsigned int)sb->s_blocksize) {
> +			map_i++;
> +			map_b = 0;
> +		}
> +	}
> +
> +	if (last_mask) {
> +		clu_bits = *(sbi->vol_amap[map_i]->b_data + map_b);
> +		clu_bits &= last_bit_mask[last_mask];
> +		count += used_bit[clu_bits];
> +	}
> +
> +	*ret_count = count;
> +	return 0;
> +}
> 

  reply	other threads:[~2019-11-20  9:24 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20191119071401epcas1p4a42c781276e89928a24d53379fe13d64@epcas1p4.samsung.com>
2019-11-19  7:10 ` [PATCH v2 00/13] add the latest exfat driver Namjae Jeon
     [not found]   ` <CGME20191119071402epcas1p138f426d591ee81b65f45d092bcad0ebc@epcas1p1.samsung.com>
2019-11-19  7:10     ` [PATCH v2 01/13] exfat: add in-memory and on-disk structures and headers Namjae Jeon
2019-11-20  9:22       ` Nikolay Borisov
     [not found]   ` <CGME20191119071403epcas1p3f3d69faad57984fa3d079cf18f0a46dc@epcas1p3.samsung.com>
2019-11-19  7:10     ` [PATCH v2 02/13] exfat: add super block operations Namjae Jeon
2019-11-19  8:56       ` Daniel Wagner
2019-11-19  9:22         ` Namjae Jeon
2019-11-19 17:17           ` Christoph Hellwig
2019-11-20  4:33             ` Namjae Jeon
     [not found]   ` <CGME20191119071403epcas1p2e2a6d2fca608587547027e46d2185cb9@epcas1p2.samsung.com>
2019-11-19  7:10     ` [PATCH v2 03/13] exfat: add inode operations Namjae Jeon
     [not found]   ` <CGME20191119071404epcas1p4a26ef051aecc9386f90111eddaa4e8f5@epcas1p4.samsung.com>
2019-11-19  7:10     ` [PATCH v2 04/13] exfat: add directory operations Namjae Jeon
     [not found]   ` <CGME20191119071404epcas1p4f8df45690c07c4dd032af9cbfb5efcc6@epcas1p4.samsung.com>
2019-11-19  7:10     ` [PATCH v2 05/13] exfat: add file operations Namjae Jeon
2019-11-20  9:14       ` Nikolay Borisov
2019-11-21  1:42         ` Namjae Jeon
2019-11-21  3:18           ` Valdis Klētnieks
2019-11-21  3:39             ` Namjae Jeon
     [not found]   ` <CGME20191119071405epcas1p29e1af8242cce221c45eb529921028e48@epcas1p2.samsung.com>
2019-11-19  7:11     ` [PATCH v2 06/13] exfat: add exfat entry operations Namjae Jeon
2019-11-20  9:19       ` Nikolay Borisov
     [not found]   ` <CGME20191119071405epcas1p2c282cb850ef14d181208554796403739@epcas1p2.samsung.com>
2019-11-19  7:11     ` [PATCH v2 07/13] exfat: add bitmap operations Namjae Jeon
2019-11-20  9:24       ` Nikolay Borisov [this message]
     [not found]   ` <CGME20191119071406epcas1p34788f32abd876190f2b0a5c2ba39e4b1@epcas1p3.samsung.com>
2019-11-19  7:11     ` [PATCH v2 08/13] exfat: add exfat cache Namjae Jeon
     [not found]   ` <CGME20191119071406epcas1p285f075eac966cfdd6f79362ecc433d6b@epcas1p2.samsung.com>
2019-11-19  7:11     ` [PATCH v2 09/13] exfat: add misc operations Namjae Jeon
2020-01-08 19:53       ` Arnd Bergmann
2020-01-09 23:35         ` Namjae Jeon
     [not found]   ` <CGME20191119071407epcas1p4af1dc25ff22a70050b87f82be4cdf731@epcas1p4.samsung.com>
2019-11-19  7:11     ` [PATCH v2 10/13] exfat: add nls operations Namjae Jeon
     [not found]   ` <CGME20191119071408epcas1p355692e5e4b48c7c08617974715ae636d@epcas1p3.samsung.com>
2019-11-19  7:11     ` [PATCH v2 11/13] exfat: add Kconfig and Makefile Namjae Jeon
     [not found]   ` <CGME20191119071409epcas1p1b2464462c7972c11ae8719528f0c43a8@epcas1p1.samsung.com>
2019-11-19  7:11     ` [PATCH v2 12/13] exfat: add exfat in fs/Kconfig and fs/Makefile Namjae Jeon
     [not found]   ` <CGME20191119071409epcas1p2253bc4b3be05ac82201126bc62bd37ac@epcas1p2.samsung.com>
2019-11-19  7:11     ` [PATCH v2 13/13] MAINTAINERS: add exfat filesystem Namjae Jeon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=09920df0-ff2d-72f9-b82f-21e0ac5499ac@suse.com \
    --to=nborisov@suse.com \
    --cc=Markus.Elfring@web.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=hch@lst.de \
    --cc=linkinjeon@gmail.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=namjae.jeon@samsung.com \
    --cc=sj1557.seo@samsung.com \
    --cc=valdis.kletnieks@vt.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®