Re: [PATCH] vfs: make open_with_fake_path() not contribute to nr_files

2018-07-19 Thread David Howells
Miklos Szeredi wrote: > What the cachefiles use case would be? Cachfiles has to open the backing file so that it can write to it, and it has to do it every time it writes because to leave a bunch of files open contributes to ENFILE/EMFILE. In the near future it's going to have to open the backi

Re: [PATCH] vfs: make open_with_fake_path() not contribute to nr_files

2018-07-19 Thread Miklos Szeredi
On Thu, Jul 19, 2018 at 10:09 AM, David Howells wrote: > Miklos Szeredi wrote: > >> Stacking file operations in overlay will store an extra open file for each >> overlay file opened. >> >> The overhead is just that of "struct file" which is about 256bytes, because >> overlay already pins an extra

Re: [PATCH] vfs: make open_with_fake_path() not contribute to nr_files

2018-07-19 Thread David Howells
Miklos Szeredi wrote: > Stacking file operations in overlay will store an extra open file for each > overlay file opened. > > The overhead is just that of "struct file" which is about 256bytes, because > overlay already pins an extra dentry and inode when the file is open, which > add up to a mu

[PATCH] vfs: make open_with_fake_path() not contribute to nr_files

2018-07-18 Thread Miklos Szeredi
Stacking file operations in overlay will store an extra open file for each overlay file opened. The overhead is just that of "struct file" which is about 256bytes, because overlay already pins an extra dentry and inode when the file is open, which add up to a much larger overhead. For fear of bre