Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S264919AbUAFTFn (ORCPT ); Tue, 6 Jan 2004 14:05:43 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S264930AbUAFTFm (ORCPT ); Tue, 6 Jan 2004 14:05:42 -0500 Received: from 195-23-16-24.nr.ip.pt ([195.23.16.24]:8357 "EHLO bipbip.comserver-pie.com") by vger.kernel.org with ESMTP id S264919AbUAFTFl (ORCPT ); Tue, 6 Jan 2004 14:05:41 -0500 Message-ID: <3FFB06AF.6000503@grupopie.com> Date: Tue, 06 Jan 2004 19:04:15 +0000 From: Paulo Marques Organization: GrupoPIE User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.4.1) Gecko/20020508 Netscape6/6.2.3 X-Accept-Language: en-us MIME-Version: 1.0 To: Linus Torvalds Cc: James Bottomley , Andrew Morton , johnstultz@us.ibm.com, Linux Kernel Subject: Re: [PATCH] fix get_jiffies_64 to work on voyager References: <1073405053.2047.28.camel@mulgrave> <3FFAFE7A.7030404@grupopie.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 986 Lines: 33 Linus Torvalds wrote: > > On Tue, 6 Jan 2004, Paulo Marques wrote: > >> .... > > Yes, we used to do things like that at some point (and your code is > buggy: by leaving out the size, the "volatile" cast casts to the implicit > "int" size in C). > Ugh, stupid mistake. Just wrote the code to fast... :( Anyway, thanks for your clarification. People like me that code a lot of user-space stuff, take a while to realize all the implications of SMP, ordering, preemption, etc., etc. By the way, after double checking your logic, it looks good as long as there are no interrupts that take more than 3 jiffies to complete :) -- Paulo Marques - www.grupopie.com "In a world without walls and fences who needs windows and gates?" - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/