From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754175AbYIEHlF (ORCPT ); Fri, 5 Sep 2008 03:41:05 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751967AbYIEHkw (ORCPT ); Fri, 5 Sep 2008 03:40:52 -0400 Received: from viefep18-int.chello.at ([213.46.255.22]:31915 "EHLO viefep17-int.chello.at" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751514AbYIEHkv (ORCPT ); Fri, 5 Sep 2008 03:40:51 -0400 X-SourceIP: 213.46.9.244 Subject: Re: [PATCH] Return value from schedule() From: Peter Zijlstra To: Stephen Hemminger Cc: Matthew Wilcox , Arjan van de Ven , Ingo Molnar , Linus Torvalds , Andrew Morton , Ben Hutchings , Jesse Barnes , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-pci@vger.kernel.org, Thomas Gleixner In-Reply-To: <20080904120522.6d5fdf8d@extreme> References: <20080827204626.4b65862f@extreme> <20080828111323.GI7908@solarflare.com> <20080903155713.7fab2e19@extreme> <20080904160739.GK2772@parisc-linux.org> <20080904161424.GA23042@elte.hu> <20080904162111.GM2772@parisc-linux.org> <20080904103049.01f748d9@infradead.org> <20080904174845.GN2772@parisc-linux.org> <20080904120522.6d5fdf8d@extreme> Content-Type: text/plain Date: Fri, 05 Sep 2008 09:40:40 +0200 Message-Id: <1220600440.7080.2.camel@twins.programming.kicks-ass.net> Mime-Version: 1.0 X-Mailer: Evolution 2.22.2 (2.22.2-2.fc9) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2008-09-04 at 12:05 -0700, Stephen Hemminger wrote: > On Thu, 4 Sep 2008 11:48:45 -0600 > Matthew Wilcox wrote: > > > On Thu, Sep 04, 2008 at 10:30:49AM -0700, Arjan van de Ven wrote: > > > use schedule_hrtimerout() for this (hopefully will be in 2.6.28); > > > see this weeks LWN for an article describing it > > > > OK, so something like: > > > > struct timespec ts = { 0, 10 * 1000 }; > > > > set_task_state(TASK_INTERRUPTIBLE); > > schedule_hrtimeout(&ts, HRTIMER_MODE_REL); > > if (fatal_signal_pending()) > > return -EINTR; > > > > should do the trick. > > > > Never mind, I changed it to just yield() in revision. Gah, not another yield in the network code we have to figure out wtf its meant to do.