Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-11-17 Thread Minchan Kim
On Tue, Nov 17, 2020 at 12:06:48PM -0800, Linus Torvalds wrote: > On Mon, Nov 16, 2020 at 7:51 AM Minchan Kim wrote: > > > > Let me send a patch with your SoB if you don't mind. > > Eric, can you ack this SoB and I'll apply it to me tree? > > Or is it already queued up somewhere else? Andrew,

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-11-17 Thread Linus Torvalds
On Mon, Nov 16, 2020 at 7:51 AM Minchan Kim wrote: > > Let me send a patch with your SoB if you don't mind. Eric, can you ack this SoB and I'll apply it to me tree? Or is it already queued up somewhere else? Linus

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-11-16 Thread Minchan Kim
On Mon, Nov 16, 2020 at 10:02:42AM +0100, Eric Dumazet wrote: < snip > > > From 02d63c6b3f61a1085f4eab80f5171bd2627b5ab0 Mon Sep 17 00:00:00 2001 > > From: Minchan Kim > > Date: Mon, 21 Sep 2020 09:31:25 -0700 > > Subject: [PATCH] mm: do not use helper functions for process_madvise > > > >

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-11-16 Thread Eric Dumazet
On 9/21/20 7:55 PM, Minchan Kim wrote: > On Mon, Sep 21, 2020 at 07:56:33AM +0100, Christoph Hellwig wrote: >> On Mon, Aug 31, 2020 at 05:06:33PM -0700, Minchan Kim wrote: >>> There is usecase that System Management Software(SMS) want to give a >>> memory hint like MADV_[COLD|PAGEEOUT] to other

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-21 Thread Minchan Kim
On Mon, Sep 21, 2020 at 07:56:33AM +0100, Christoph Hellwig wrote: > On Mon, Aug 31, 2020 at 05:06:33PM -0700, Minchan Kim wrote: > > There is usecase that System Management Software(SMS) want to give a > > memory hint like MADV_[COLD|PAGEEOUT] to other processes and in the > > case of Android, it

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-21 Thread Michal Hocko
On Mon 21-09-20 07:56:33, Christoph Hellwig wrote: > On Mon, Aug 31, 2020 at 05:06:33PM -0700, Minchan Kim wrote: > > There is usecase that System Management Software(SMS) want to give a > > memory hint like MADV_[COLD|PAGEEOUT] to other processes and in the > > case of Android, it is the

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-21 Thread Christoph Hellwig
On Mon, Aug 31, 2020 at 05:06:33PM -0700, Minchan Kim wrote: > There is usecase that System Management Software(SMS) want to give a > memory hint like MADV_[COLD|PAGEEOUT] to other processes and in the > case of Android, it is the ActivityManagerService. > > The information required to make the

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-04 Thread Christian Brauner
On Thu, Sep 03, 2020 at 10:59:49AM -0700, Minchan Kim wrote: > On Thu, Sep 03, 2020 at 07:34:58PM +0200, Florian Weimer wrote: > > * Minchan Kim: > > > > > On Tue, Sep 01, 2020 at 08:46:02PM +0200, Florian Weimer wrote: > > >> * Minchan Kim: > > >> > > >> > ssize_t process_madvise(int

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-03 Thread Minchan Kim
tency to be had here. Okay, I changed it with size_t. >From c9f079a2008fd05a38d3f0fd69c2c0e586e8ee7a Mon Sep 17 00:00:00 2001 From: Minchan Kim Date: Mon, 31 Aug 2020 15:36:30 -0700 Subject: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API There

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-03 Thread Florian Weimer
* Minchan Kim: > On Tue, Sep 01, 2020 at 08:46:02PM +0200, Florian Weimer wrote: >> * Minchan Kim: >> >> > ssize_t process_madvise(int pidfd, const struct iovec *iovec, >> > unsigned long vlen, int advice, unsigned int flags); >> >> size_t for vlen provides a clearer hint

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-03 Thread Minchan Kim
On Tue, Sep 01, 2020 at 08:46:02PM +0200, Florian Weimer wrote: > * Minchan Kim: > > > ssize_t process_madvise(int pidfd, const struct iovec *iovec, > > unsigned long vlen, int advice, unsigned int flags); > > size_t for vlen provides a clearer hint regarding the type of

Re: [PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-09-01 Thread Florian Weimer
* Minchan Kim: > ssize_t process_madvise(int pidfd, const struct iovec *iovec, > unsigned long vlen, int advice, unsigned int flags); size_t for vlen provides a clearer hint regarding the type of special treatment needed for ILP32 here (zero extension, not changing the type

[PATCH v9 3/3] mm/madvise: introduce process_madvise() syscall: an external memory hinting API

2020-08-31 Thread Minchan Kim
There is usecase that System Management Software(SMS) want to give a memory hint like MADV_[COLD|PAGEEOUT] to other processes and in the case of Android, it is the ActivityManagerService. The information required to make the reclaim decision is not known to the app. Instead, it is known to the