diff options
| author |  Yuezhang Mo <Yuezhang.Mo@sony.com>
| 2023-02-09 17:44:50 +0800 |
|---|---|---|
| committer |  Namjae Jeon <linkinjeon@kernel.org>
| 2023-02-28 20:01:36 +0900 |
| commit | 3ce937cb8ca9becec406611c6072e93030fdde76 (patch) | |
| tree | 9e5f62012b8273902ee9a359900592cdfc94198d /fs/exfat | |
| parent | exfat: remove unneeded code from exfat_alloc_cluster() (diff) | |
| download | linux-3ce937cb8ca9becec406611c6072e93030fdde76.tar.gz linux-3ce937cb8ca9becec406611c6072e93030fdde76.tar.bz2 linux-3ce937cb8ca9becec406611c6072e93030fdde76.zip | |
exfat: don't print error log in normal case
When allocating a new cluster, exFAT first allocates from the
next cluster of the last cluster of the file. If the last cluster
of the file is the last cluster of the volume, allocate from the
first cluster. This is a normal case, but the following error log
will be printed. It makes users confused, so this commit removes
the error log.
[1960905.181545] exFAT-fs (sdb1): hint_cluster is invalid (262130)
Signed-off-by: Yuezhang Mo <Yuezhang.Mo@sony.com>
Reviewed-by: Andy Wu <Andy.Wu@sony.com>
Reviewed-by: Sungjong Seo <sj1557.seo@samsung.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Diffstat (limited to 'fs/exfat')
| -rw-r--r-- | fs/exfat/fatent.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/exfat/fatent.c b/fs/exfat/fatent.c index 65a8c9fb072c..c75c5a2cad42 100644 --- a/fs/exfat/fatent.c +++ b/fs/exfat/fatent.c @@ -344,8 +344,9 @@ int exfat_alloc_cluster(struct inode *inode, unsigned int num_alloc, /* check cluster validation */ if (!is_valid_cluster(sbi, hint_clu)) { - exfat_err(sb, "hint_cluster is invalid (%u)", - hint_clu); + if (hint_clu != sbi->num_clusters) + exfat_err(sb, "hint_cluster is invalid (%u), rewind to the first cluster", + hint_clu); hint_clu = EXFAT_FIRST_CLUSTER; p_chain->flags = ALLOC_FAT_CHAIN; } |