[PATCH v4] ubifs: introduce UBIFS_ATIME_SUPPORT to ubifs
Dongsheng Yang
yangds.fnst at cn.fujitsu.com
Mon Aug 10 19:02:33 PDT 2015
On 08/10/2015 04:32 PM, Artem Bityutskiy wrote:
> Could you please describe how you tested your patch. Did you run stress
> tests with atime enabled, which ones?
About the functionality testing, I tried xfstests, it passed.
About the stress tests, I tried the fio and xfstests for stress.
Below is my result:
* I added a counter in ubifs_dirty_inode() to see how many times
we dirtied inode in my testing.*
diff --git a/fs/ubifs/super.c b/fs/ubifs/super.c
index 1f2415f..aac1da2 100644
--- a/fs/ubifs/super.c
+++ b/fs/ubifs/super.c
@@ -398,6 +398,7 @@ static void ubifs_dirty_inode(struct inode *inode,
int flags)
int locked = mutex_is_locked(&ui->ui_mutex);
struct ubifs_info *c = inode->i_sb->s_fs_info;
int ret = 0;
+ static unsigned long long count = 0;
if (!locked)
mutex_lock(&ui->ui_mutex);
@@ -412,6 +413,7 @@ static void ubifs_dirty_inode(struct inode *inode,
int flags)
}
ui->dirty = 1;
dbg_gen("inode %lu", inode->i_ino);
+ pr_info("count: %llu\n", ++count);
}
RESULT:
stress test steps: (for ((i=0;i<10;i++)); do ./check -g stress; done)
noatime: 67169
relatime: 68084
strictatime: 68643
Atem, Do you want some other testing for it?
>
> On Fri, 2015-07-31 at 09:09 +0800, Dongsheng Yang wrote:
>> +> > > sb->s_flags |= MS_ACTIVE;
>> +#ifndef CONFIG_UBIFS_ATIME_SUPPORT
>> +> > > sb->s_flags |= MS_NOATIME;
>> +#else
>> +> > > ubifs_warn(c, "************WARNING START****************");
>> +> > > ubifs_warn(c, "Ubifs is supporting atime now, that would");
>> +> > > ubifs_warn(c, "probably damage your flash. If you are not");
>> +> > > ubifs_warn(c, "sure about it, please set UBIFS_ATIME_SUPPORT");
>> +> > > ubifs_warn(c, "to 'N'.");
>> +> > > ubifs_warn(c, "************WARNING END******************");
>
> Could we please be a little be less scary.
>
> 1. I think these starting and ending "*****" lines are unnecessary.
> 2. Ah, and use "UBIFS", not "Ubifs" please.
> 3. Let's be shorter, just this should be enough:
>
> ubifs_warn("full atime support is enabled, which may wear out your
> flash faster");
Okey, will update it.
Yang
>
> Artem.
> .
>
More information about the linux-mtd
mailing list