From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: yield() in i2c non-happy paths hits BUG under -rt patch Date: Thu, 19 Nov 2009 14:18:40 +0100 Message-ID: <1258636720.4372.350.camel@twins> References: <20091107210147.3e754278@hyperion.delvare> <4AF7148C.9090706@thebigcorporation.com> <20091112211255.09cd884a@hyperion.delvare> <20091116155606.GC29479@sirena.org.uk> <20091118010520.4cd397d4@lxorguk.ukuu.org.uk> <20091118175202.490989d8@hyperion.delvare> <20091119130526.23a69b85@hyperion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Cc: Thomas Gleixner , Leon Woestenberg , Alan Cox , Mark Brown , Sven-Thorsten Dietrich , linux-i2c@vger.kernel.org, rt-users , "Ben Dooks (embedded platforms)" , LKML To: Jean Delvare Return-path: Received: from bombadil.infradead.org ([18.85.46.34]:55957 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752697AbZKSNRQ convert rfc822-to-8bit (ORCPT ); Thu, 19 Nov 2009 08:17:16 -0500 In-Reply-To: <20091119130526.23a69b85@hyperion.delvare> Sender: linux-rt-users-owner@vger.kernel.org List-ID: On Thu, 2009-11-19 at 13:05 +0100, Jean Delvare wrote: > > Was yield() turned into NOOP by design, or was it a bug? Design. yield() for SCHED_OTHER is not specified, so everything goes. There's two possible 'sane' options for yield for (CFS's) SCHED_OTHER: - place the task behind all other tasks of the same nice level This is however an O(n) operation for CFS since we don't separate things out based on nice level, hence we don't do that. - service the task that is most starved of service That fits nicely into the fairness thing, and is what we default to. The thing is, that's current in 99% of the cases, otherwise we would already be running another task. So its not strictly a NOP, but in practice it is. There is also another option, place the task behind _all_ other tasks, but that also surprises people and causes regressions, because they don't expect yield() to wait _that_ long. And all this is irrespective of the question of whether yield() is ever a sane thing to do (I say not).