lkml.org 
[lkml]   [2017]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] writeback: synchronize sync(2) against cgroup writeback membership switches
Hello,

On Wed, Dec 13, 2017 at 12:00:04PM +0100, Jan Kara wrote:
> OK, but this effectively prevents writeback from sync_inodes_sb() to ever
> make inode switch wbs. Cannot that be abused in some way like making sure
> writeback of our memcg is "invisible" by forcing it out using sync(2)? It
> just looks a bit dangerous to me...

That's true. There are a couple mitigating factors tho.

* While it can delay switching during sync(2), it'll all still be
recorded and the switch will happen soon if needed.

* sync(2) is hugely disruptive with or without this and can easily be
used to DOS the whole system. People are working on restricting the
blast radius of sync(2) to mitigate this problem, which most likely
make this a non-problem too.

If you can think of a better solution, I'm all ears.

Thanks.

--
tejun

\
 
 \ /
  Last update: 2017-12-13 16:39    [W:0.065 / U:1.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site