Re: [PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-28 Thread Konstantin Khorenko
On 12/27/2018 07:50 PM, Michal Hocko wrote: > On Thu 27-12-18 16:05:18, Konstantin Khorenko wrote: >> On 12/26/2018 11:40 AM, Michal Hocko wrote: >>> Appart from general comments as a reply to the cover (btw. this all >>> should be in the changelog because this is the _why_ part of the >>>

Re: [PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-27 Thread Michal Hocko
On Thu 27-12-18 16:05:18, Konstantin Khorenko wrote: > On 12/26/2018 11:40 AM, Michal Hocko wrote: > > Appart from general comments as a reply to the cover (btw. this all > > should be in the changelog because this is the _why_ part of the > > justification which should be _always_ part of the

Re: [PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-27 Thread Konstantin Khorenko
On 12/26/2018 11:40 AM, Michal Hocko wrote: > Appart from general comments as a reply to the cover (btw. this all > should be in the changelog because this is the _why_ part of the > justification which should be _always_ part of the changelog). Thank you, will add in the next version of the

Re: [PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-26 Thread Michal Hocko
On Wed 26-12-18 09:40:51, Michal Hocko wrote: > Appart from general comments as a reply to the cover (btw. this all > should be in the changelog because this is the _why_ part of the > justification which should be _always_ part of the changelog). > > On Tue 25-12-18 18:39:27, Konstantin Khorenko

Re: [PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-26 Thread Michal Hocko
Appart from general comments as a reply to the cover (btw. this all should be in the changelog because this is the _why_ part of the justification which should be _always_ part of the changelog). On Tue 25-12-18 18:39:27, Konstantin Khorenko wrote: [...] > +config WARN_HIGH_ORDER > + bool

[PATCH 1/1] mm/page_alloc: add a warning about high order allocations

2018-12-25 Thread Konstantin Khorenko
Adds sysctl "vm.warn_high_order". If set it will warn about allocations with order >= vm.warn_high_order. Prints only 32 warnings at most and skips all __GFP_NOWARN allocations. The code is under config option, disabled by default. If enabled, default vm.warn_high_order is 3