From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Thu, 11 Nov 2004 17:10:48 -0600 From: Brent Casavant Reply-To: Brent Casavant Subject: Re: [PATCH] Use MPOL_INTERLEAVE for tmpfs files In-Reply-To: Message-ID: References: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Hugh Dickins Cc: "Martin J. Bligh" , Andi Kleen , "Adam J. Richter" , colpatch@us.ibm.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org List-ID: On Thu, 11 Nov 2004, Hugh Dickins wrote: > The first (against 2.6.10-rc1-mm5) being my reversion of NULL sbinfo > in shmem.c, to make it easier for others to add things into sbinfo > without having to worry about NULL cases. So that goes back to > allocating an sbinfo even for the internal mount: I've rounded up to > L1_CACHE_BYTES to avoid false sharing, but even so, please test it out > on your 512-way to make sure I haven't screwed up the scalability we > got before - thanks. If you find it okay, I'll send to akpm soonish. I won't be able to get a 512 run in until Monday, due to test machine availability. However runs at 32P and 64P indicate nothing disastrous. Results seem to be in line with the numbers we were getting when doing the NULL sbinfo work. So, thus far a preliminary "Looks good". > The second (against the first) being my take on your patch, with > mpol=interleave, and minor alterations which may irritate you so much > you'll revert them immediately! (mainly, using MPOL_INTERLEAVE and > MPOL_DEFAULT within shmem.c rather than defining separate flags). > Only slightly tested at this end. Seems to work just fine, and I rather like how this was made a bit more general. Thumbs up! Brent -- Brent Casavant If you had nothing to fear, bcasavan@sgi.com how then could you be brave? Silicon Graphics, Inc. -- Queen Dama, Source Wars -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: aart@kvack.org