From: "Pali Rohár" <pali.rohar@gmail.com>
To: Namjae Jeon <namjae.jeon@samsung.com>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
gregkh@linuxfoundation.org, valdis.kletnieks@vt.edu, hch@lst.de,
sj1557.seo@samsung.com, linkinjeon@gmail.com
Subject: Re: [PATCH v8 08/13] exfat: add exfat cache
Date: Sun, 29 Dec 2019 15:23:40 +0100 [thread overview]
Message-ID: <20191229142340.sr2xbmwlczb2ytjj@pali> (raw)
In-Reply-To: <20191220062419.23516-9-namjae.jeon@samsung.com>
[-- Attachment #1: Type: text/plain, Size: 9911 bytes --]
Hello! This patch looks like a copy-pase of fat cache implementation.
It is possible to avoid duplicating code?
On Friday 20 December 2019 01:24:14 Namjae Jeon wrote:
> This adds the implementation of exfat cache.
>
> Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
> Signed-off-by: Sungjong Seo <sj1557.seo@samsung.com>
> ---
> fs/exfat/cache.c | 325 +++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 325 insertions(+)
> create mode 100644 fs/exfat/cache.c
>
> diff --git a/fs/exfat/cache.c b/fs/exfat/cache.c
> new file mode 100644
> index 000000000000..03d0824fc368
> --- /dev/null
> +++ b/fs/exfat/cache.c
> @@ -0,0 +1,325 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +/*
> + * linux/fs/fat/cache.c
> + *
> + * Written 1992,1993 by Werner Almesberger
> + *
> + * Mar 1999. AV. Changed cache, so that it uses the starting cluster instead
> + * of inode number.
> + * May 1999. AV. Fixed the bogosity with FAT32 (read "FAT28"). Fscking lusers.
> + * Copyright (C) 2012-2013 Samsung Electronics Co., Ltd.
> + */
> +
> +#include <linux/slab.h>
> +#include <asm/unaligned.h>
> +#include <linux/buffer_head.h>
> +
> +#include "exfat_raw.h"
> +#include "exfat_fs.h"
> +
> +#define EXFAT_CACHE_VALID 0
> +#define EXFAT_MAX_CACHE 16
> +
> +struct exfat_cache {
> + struct list_head cache_list;
> + unsigned int nr_contig; /* number of contiguous clusters */
> + unsigned int fcluster; /* cluster number in the file. */
> + unsigned int dcluster; /* cluster number on disk. */
> +};
> +
> +struct exfat_cache_id {
> + unsigned int id;
> + unsigned int nr_contig;
> + unsigned int fcluster;
> + unsigned int dcluster;
> +};
> +
> +static struct kmem_cache *exfat_cachep;
> +
> +static void exfat_cache_init_once(void *c)
> +{
> + struct exfat_cache *cache = (struct exfat_cache *)c;
> +
> + INIT_LIST_HEAD(&cache->cache_list);
> +}
> +
> +int exfat_cache_init(void)
> +{
> + exfat_cachep = kmem_cache_create("exfat_cache",
> + sizeof(struct exfat_cache),
> + 0, SLAB_RECLAIM_ACCOUNT|SLAB_MEM_SPREAD,
> + exfat_cache_init_once);
> + if (!exfat_cachep)
> + return -ENOMEM;
> + return 0;
> +}
> +
> +void exfat_cache_shutdown(void)
> +{
> + if (!exfat_cachep)
> + return;
> + kmem_cache_destroy(exfat_cachep);
> +}
> +
> +void exfat_cache_init_inode(struct inode *inode)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> +
> + spin_lock_init(&ei->cache_lru_lock);
> + ei->nr_caches = 0;
> + ei->cache_valid_id = EXFAT_CACHE_VALID + 1;
> + INIT_LIST_HEAD(&ei->cache_lru);
> +}
> +
> +static inline struct exfat_cache *exfat_cache_alloc(void)
> +{
> + return kmem_cache_alloc(exfat_cachep, GFP_NOFS);
> +}
> +
> +static inline void exfat_cache_free(struct exfat_cache *cache)
> +{
> + WARN_ON(!list_empty(&cache->cache_list));
> + kmem_cache_free(exfat_cachep, cache);
> +}
> +
> +static inline void exfat_cache_update_lru(struct inode *inode,
> + struct exfat_cache *cache)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> +
> + if (ei->cache_lru.next != &cache->cache_list)
> + list_move(&cache->cache_list, &ei->cache_lru);
> +}
> +
> +static unsigned int exfat_cache_lookup(struct inode *inode,
> + unsigned int fclus, struct exfat_cache_id *cid,
> + unsigned int *cached_fclus, unsigned int *cached_dclus)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> + static struct exfat_cache nohit = { .fcluster = 0, };
> + struct exfat_cache *hit = &nohit, *p;
> + unsigned int offset = EXFAT_EOF_CLUSTER;
> +
> + spin_lock(&ei->cache_lru_lock);
> + list_for_each_entry(p, &ei->cache_lru, cache_list) {
> + /* Find the cache of "fclus" or nearest cache. */
> + if (p->fcluster <= fclus && hit->fcluster < p->fcluster) {
> + hit = p;
> + if (hit->fcluster + hit->nr_contig < fclus) {
> + offset = hit->nr_contig;
> + } else {
> + offset = fclus - hit->fcluster;
> + break;
> + }
> + }
> + }
> + if (hit != &nohit) {
> + exfat_cache_update_lru(inode, hit);
> +
> + cid->id = ei->cache_valid_id;
> + cid->nr_contig = hit->nr_contig;
> + cid->fcluster = hit->fcluster;
> + cid->dcluster = hit->dcluster;
> + *cached_fclus = cid->fcluster + offset;
> + *cached_dclus = cid->dcluster + offset;
> + }
> + spin_unlock(&ei->cache_lru_lock);
> +
> + return offset;
> +}
> +
> +static struct exfat_cache *exfat_cache_merge(struct inode *inode,
> + struct exfat_cache_id *new)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> + struct exfat_cache *p;
> +
> + list_for_each_entry(p, &ei->cache_lru, cache_list) {
> + /* Find the same part as "new" in cluster-chain. */
> + if (p->fcluster == new->fcluster) {
> + if (new->nr_contig > p->nr_contig)
> + p->nr_contig = new->nr_contig;
> + return p;
> + }
> + }
> + return NULL;
> +}
> +
> +static void exfat_cache_add(struct inode *inode,
> + struct exfat_cache_id *new)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> + struct exfat_cache *cache, *tmp;
> +
> + if (new->fcluster == EXFAT_EOF_CLUSTER) /* dummy cache */
> + return;
> +
> + spin_lock(&ei->cache_lru_lock);
> + if (new->id != EXFAT_CACHE_VALID &&
> + new->id != ei->cache_valid_id)
> + goto unlock; /* this cache was invalidated */
> +
> + cache = exfat_cache_merge(inode, new);
> + if (cache == NULL) {
> + if (ei->nr_caches < EXFAT_MAX_CACHE) {
> + ei->nr_caches++;
> + spin_unlock(&ei->cache_lru_lock);
> +
> + tmp = exfat_cache_alloc();
> + if (!tmp) {
> + spin_lock(&ei->cache_lru_lock);
> + ei->nr_caches--;
> + spin_unlock(&ei->cache_lru_lock);
> + return;
> + }
> +
> + spin_lock(&ei->cache_lru_lock);
> + cache = exfat_cache_merge(inode, new);
> + if (cache != NULL) {
> + ei->nr_caches--;
> + exfat_cache_free(tmp);
> + goto out_update_lru;
> + }
> + cache = tmp;
> + } else {
> + struct list_head *p = ei->cache_lru.prev;
> +
> + cache = list_entry(p,
> + struct exfat_cache, cache_list);
> + }
> + cache->fcluster = new->fcluster;
> + cache->dcluster = new->dcluster;
> + cache->nr_contig = new->nr_contig;
> + }
> +out_update_lru:
> + exfat_cache_update_lru(inode, cache);
> +unlock:
> + spin_unlock(&ei->cache_lru_lock);
> +}
> +
> +/*
> + * Cache invalidation occurs rarely, thus the LRU chain is not updated. It
> + * fixes itself after a while.
> + */
> +static void __exfat_cache_inval_inode(struct inode *inode)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> + struct exfat_cache *cache;
> +
> + while (!list_empty(&ei->cache_lru)) {
> + cache = list_entry(ei->cache_lru.next,
> + struct exfat_cache, cache_list);
> + list_del_init(&cache->cache_list);
> + ei->nr_caches--;
> + exfat_cache_free(cache);
> + }
> + /* Update. The copy of caches before this id is discarded. */
> + ei->cache_valid_id++;
> + if (ei->cache_valid_id == EXFAT_CACHE_VALID)
> + ei->cache_valid_id++;
> +}
> +
> +void exfat_cache_inval_inode(struct inode *inode)
> +{
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> +
> + spin_lock(&ei->cache_lru_lock);
> + __exfat_cache_inval_inode(inode);
> + spin_unlock(&ei->cache_lru_lock);
> +}
> +
> +static inline int cache_contiguous(struct exfat_cache_id *cid,
> + unsigned int dclus)
> +{
> + cid->nr_contig++;
> + return cid->dcluster + cid->nr_contig == dclus;
> +}
> +
> +static inline void cache_init(struct exfat_cache_id *cid,
> + unsigned int fclus, unsigned int dclus)
> +{
> + cid->id = EXFAT_CACHE_VALID;
> + cid->fcluster = fclus;
> + cid->dcluster = dclus;
> + cid->nr_contig = 0;
> +}
> +
> +int exfat_get_cluster(struct inode *inode, unsigned int cluster,
> + unsigned int *fclus, unsigned int *dclus,
> + unsigned int *last_dclus, int allow_eof)
> +{
> + struct super_block *sb = inode->i_sb;
> + struct exfat_sb_info *sbi = EXFAT_SB(sb);
> + unsigned int limit = sbi->num_clusters;
> + struct exfat_inode_info *ei = EXFAT_I(inode);
> + struct exfat_cache_id cid;
> + unsigned int content;
> +
> + if (ei->start_clu == EXFAT_FREE_CLUSTER) {
> + exfat_fs_error(sb,
> + "invalid access to exfat cache (entry 0x%08x)",
> + ei->start_clu);
> + return -EIO;
> + }
> +
> + *fclus = 0;
> + *dclus = ei->start_clu;
> + *last_dclus = *dclus;
> +
> + /*
> + * Don`t use exfat_cache if zero offset or non-cluster allocation
> + */
> + if (cluster == 0 || *dclus == EXFAT_EOF_CLUSTER)
> + return 0;
> +
> + cache_init(&cid, EXFAT_EOF_CLUSTER, EXFAT_EOF_CLUSTER);
> +
> + if (exfat_cache_lookup(inode, cluster, &cid, fclus, dclus) ==
> + EXFAT_EOF_CLUSTER) {
> + /*
> + * dummy, always not contiguous
> + * This is reinitialized by cache_init(), later.
> + */
> + WARN_ON(cid.id != EXFAT_CACHE_VALID ||
> + cid.fcluster != EXFAT_EOF_CLUSTER ||
> + cid.dcluster != EXFAT_EOF_CLUSTER ||
> + cid.nr_contig != 0);
> + }
> +
> + if (*fclus == cluster)
> + return 0;
> +
> + while (*fclus < cluster) {
> + /* prevent the infinite loop of cluster chain */
> + if (*fclus > limit) {
> + exfat_fs_error(sb,
> + "detected the cluster chain loop (i_pos %u)",
> + (*fclus));
> + return -EIO;
> + }
> +
> + if (exfat_ent_get(sb, *dclus, &content))
> + return -EIO;
> +
> + *last_dclus = *dclus;
> + *dclus = content;
> + (*fclus)++;
> +
> + if (content == EXFAT_EOF_CLUSTER) {
> + if (!allow_eof) {
> + exfat_fs_error(sb,
> + "invalid cluster chain (i_pos %u, last_clus 0x%08x is EOF)",
> + *fclus, (*last_dclus));
> + return -EIO;
> + }
> +
> + break;
> + }
> +
> + if (!cache_contiguous(&cid, *dclus))
> + cache_init(&cid, *fclus, *dclus);
> + }
> +
> + exfat_cache_add(inode, &cid);
> + return 0;
> +}
--
Pali Rohár
pali.rohar@gmail.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
next prev parent reply other threads:[~2019-12-29 14:23 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20191220062731epcas1p475b8da9288b08c87e474a0c4e88ce219@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 00/13] add the latest exfat driver Namjae Jeon
[not found] ` <CGME20191220062732epcas1p17f3b1066fb4d6496559f349f950e1751@epcas1p1.samsung.com>
2019-12-20 6:24 ` [PATCH v8 01/13] exfat: add in-memory and on-disk structures and headers Namjae Jeon
2019-12-29 14:11 ` Pali Rohár
2019-12-31 13:46 ` Namjae Jeon
2020-01-02 11:06 ` Pali Rohár
2020-01-02 11:11 ` Namjae Jeon
2019-12-31 13:25 ` Markus Elfring
2019-12-31 14:05 ` Markus Elfring
[not found] ` <CGME20191220062733epcas1p31665a3ae968ab8c70d91a3cddf529e73@epcas1p3.samsung.com>
2019-12-20 6:24 ` [PATCH v8 02/13] exfat: add super block operations Namjae Jeon
2019-12-29 13:40 ` Pali Rohár
2020-01-02 6:06 ` Namjae Jeon
2020-01-02 8:30 ` Pali Rohár
2020-01-02 13:13 ` Namjae Jeon
2020-01-02 13:16 ` Pali Rohár
2020-01-02 13:41 ` Namjae Jeon
2020-01-02 14:13 ` Pali Rohár
2019-12-29 13:55 ` Pali Rohár
2019-12-31 13:32 ` Namjae Jeon
2020-01-08 19:55 ` Arnd Bergmann
2020-01-09 23:36 ` Namjae Jeon
[not found] ` <CGME20191220062733epcas1p1afd7af6ca2bfbde3d9a883f55f4f3b60@epcas1p1.samsung.com>
2019-12-20 6:24 ` [PATCH v8 03/13] exfat: add inode operations Namjae Jeon
2019-12-29 14:15 ` Pali Rohár
2019-12-31 13:53 ` Namjae Jeon
[not found] ` <CGME20191220062734epcas1p4888919aeb168f55595fec24b624eacc6@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 04/13] exfat: add directory operations Namjae Jeon
[not found] ` <CGME20191220062734epcas1p2ba4db24017a2eca92eeeedd101200a7f@epcas1p2.samsung.com>
2019-12-20 6:24 ` [PATCH v8 05/13] exfat: add file operations Namjae Jeon
[not found] ` <CGME20191220062735epcas1p4aaffe2759e136136fc42b9764b6eb68a@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 06/13] exfat: add exfat entry operations Namjae Jeon
[not found] ` <CGME20191220062735epcas1p1a4b960f26a520d26b5cad7aebc7e91cb@epcas1p1.samsung.com>
2019-12-20 6:24 ` [PATCH v8 07/13] exfat: add bitmap operations Namjae Jeon
[not found] ` <CGME20191220062736epcas1p3c58bf86018ba9caef90b3a6476b4b925@epcas1p3.samsung.com>
2019-12-20 6:24 ` [PATCH v8 08/13] exfat: add exfat cache Namjae Jeon
2019-12-29 14:23 ` Pali Rohár [this message]
2019-12-31 13:52 ` Namjae Jeon
2020-01-02 10:19 ` [v8 " Markus Elfring
2020-01-08 18:08 ` Pali Rohár
2020-01-08 18:46 ` Christoph Hellwig
2020-01-08 19:01 ` Pali Rohár
[not found] ` <CGME20191220062737epcas1p3c0f9e408640148c9186b84efc6d6658b@epcas1p3.samsung.com>
2019-12-20 6:24 ` [PATCH v8 09/13] exfat: add misc operations Namjae Jeon
2019-12-29 14:00 ` Pali Rohár
2019-12-31 13:34 ` Namjae Jeon
[not found] ` <CGME20191220062737epcas1p436e6ecee04e4bee088d915f84b53c899@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 10/13] exfat: add nls operations Namjae Jeon
2019-12-31 14:23 ` Markus Elfring
2019-12-31 18:01 ` Joe Perches
2020-01-02 9:56 ` Markus Elfring
[not found] ` <CGME20191220062738epcas1p2aa7a91f04263efc3d6d7200eb04c1296@epcas1p2.samsung.com>
2019-12-20 6:24 ` [PATCH v8 11/13] exfat: add Kconfig and Makefile Namjae Jeon
[not found] ` <CGME20191220062739epcas1p4fcda127a4b5a4e06bf53f7da4dbd045c@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 12/13] exfat: add exfat in fs/Kconfig and fs/Makefile Namjae Jeon
2019-12-23 22:17 ` kbuild test robot
[not found] ` <CGME20191220062739epcas1p42f6096d78247630cfd02fc069ca072d3@epcas1p4.samsung.com>
2019-12-20 6:24 ` [PATCH v8 13/13] MAINTAINERS: add exfat filesystem Namjae Jeon
2019-12-31 13:00 ` [PATCH v8 00/13] add the latest exfat driver Markus Elfring
2019-12-31 15:14 ` Matthew Wilcox
2019-12-31 23:56 ` 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=20191229142340.sr2xbmwlczb2ytjj@pali \
--to=pali.rohar@gmail.com \
--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;
as well as URLs for NNTP newsgroup(s).