From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030245AbWGYXO2 (ORCPT ); Tue, 25 Jul 2006 19:14:28 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1030246AbWGYXO1 (ORCPT ); Tue, 25 Jul 2006 19:14:27 -0400 Received: from ra.tuxdriver.com ([70.61.120.52]:24076 "EHLO ra.tuxdriver.com") by vger.kernel.org with ESMTP id S1030245AbWGYXO1 (ORCPT ); Tue, 25 Jul 2006 19:14:27 -0400 Date: Tue, 25 Jul 2006 19:10:43 -0400 From: Neil Horman To: "H. Peter Anvin" Cc: Dave Airlie , Segher Boessenkool , linux-kernel@vger.kernel.org, a.zummo@towertech.it, jg@freedesktop.org Subject: Re: [PATCH] RTC: Add mmap method to rtc character driver Message-ID: <20060725231043.GA4661@localhost.localdomain> References: <44C66C91.8090700@zytor.com> <20060725192138.GI4608@hmsreliant.homelinux.net> <20060725194733.GJ4608@hmsreliant.homelinux.net> <21d7e9970607251304n5681bf44gc751c21fd79be99d@mail.gmail.com> <44C67E1A.7050105@zytor.com> <20060725204736.GK4608@hmsreliant.homelinux.net> <44C6842C.8020501@zytor.com> <20060725222547.GA3973@localhost.localdomain> <44C69C2E.7000609@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <44C69C2E.7000609@zytor.com> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 25, 2006 at 03:33:18PM -0700, H. Peter Anvin wrote: > Neil Horman wrote: > >> > >Yes, but if its in trade for something thats being used currently which > >hurts > >more (case in point being the X server), using this solution is a net gain. > > > >I'm not arguing with you that adding a low res gettimeofday vsyscall is a > >better > >long term solution, but doing that requires potentially several > >implementations > >in the C library accross a range of architectures, some of which may not > >be able > >to provide a time solution any better than what the gettimeofday syscall > >provides today. The /dev/rtc solution is easy, available right now, and > >applies > >to all arches. It has zero impact for systems which do not use it, and for > >those applications which make a decision to use it instead of an alternate > >method, the result I expect will be a net gain, until such time as we code > >up, > >test and roll out a vsyscall solution > > > > Quick hacks are frowned upon in the Linux universe. The kernel-user > space interface is supposed to be stable, and thus a hack like this has > to be maintained indefinitely. > > Putting temporary hacks like this in is not a good idea. > Only if you make the mental leap that this is a hack; its not. Its a new feature for a driver. mmap on device drivers is a well known and understood interface. There is nothing hackish about it. And there is no need for it to be temporary either. Why shouldn't the rtc driver be able to export a monotonic counter via the mmap interface? mmtimer does it already, as do many other drivers. Theres nothing unstable about this interface, and it need not be short lived. It can live in perpituity, and applications can choose to use it, or migrate away from it should something else more efficient become available (a gettimeofday vsyscall). More importantly, it can continue to be used in those situations where a vsyscall is not feasable, or simply maps to the nominal slow path kernel trap that one would find to heavy-weight to use in comparison to an mmaped page. Neil > -hpa