On Wed, 6 Jul 2016, Joonsoo Kim wrote:
> > diff --git a/mm/compaction.c b/mm/compaction.c
> > --- a/mm/compaction.c
> > +++ b/mm/compaction.c
> > @@ -1009,8 +1009,6 @@ static void isolate_freepages(struct compact_control
> > *cc)
> > block_end_pfn = block_start_pfn,
>
On Wed, 6 Jul 2016, Joonsoo Kim wrote:
> > diff --git a/mm/compaction.c b/mm/compaction.c
> > --- a/mm/compaction.c
> > +++ b/mm/compaction.c
> > @@ -1009,8 +1009,6 @@ static void isolate_freepages(struct compact_control
> > *cc)
> > block_end_pfn = block_start_pfn,
>
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
On Wed, Jul 06, 2016 at 10:41:09AM +0900, Joonsoo Kim wrote:
< snip >
> > Do you have any objection to this fix for 4.7?
> >
> > Joonson and/or Minchan, does this address the issue that you reported?
>
> Unfortunately, I have no test case to trigger it. But, I think that
> this patch will
On Wed, Jul 06, 2016 at 10:41:09AM +0900, Joonsoo Kim wrote:
< snip >
> > Do you have any objection to this fix for 4.7?
> >
> > Joonson and/or Minchan, does this address the issue that you reported?
>
> Unfortunately, I have no test case to trigger it. But, I think that
> this patch will
On Tue, Jul 05, 2016 at 02:01:29PM -0700, David Rientjes wrote:
> On Thu, 30 Jun 2016, Vlastimil Babka wrote:
>
> > > Note: I really dislike the low watermark check in split_free_page() and
> > > consider it poor software engineering. The function should split a free
> > > page, nothing more.
On Tue, Jul 05, 2016 at 02:01:29PM -0700, David Rientjes wrote:
> On Thu, 30 Jun 2016, Vlastimil Babka wrote:
>
> > > Note: I really dislike the low watermark check in split_free_page() and
> > > consider it poor software engineering. The function should split a free
> > > page, nothing more.
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
On 07/05/2016 11:01 PM, David Rientjes wrote:
> On Thu, 30 Jun 2016, Vlastimil Babka wrote:
>
>>> Note: I really dislike the low watermark check in split_free_page() and
>>> consider it poor software engineering. The function should split a free
>>> page, nothing more. Terminating memory
On 07/05/2016 11:01 PM, David Rientjes wrote:
> On Thu, 30 Jun 2016, Vlastimil Babka wrote:
>
>>> Note: I really dislike the low watermark check in split_free_page() and
>>> consider it poor software engineering. The function should split a free
>>> page, nothing more. Terminating memory
On Thu, 30 Jun 2016, Vlastimil Babka wrote:
> > Note: I really dislike the low watermark check in split_free_page() and
> > consider it poor software engineering. The function should split a free
> > page, nothing more. Terminating memory compaction because of a low
> > watermark check when
On Thu, 30 Jun 2016, Vlastimil Babka wrote:
> > Note: I really dislike the low watermark check in split_free_page() and
> > consider it poor software engineering. The function should split a free
> > page, nothing more. Terminating memory compaction because of a low
> > watermark check when
On 06/29/2016 11:47 PM, David Rientjes wrote:
It's possible to isolate some freepages in a pageblock and then fail
split_free_page() due to the low watermark check. In this case, we hit
VM_BUG_ON() because the freeing scanner terminated early without a
contended lock or enough freepages.
This
On 06/29/2016 11:47 PM, David Rientjes wrote:
It's possible to isolate some freepages in a pageblock and then fail
split_free_page() due to the low watermark check. In this case, we hit
VM_BUG_ON() because the freeing scanner terminated early without a
contended lock or enough freepages.
This
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
On Wed, Jun 29, 2016 at 02:47:20PM -0700, David Rientjes wrote:
> It's possible to isolate some freepages in a pageblock and then fail
> split_free_page() due to the low watermark check. In this case, we hit
> VM_BUG_ON() because the freeing scanner terminated early without a
> contended lock
It's possible to isolate some freepages in a pageblock and then fail
split_free_page() due to the low watermark check. In this case, we hit
VM_BUG_ON() because the freeing scanner terminated early without a
contended lock or enough freepages.
This should never have been a VM_BUG_ON() since
It's possible to isolate some freepages in a pageblock and then fail
split_free_page() due to the low watermark check. In this case, we hit
VM_BUG_ON() because the freeing scanner terminated early without a
contended lock or enough freepages.
This should never have been a VM_BUG_ON() since
20 matches
Mail list logo