Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-15 Thread Mike Rapoport
On Wed, Nov 14, 2018 at 05:55:12PM -0800, Mike Rapoport wrote: > On Wed, Nov 14, 2018 at 04:50:23PM -0800, Alexander Duyck wrote: > > > > > > On 11/14/2018 7:07 AM, Michal Hocko wrote: > > >On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > > >>This patchset is essentially a refactor of the page

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-15 Thread Michal Hocko
On Thu 15-11-18 08:02:33, Alexander Duyck wrote: > On 11/15/2018 12:10 AM, Michal Hocko wrote: > > On Wed 14-11-18 16:50:23, Alexander Duyck wrote: [...] > > > I plan to remove it, but don't think I can get to it in this patch set. > > > > What I am trying to argue is that we should simply drop th

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-15 Thread Alexander Duyck
On 11/15/2018 12:10 AM, Michal Hocko wrote: On Wed 14-11-18 16:50:23, Alexander Duyck wrote: On 11/14/2018 7:07 AM, Michal Hocko wrote: On Mon 05-11-18 13:19:25, Alexander Duyck wrote: This patchset is essentially a refactor of the page initialization logic that is meant to provide for bette

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-15 Thread Michal Hocko
On Wed 14-11-18 16:50:23, Alexander Duyck wrote: > > > On 11/14/2018 7:07 AM, Michal Hocko wrote: > > On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > > > This patchset is essentially a refactor of the page initialization logic > > > that is meant to provide for better code reuse while providin

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-14 Thread Mike Rapoport
On Wed, Nov 14, 2018 at 04:50:23PM -0800, Alexander Duyck wrote: > > > On 11/14/2018 7:07 AM, Michal Hocko wrote: > >On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > >>This patchset is essentially a refactor of the page initialization logic > >>that is meant to provide for better code reuse whi

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-14 Thread Alexander Duyck
On 11/14/2018 7:07 AM, Michal Hocko wrote: On Mon 05-11-18 13:19:25, Alexander Duyck wrote: This patchset is essentially a refactor of the page initialization logic that is meant to provide for better code reuse while providing a significant improvement in deferred page initialization perform

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-14 Thread Michal Hocko
On Wed 14-11-18 19:12:53, Pavel Tatashin wrote: > On 18-11-14 16:07:42, Michal Hocko wrote: > > On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > > > This patchset is essentially a refactor of the page initialization logic > > > that is meant to provide for better code reuse while providing a > >

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-14 Thread Pavel Tatashin
On 18-11-14 16:07:42, Michal Hocko wrote: > On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > > This patchset is essentially a refactor of the page initialization logic > > that is meant to provide for better code reuse while providing a > > significant improvement in deferred page initialization

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-14 Thread Michal Hocko
On Mon 05-11-18 13:19:25, Alexander Duyck wrote: > This patchset is essentially a refactor of the page initialization logic > that is meant to provide for better code reuse while providing a > significant improvement in deferred page initialization performance. > > In my testing on an x86_64 syste

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-12 Thread Pavel Tatashin
On 18-11-12 11:10:35, Alexander Duyck wrote: > > The point I was trying to make is that it doesn't. You say it is an > order of magnitude better but it is essentially 3.5x vs 3.8x and to > achieve the 3.8x you are using a ton of system resources. My approach > is meant to do more with less, while

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-12 Thread Alexander Duyck
On Fri, Nov 9, 2018 at 5:17 PM Pavel Tatashin wrote: > > On 18-11-09 16:46:02, Alexander Duyck wrote: > > On Fri, 2018-11-09 at 19:00 -0500, Pavel Tatashin wrote: > > > On 18-11-09 15:14:35, Alexander Duyck wrote: > > > > On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > > > > > On 18-11-

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-12 Thread Daniel Jordan
On Fri, Nov 09, 2018 at 07:00:06PM -0500, Pavel Tatashin wrote: > On 18-11-09 15:14:35, Alexander Duyck wrote: > > On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > > > On 18-11-05 13:19:25, Alexander Duyck wrote: > > > > This patchset is essentially a refactor of the page initialization l

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-09 Thread Pavel Tatashin
On 18-11-09 16:46:02, Alexander Duyck wrote: > On Fri, 2018-11-09 at 19:00 -0500, Pavel Tatashin wrote: > > On 18-11-09 15:14:35, Alexander Duyck wrote: > > > On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > > > > On 18-11-05 13:19:25, Alexander Duyck wrote: > > > > > This patchset is ess

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-09 Thread Alexander Duyck
On Fri, 2018-11-09 at 19:00 -0500, Pavel Tatashin wrote: > On 18-11-09 15:14:35, Alexander Duyck wrote: > > On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > > > On 18-11-05 13:19:25, Alexander Duyck wrote: > > > > This patchset is essentially a refactor of the page initialization logic >

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-09 Thread Pavel Tatashin
On 18-11-09 15:14:35, Alexander Duyck wrote: > On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > > On 18-11-05 13:19:25, Alexander Duyck wrote: > > > This patchset is essentially a refactor of the page initialization logic > > > that is meant to provide for better code reuse while providin

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-09 Thread Alexander Duyck
On Fri, 2018-11-09 at 16:15 -0500, Pavel Tatashin wrote: > On 18-11-05 13:19:25, Alexander Duyck wrote: > > This patchset is essentially a refactor of the page initialization logic > > that is meant to provide for better code reuse while providing a > > significant improvement in deferred page init

Re: [mm PATCH v5 0/7] Deferred page init improvements

2018-11-09 Thread Pavel Tatashin
On 18-11-05 13:19:25, Alexander Duyck wrote: > This patchset is essentially a refactor of the page initialization logic > that is meant to provide for better code reuse while providing a > significant improvement in deferred page initialization performance. > > In my testing on an x86_64 system wi

[mm PATCH v5 0/7] Deferred page init improvements

2018-11-05 Thread Alexander Duyck
This patchset is essentially a refactor of the page initialization logic that is meant to provide for better code reuse while providing a significant improvement in deferred page initialization performance. In my testing on an x86_64 system with 384GB of RAM and 3TB of persistent memory per node I