Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-25 Thread Michal Hocko
On Wed 24-10-18 01:39:18, Joe Perches wrote: > On Wed, 2018-10-24 at 10:23 +0200, Michal Hocko wrote: > > On Tue 23-10-18 23:26:16, Joe Perches wrote: > > > On Wed, 2018-10-24 at 08:15 +0200, Michal Hocko wrote: > > > > On Wed 24-10-18 10:47:52, Arun KS wrote: > > > > > On 2018-10-24 01:34, Kees

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-24 Thread Arun KS
On 2018-10-23 11:43, Julia Lawall wrote: On Tue, 23 Oct 2018, Michal Hocko wrote: [Trimmed CC list + Julia - there is indeed no need to CC everybody maintain a file you are updating for the change like this] On Tue 23-10-18 10:16:51, Arun Sudhilal wrote: > On Mon, Oct 22, 2018 at 11:41 PM

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-24 Thread Joe Perches
On Wed, 2018-10-24 at 10:23 +0200, Michal Hocko wrote: > On Tue 23-10-18 23:26:16, Joe Perches wrote: > > On Wed, 2018-10-24 at 08:15 +0200, Michal Hocko wrote: > > > On Wed 24-10-18 10:47:52, Arun KS wrote: > > > > On 2018-10-24 01:34, Kees Cook wrote: > > > [...] > > > > > Thank you -- I was

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-24 Thread Michal Hocko
On Tue 23-10-18 23:26:16, Joe Perches wrote: > On Wed, 2018-10-24 at 08:15 +0200, Michal Hocko wrote: > > On Wed 24-10-18 10:47:52, Arun KS wrote: > > > On 2018-10-24 01:34, Kees Cook wrote: > > [...] > > > > Thank you -- I was struggling to figure out the best way to reply to > > > > this. :) > >

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-24 Thread Joe Perches
On Wed, 2018-10-24 at 08:15 +0200, Michal Hocko wrote: > On Wed 24-10-18 10:47:52, Arun KS wrote: > > On 2018-10-24 01:34, Kees Cook wrote: > [...] > > > Thank you -- I was struggling to figure out the best way to reply to > > > this. :) > > I'm sorry for the trouble caused. Sent the email using,

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-24 Thread Michal Hocko
On Wed 24-10-18 10:47:52, Arun KS wrote: > On 2018-10-24 01:34, Kees Cook wrote: [...] > > Thank you -- I was struggling to figure out the best way to reply to > > this. :) > I'm sorry for the trouble caused. Sent the email using, > git send-email --to-cmd="scripts/get_maintainer.pl -i" >

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-23 Thread Arun KS
On 2018-10-24 01:34, Kees Cook wrote: On Mon, Oct 22, 2018 at 10:11 PM, Konstantin Khlebnikov wrote: On 23.10.2018 7:15, Joe Perches wrote:> On Mon, 2018-10-22 at 22:53 +0530, Arun KS wrote: Remove managed_page_count_lock spinlock and instead use atomic variables. Perhaps better to define

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-23 Thread Kees Cook
On Mon, Oct 22, 2018 at 10:11 PM, Konstantin Khlebnikov wrote: > On 23.10.2018 7:15, Joe Perches wrote:> On Mon, 2018-10-22 at 22:53 +0530, > Arun KS wrote: >>> Remove managed_page_count_lock spinlock and instead use atomic >>> variables. >> >> Perhaps better to define and use macros for the

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-23 Thread Arun Sudhilal
On Tue, Oct 23, 2018 at 12:11 PM Julia Lawall wrote: > > > > On Tue, 23 Oct 2018, Michal Hocko wrote: > > > [Trimmed CC list + Julia - there is indeed no need to CC everybody maintain > > a > > file you are updating for the change like this] > > > > On Tue 23-10-18 10:16:51, Arun Sudhilal wrote:

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-23 Thread Julia Lawall
On Tue, 23 Oct 2018, Michal Hocko wrote: > [Trimmed CC list + Julia - there is indeed no need to CC everybody maintain a > file you are updating for the change like this] > > On Tue 23-10-18 10:16:51, Arun Sudhilal wrote: > > On Mon, Oct 22, 2018 at 11:41 PM Michal Hocko wrote: > > > > > > On

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-22 Thread Michal Hocko
[Trimmed CC list + Julia - there is indeed no need to CC everybody maintain a file you are updating for the change like this] On Tue 23-10-18 10:16:51, Arun Sudhilal wrote: > On Mon, Oct 22, 2018 at 11:41 PM Michal Hocko wrote: > > > > On Mon 22-10-18 22:53:22, Arun KS wrote: > > > Remove

Re: [PATCH] mm: convert totalram_pages, totalhigh_pages and managed_pages to atomic.

2018-10-22 Thread Konstantin Khlebnikov
On 23.10.2018 7:15, Joe Perches wrote:> On Mon, 2018-10-22 at 22:53 +0530, Arun KS wrote: >> Remove managed_page_count_lock spinlock and instead use atomic >> variables. > > Perhaps better to define and use macros for the accesses > instead of specific uses of atomic_long_ > > Something like: >