Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-02-11 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/10, Jaegeuk Kim wrote: > On 02/10, Matthew Wilcox wrote: > > On Mon, Feb 10, 2025 at 05:00:47PM +, Jaegeuk Kim wrote: > > > On 02/04, Jaegeuk Kim wrote: > > > > On 01/17, Jaegeuk Kim wrote: > > > > > On 01/17, Matthew Wilcox wrote: > > > > > > On Fri, Jan 17, 2025 at 06:48:55PM +, Jae

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-02-10 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/10, Matthew Wilcox wrote: > On Mon, Feb 10, 2025 at 05:00:47PM +, Jaegeuk Kim wrote: > > On 02/04, Jaegeuk Kim wrote: > > > On 01/17, Jaegeuk Kim wrote: > > > > On 01/17, Matthew Wilcox wrote: > > > > > On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > > > > > > I don't und

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-02-10 Thread Matthew Wilcox
On Mon, Feb 10, 2025 at 05:00:47PM +, Jaegeuk Kim wrote: > On 02/04, Jaegeuk Kim wrote: > > On 01/17, Jaegeuk Kim wrote: > > > On 01/17, Matthew Wilcox wrote: > > > > On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > > > > > I don't understand how this is different from MADV_COLD

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-02-10 Thread Jaegeuk Kim via Linux-f2fs-devel
On 02/04, Jaegeuk Kim wrote: > On 01/17, Jaegeuk Kim wrote: > > On 01/17, Matthew Wilcox wrote: > > > On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > > > > I don't understand how this is different from MADV_COLD. Please > > > > > explain. > > > > > > > > MADV_COLD is a vma range,

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-02-04 Thread Jaegeuk Kim via Linux-f2fs-devel
On 01/17, Jaegeuk Kim wrote: > On 01/17, Matthew Wilcox wrote: > > On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > > > I don't understand how this is different from MADV_COLD. Please > > > > explain. > > > > > > MADV_COLD is a vma range, while this is a file range. So, it's more

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-01-17 Thread Jaegeuk Kim via Linux-f2fs-devel
On 01/17, Matthew Wilcox wrote: > On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > > I don't understand how this is different from MADV_COLD. Please > > > explain. > > > > MADV_COLD is a vma range, while this is a file range. So, it's more close to > > fadvise(POSIX_FADV_DONTNEED)

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-01-17 Thread Matthew Wilcox
On Fri, Jan 17, 2025 at 06:48:55PM +, Jaegeuk Kim wrote: > > I don't understand how this is different from MADV_COLD. Please > > explain. > > MADV_COLD is a vma range, while this is a file range. So, it's more close to > fadvise(POSIX_FADV_DONTNEED) which tries to reclaim the file-backed page

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-01-17 Thread Jaegeuk Kim via Linux-f2fs-devel
On 01/17, Matthew Wilcox wrote: > On Fri, Jan 17, 2025 at 04:41:16PM +, Jaegeuk Kim wrote: > > If users clearly know which file-backed pages to reclaim in system view, > > they > > can use this ioctl() to register in advance and reclaim all at once later. > > > > To MM and others, > > > > I'

Re: [f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-01-17 Thread Matthew Wilcox
On Fri, Jan 17, 2025 at 04:41:16PM +, Jaegeuk Kim wrote: > If users clearly know which file-backed pages to reclaim in system view, they > can use this ioctl() to register in advance and reclaim all at once later. > > To MM and others, > > I'd like to propose this API in F2FS only, since > 1)

[f2fs-dev] [PATCH 0/2 v6] add ioctl/sysfs to donate file-backed pages

2025-01-17 Thread Jaegeuk Kim via Linux-f2fs-devel
If users clearly know which file-backed pages to reclaim in system view, they can use this ioctl() to register in advance and reclaim all at once later. To MM and others, I'd like to propose this API in F2FS only, since 1) the use-case is quite limited in Android at the moment. Once it's generall