From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Wed, 15 Aug 2001 19:00:41 -0300 (BRST) From: Rik van Riel Subject: Re: 0-order allocation problem In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org Return-Path: To: Linus Torvalds Cc: Hugh Dickins , Marcelo Tosatti , linux-mm@kvack.org List-ID: On Wed, 15 Aug 2001, Linus Torvalds wrote: > Btw, the whole comment around the fs/buffer.c braindamage is telling: > > /* We're _really_ low on memory. Now we just > * wait for old buffer heads to become free due to > * finishing IO. Since this is an async request and > * the reserve list is empty, we're sure there are > * async buffer heads in use. > */ > run_task_queue(&tq_disk); > > current->policy |= SCHED_YIELD; > __set_current_state(TASK_RUNNING); > schedule(); > goto try_again; > > It used to be correct, say about a few years ago. IIRC this code was introduced less than two months ago due to a race condition in the old code, where the allocator just went to sleep waiting for things to improve. ;) It's good to see you've reversed your position that there would be nothing we could do in this situation. The patch looks good at first sight, lets hope there are no hidden locking issues in obscure situations... regards, Rik -- IA64: a worthy successor to i860. http://www.surriel.com/ http://distro.conectiva.com/ Send all your spam to aardvark@nl.linux.org (spam digging piggy) -- 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/