On Mon, Mar 23, 2009 at 09:20:24PM +0530, Aneesh Kumar K.V wrote: > On Sat, Feb 28, 2009 at 12:01:52AM -0500, Theodore Ts'o wrote: > > > > Add a new superblock value which tracks the lifetime amount of writes > > to the filesystem. This is useful in estimating the amount of wear on > > solid state drives (SSD's) caused by writes to the filesystem. > > > > Signed-off-by: "Theodore Ts'o" <tytso@xxxxxxx> > > --- > > fs/ext4/ext4.h | 3 ++- > > fs/ext4/ext4_sb.h | 4 ++++ > > fs/ext4/super.c | 7 +++++++ > > 3 files changed, 13 insertions(+), 1 deletions(-) > > > > diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h > > index 8599e37..f9a3bc3 100644 > > --- a/fs/ext4/ext4.h > > +++ b/fs/ext4/ext4.h > > @@ -685,7 +685,8 @@ struct ext4_super_block { > > __u8 s_log_groups_per_flex; /* FLEX_BG group size */ > > __u8 s_reserved_char_pad2; > > __le16 s_reserved_pad; > > - __u32 s_reserved[162]; /* Padding to the end of the block */ > > + __le64 s_kbytes_written; /* nr of lifetime kilobytes written */ > > + __u32 s_reserved[160]; /* Padding to the end of the block */ > > }; > > > > #ifdef __KERNEL__ > > diff --git a/fs/ext4/ext4_sb.h b/fs/ext4/ext4_sb.h > > index 4e4d9cc..50ab116 100644 > > --- a/fs/ext4/ext4_sb.h > > +++ b/fs/ext4/ext4_sb.h > > @@ -142,6 +142,10 @@ struct ext4_sb_info { > > /* locality groups */ > > struct ext4_locality_group *s_locality_groups; > > > > + /* for write statistics */ > > + unsigned long s_sectors_written_start; > > + u64 s_kbytes_written; > > + > > unsigned int s_log_groups_per_flex; > > struct flex_groups *s_flex_groups; > > }; > > Do we need this change ? We don't seems to be using it anywhere. > Looked at the patchqueue and found that you have users in sysfs-support patch. I guess it would be better if you put this patch after sysfs-support and moved session_write_kbytes_show and others to this patch. -aneesh -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html