Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Alexander Duyck
On Thu, 2019-09-12 at 17:35 +0100, Mel Gorman wrote: > On Thu, Sep 12, 2019 at 11:19:25AM +0200, Michal Hocko wrote: > > On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > > > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > > >

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Mel Gorman
On Thu, Sep 12, 2019 at 11:19:25AM +0200, Michal Hocko wrote: > On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > > [...] > > > > We don't put any limitations on the allocator

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Alexander Duyck
On Thu, 2019-09-12 at 11:19 +0200, Michal Hocko wrote: > On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > > [...] > > > > We don't put any limitations on the allocator other then

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Nitesh Narayan Lal
On 9/12/19 3:47 AM, David Hildenbrand wrote: > On 12.09.19 09:16, Michal Hocko wrote: >> On Wed 11-09-19 18:09:18, David Hildenbrand wrote: >>> On 11.09.19 15:51, Michal Hocko wrote: On Wed 11-09-19 15:20:02, Michal Hocko wrote: [...] >> 4. Continuously report, not the "one time

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Michal Hocko
On Thu 12-09-19 13:24:25, Kirill A. Shutemov wrote: > On Thu, Sep 12, 2019 at 11:19:25AM +0200, Michal Hocko wrote: > > On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > > > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > > > > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > >

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Kirill A. Shutemov
On Thu, Sep 12, 2019 at 11:19:25AM +0200, Michal Hocko wrote: > On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > > [...] > > > > We don't put any limitations on the allocator

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Michal Hocko
On Thu 12-09-19 09:47:30, David Hildenbrand wrote: > On 12.09.19 09:16, Michal Hocko wrote: > > On Wed 11-09-19 18:09:18, David Hildenbrand wrote: > >> On 11.09.19 15:51, Michal Hocko wrote: > >>> On Wed 11-09-19 15:20:02, Michal Hocko wrote: > >>> [...] > > 4. Continuously report, not the

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Michal Hocko
On Wed 11-09-19 08:12:03, Alexander Duyck wrote: > On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > [...] > > > We don't put any limitations on the allocator other then that it needs to > > > clean up the metadata on allocation, and

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread David Hildenbrand
On 12.09.19 09:16, Michal Hocko wrote: > On Wed 11-09-19 18:09:18, David Hildenbrand wrote: >> On 11.09.19 15:51, Michal Hocko wrote: >>> On Wed 11-09-19 15:20:02, Michal Hocko wrote: >>> [...] > 4. Continuously report, not the "one time report everything" approach. So you mean the

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-12 Thread Michal Hocko
On Wed 11-09-19 18:09:18, David Hildenbrand wrote: > On 11.09.19 15:51, Michal Hocko wrote: > > On Wed 11-09-19 15:20:02, Michal Hocko wrote: > > [...] > >>> 4. Continuously report, not the "one time report everything" approach. > >> > >> So you mean the allocator reporting this rather than an

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
On 11.09.19 15:51, Michal Hocko wrote: > On Wed 11-09-19 15:20:02, Michal Hocko wrote: > [...] >>> 4. Continuously report, not the "one time report everything" approach. >> >> So you mean the allocator reporting this rather than an external code to >> poll right? I do not know, how much this is

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
>> Something slightly similar is being performed by Nitesh's patch set. On >> every free of a certain granularity, he records it in the bitmap. These >> bits are "hints of free pages". >> >> A thread then walks over the bitmap and tries to allocate the "hints". >> If the pages were already reused,

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Alexander Duyck
On Wed, Sep 11, 2019 at 4:36 AM Michal Hocko wrote: > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > [...] > > We don't put any limitations on the allocator other then that it needs to > > clean up the metadata on allocation, and that it cannot allocate a page > > that is in the process of

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Nitesh Narayan Lal
On 9/11/19 9:20 AM, Michal Hocko wrote: > On Wed 11-09-19 15:03:39, David Hildenbrand wrote: >> On 11.09.19 14:54, Michal Hocko wrote: >>> On Wed 11-09-19 14:42:41, David Hildenbrand wrote: On 11.09.19 14:25, Michal Hocko wrote: > On Wed 11-09-19 14:19:41, Michal Hocko wrote: >> On

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Wed 11-09-19 15:20:02, Michal Hocko wrote: [...] > > 4. Continuously report, not the "one time report everything" approach. > > So you mean the allocator reporting this rather than an external code to > poll right? I do not know, how much this is nice to have than must have? Another idea that

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Nitesh Narayan Lal
On 9/11/19 8:54 AM, Michal Hocko wrote: > On Wed 11-09-19 14:42:41, David Hildenbrand wrote: >> On 11.09.19 14:25, Michal Hocko wrote: >>> On Wed 11-09-19 14:19:41, Michal Hocko wrote: On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: > On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Wed 11-09-19 15:03:39, David Hildenbrand wrote: > On 11.09.19 14:54, Michal Hocko wrote: > > On Wed 11-09-19 14:42:41, David Hildenbrand wrote: > >> On 11.09.19 14:25, Michal Hocko wrote: > >>> On Wed 11-09-19 14:19:41, Michal Hocko wrote: > On Wed 11-09-19 08:08:38, Michael S. Tsirkin

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
On 11.09.19 14:54, Michal Hocko wrote: > On Wed 11-09-19 14:42:41, David Hildenbrand wrote: >> On 11.09.19 14:25, Michal Hocko wrote: >>> On Wed 11-09-19 14:19:41, Michal Hocko wrote: On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: > On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Nitesh Narayan Lal
On 9/11/19 8:42 AM, David Hildenbrand wrote: > On 11.09.19 14:25, Michal Hocko wrote: >> On Wed 11-09-19 14:19:41, Michal Hocko wrote: >>> On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: > On Tue 10-09-19 14:23:40,

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Wed 11-09-19 14:42:41, David Hildenbrand wrote: > On 11.09.19 14:25, Michal Hocko wrote: > > On Wed 11-09-19 14:19:41, Michal Hocko wrote: > >> On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: > >>> On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: > On Tue 10-09-19

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
On 11.09.19 14:25, Michal Hocko wrote: > On Wed 11-09-19 14:19:41, Michal Hocko wrote: >> On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: >>> On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: On Tue 10-09-19 14:23:40, Alexander Duyck wrote: [...] > We don't put any

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Wed 11-09-19 14:19:41, Michal Hocko wrote: > On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: > > On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: > > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > > [...] > > > > We don't put any limitations on the allocator other then

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Wed 11-09-19 08:08:38, Michael S. Tsirkin wrote: > On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: > > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > > [...] > > > We don't put any limitations on the allocator other then that it needs to > > > clean up the metadata on

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michael S. Tsirkin
On Wed, Sep 11, 2019 at 01:36:19PM +0200, Michal Hocko wrote: > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > [...] > > We don't put any limitations on the allocator other then that it needs to > > clean up the metadata on allocation, and that it cannot allocate a page > > that is in the

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
On 11.09.19 13:36, Michal Hocko wrote: > On Tue 10-09-19 14:23:40, Alexander Duyck wrote: > [...] >> We don't put any limitations on the allocator other then that it needs to >> clean up the metadata on allocation, and that it cannot allocate a page >> that is in the process of being reported

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michal Hocko
On Tue 10-09-19 14:23:40, Alexander Duyck wrote: [...] > We don't put any limitations on the allocator other then that it needs to > clean up the metadata on allocation, and that it cannot allocate a page > that is in the process of being reported since we pulled it from the > free_list. If the

Re: [virtio-dev] Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread David Hildenbrand
On 11.09.19 11:23, Michael S. Tsirkin wrote: > On Tue, Sep 10, 2019 at 06:22:37PM +0200, David Hildenbrand wrote: >> On 10.09.19 18:18, Dr. David Alan Gilbert wrote: >>> * Alexander Duyck (alexander.du...@gmail.com) wrote: On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > > On

Re: [virtio-dev] Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-11 Thread Michael S. Tsirkin
On Tue, Sep 10, 2019 at 06:22:37PM +0200, David Hildenbrand wrote: > On 10.09.19 18:18, Dr. David Alan Gilbert wrote: > > * Alexander Duyck (alexander.du...@gmail.com) wrote: > >> On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > >>> > >>> On Tue 10-09-19 07:42:43, Alexander Duyck wrote: >

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Alexander Duyck
On Tue, 2019-09-10 at 19:52 +0200, Michal Hocko wrote: > On Tue 10-09-19 09:05:43, Alexander Duyck wrote: > > On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > > > On Tue 10-09-19 07:42:43, Alexander Duyck wrote: > > > > On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > > > > I wanted

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Alexander Duyck
On Tue, 2019-09-10 at 20:00 +0200, Michal Hocko wrote: > On Tue 10-09-19 19:52:13, Michal Hocko wrote: > > On Tue 10-09-19 09:05:43, Alexander Duyck wrote: > [...] > > > All this is providing is just a report and it is optional if the > > > hypervisor will act on it or not. If the hypervisor takes

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Michal Hocko
On Tue 10-09-19 19:52:13, Michal Hocko wrote: > On Tue 10-09-19 09:05:43, Alexander Duyck wrote: [...] > > All this is providing is just a report and it is optional if the > > hypervisor will act on it or not. If the hypervisor takes some sort of > > action on the page, then the expectation is

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Michal Hocko
On Tue 10-09-19 09:05:43, Alexander Duyck wrote: > On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > > > > On Tue 10-09-19 07:42:43, Alexander Duyck wrote: > > > On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > > > > > > > I wanted to review "mm: Introduce Reported pages" just realize

Re: [virtio-dev] Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread David Hildenbrand
On 10.09.19 18:18, Dr. David Alan Gilbert wrote: > * Alexander Duyck (alexander.du...@gmail.com) wrote: >> On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: >>> >>> On Tue 10-09-19 07:42:43, Alexander Duyck wrote: On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > I wanted to

Re: [virtio-dev] Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Dr. David Alan Gilbert
* Alexander Duyck (alexander.du...@gmail.com) wrote: > On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > > > > On Tue 10-09-19 07:42:43, Alexander Duyck wrote: > > > On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > > > > > > > I wanted to review "mm: Introduce Reported pages" just

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Alexander Duyck
On Tue, Sep 10, 2019 at 7:47 AM Michal Hocko wrote: > > On Tue 10-09-19 07:42:43, Alexander Duyck wrote: > > On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > > > > > I wanted to review "mm: Introduce Reported pages" just realize that I > > > have no clue on what is going on so returned to

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Michal Hocko
On Tue 10-09-19 07:42:43, Alexander Duyck wrote: > On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > > > I wanted to review "mm: Introduce Reported pages" just realize that I > > have no clue on what is going on so returned to the cover and it didn't > > really help much. I am completely

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Alexander Duyck
On Tue, Sep 10, 2019 at 5:42 AM Michal Hocko wrote: > > I wanted to review "mm: Introduce Reported pages" just realize that I > have no clue on what is going on so returned to the cover and it didn't > really help much. I am completely unfamiliar with virtio so please bear > with me. > > On Sat

Re: [PATCH v9 0/8] stg mail -e --version=v9 \

2019-09-10 Thread Michal Hocko
I wanted to review "mm: Introduce Reported pages" just realize that I have no clue on what is going on so returned to the cover and it didn't really help much. I am completely unfamiliar with virtio so please bear with me. On Sat 07-09-19 10:25:03, Alexander Duyck wrote: [...] > This series