linux-kernel - Re: [PATCH 12/19] mm: migrate: Introduce migrate_misplaced

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 13 Nov 2012 09:49:09 -0500
From:	Rik van Riel <riel@...hat.com>
To:	Ingo Molnar <mingo@...nel.org>
CC:	Mel Gorman <mgorman@...e.de>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Andrea Arcangeli <aarcange@...hat.com>,
	Johannes Weiner <hannes@...xchg.org>,
	Hugh Dickins <hughd@...gle.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux-MM <linux-mm@...ck.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 12/19] mm: migrate: Introduce migrate_misplaced_page()

On 11/13/2012 06:43 AM, Ingo Molnar wrote:
>
> * Ingo Molnar <mingo@...nel.org> wrote:
>
>>
>> * Mel Gorman <mgorman@...e.de> wrote:
>>
>>> From: Peter Zijlstra <a.p.zijlstra@...llo.nl>
>>>
>>> Note: This was originally based on Peter's patch "mm/migrate: Introduce
>>> 	migrate_misplaced_page()" but borrows extremely heavily from Andrea's
>>> 	"autonuma: memory follows CPU algorithm and task/mm_autonuma stats
>>> 	collection". The end result is barely recognisable so signed-offs
>>> 	had to be dropped. If original authors are ok with it, I'll
>>> 	re-add the signed-off-bys.
>>>
>>> Add migrate_misplaced_page() which deals with migrating pages from
>>> faults.
>>>
>>> Based-on-work-by: Lee Schermerhorn <Lee.Schermerhorn@...com>
>>> Based-on-work-by: Peter Zijlstra <a.p.zijlstra@...llo.nl>
>>> Based-on-work-by: Andrea Arcangeli <aarcange@...hat.com>
>>> Signed-off-by: Mel Gorman <mgorman@...e.de>
>>> ---
>>>   include/linux/migrate.h |    8 ++++
>>>   mm/migrate.c            |  104 ++++++++++++++++++++++++++++++++++++++++++++++-
>>>   2 files changed, 110 insertions(+), 2 deletions(-)
>>
>> That's a nice patch - the TASK_NUMA_FAULT approach in the
>> original patch was not very elegant.
>>
>> I've started testing it to see how well your version works.
>
> Hm, I'm seeing some instability - see the boot crash below. If I
> undo your patch it goes away.
>
> ( To help debugging this I've attached migration.patch which
>    applies your patch on top of Peter's latest queue of patches.
>    If I revert this patch then the crash goes away. )
>
> I've gone back to the well-tested page migration code from Peter
> for the time being.

Is there a place we can see your code?

Peter's patch with MIGRATE_FAULT is very much NAKed, so
this approach does need to be made to work...

You can either make the working tree public somewhere,
so we can help, or figure it out yourself. Your choice :)

-- 
All rights reversed
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/