From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261730AbVDODZ2 (ORCPT ); Thu, 14 Apr 2005 23:25:28 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261731AbVDODZ2 (ORCPT ); Thu, 14 Apr 2005 23:25:28 -0400 Received: from smtp203.mail.sc5.yahoo.com ([216.136.129.93]:59261 "HELO smtp203.mail.sc5.yahoo.com") by vger.kernel.org with SMTP id S261730AbVDODZY (ORCPT ); Thu, 14 Apr 2005 23:25:24 -0400 Subject: Re: [PATCH] sched: fix never executed code due to expression always false From: Nick Piggin To: Herbert Xu Cc: Jesper Juhl , Ingo Molnar , rml@tech9.net, torvalds@osdl.org, lkml In-Reply-To: References: Content-Type: text/plain Date: Fri, 15 Apr 2005 13:25:20 +1000 Message-Id: <1113535520.6517.18.camel@npiggin-nld.site> Mime-Version: 1.0 X-Mailer: Evolution 2.0.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2005-04-15 at 12:59 +1000, Herbert Xu wrote: > Jesper Juhl wrote: > > > > - if (unlikely((long long)now - prev->timestamp < 0)) > > + if (unlikely(((long long)now - (long long)prev->timestamp) < 0)) > > You can write this as > > (long long)(now - prev->timestamp) > True. Combined that with Matt's suggestion, and we probably have the cleanest solution. Thanks.