diff mbox

writeback: initialize inode members that track writeback history

Message ID 1455658479-12433-1-git-send-email-tahsin@google.com (mailing list archive)
State New, archived
Headers show

Commit Message

Tahsin Erdogan Feb. 16, 2016, 9:34 p.m. UTC 
inode struct members that track cgroup writeback information
should be reinitialized when inode gets allocated from
kmem_cache. Otherwise, their values remain and get used by the
new inode.

Signed-off-by: Tahsin Erdogan <tahsin@google.com>
Acked-by: Tejun Heo <tj@kernel.org>
---
 fs/inode.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Tejun Heo Feb. 16, 2016, 9:49 p.m. UTC | #1 
Hello,

On Tue, Feb 16, 2016 at 01:34:39PM -0800, Tahsin Erdogan wrote:
> inode struct members that track cgroup writeback information
> should be reinitialized when inode gets allocated from
> kmem_cache. Otherwise, their values remain and get used by the
> new inode.
> 
> Signed-off-by: Tahsin Erdogan <tahsin@google.com>
> Acked-by: Tejun Heo <tj@kernel.org>

This one should go for 4.5 and also prolly have the following tag.

Fixes: d10c80955265 ("writeback: implement foreign cgroup inode bdi_writeback switching")

Thanks.
Jens Axboe Feb. 17, 2016, 3:30 p.m. UTC | #2 
On 02/16/2016 02:49 PM, Tejun Heo wrote:
> Hello,
>
> On Tue, Feb 16, 2016 at 01:34:39PM -0800, Tahsin Erdogan wrote:
>> inode struct members that track cgroup writeback information
>> should be reinitialized when inode gets allocated from
>> kmem_cache. Otherwise, their values remain and get used by the
>> new inode.
>>
>> Signed-off-by: Tahsin Erdogan <tahsin@google.com>
>> Acked-by: Tejun Heo <tj@kernel.org>
>
> This one should go for 4.5 and also prolly have the following tag.
>
> Fixes: d10c80955265 ("writeback: implement foreign cgroup inode bdi_writeback switching")

I've added it for 4.5 with the fixes tag.
diff mbox

Patch

diff --git a/fs/inode.c b/fs/inode.c
index 9f62db3..69b8b52 100644
--- a/fs/inode.c
+++ b/fs/inode.c
@@ -154,6 +154,12 @@  int inode_init_always(struct super_block *sb, struct inode *inode)
 	inode->i_rdev = 0;
 	inode->dirtied_when = 0;
 
+#ifdef CONFIG_CGROUP_WRITEBACK
+	inode->i_wb_frn_winner = 0;
+	inode->i_wb_frn_avg_time = 0;
+	inode->i_wb_frn_history = 0;
+#endif
+
 	if (security_inode_alloc(inode))
 		goto out;
 	spin_lock_init(&inode->i_lock);