From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail203.messagelabs.com (mail203.messagelabs.com [216.82.254.243]) by kanga.kvack.org (Postfix) with ESMTP id 4D3356B003D for ; Mon, 14 Dec 2009 08:03:21 -0500 (EST) Date: Mon, 14 Dec 2009 08:03:02 -0500 From: Christoph Hellwig Subject: Re: [PATCH 2/8] Mark sleep_on as deprecated Message-ID: <20091214130302.GA25797@infradead.org> References: <20091211164651.036f5340@annuminas.surriel.com> <20091214210823.BBAE.A69D9226@jp.fujitsu.com> <20091214212351.BBB4.A69D9226@jp.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091214212351.BBB4.A69D9226@jp.fujitsu.com> Sender: owner-linux-mm@kvack.org To: KOSAKI Motohiro Cc: Rik van Riel , lwoodman@redhat.com, akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, minchan.kim@gmail.com List-ID: On Mon, Dec 14, 2009 at 09:24:40PM +0900, KOSAKI Motohiro wrote: > > > sleep_on() function is SMP and/or kernel preemption unsafe. we shouldn't > use it on new code. And the best way to archive this is to remove the function. In Linus' current tree I find: - 5 instances of sleep_on(), all in old and obscure block drivers - 2 instances of sleep_on_timeout(), both in old and obscure drivers - 28 instances of interruptible_sleep_on_timeout(), mostly in obscure drivers with a high concentration in the old oss core which should be killed anyway. And unfortunately a few relatively recent additions like the SGI xpc driver or usbvision driver - tons of instances of interruptible_sleep_on all over the drivers code So I think you're be better off just killing the first two ASAP instead of just deprecating them. for the other two deprecating and removing some of the horrible drivers still using them might be best. -- 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: email@kvack.org