From owner-svn-src-head@FreeBSD.ORG  Tue May 21 17:31:52 2013
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from hammer.pct.niksun.com (freefall.freebsd.org
 [IPv6:2001:1900:2254:206c::16:87])
 by hub.freebsd.org (Postfix) with ESMTP id 3A793362;
 Tue, 21 May 2013 17:31:52 +0000 (UTC)
 (envelope-from jkim@FreeBSD.org)
Message-ID: <519BAEB6.9040903@FreeBSD.org>
Date: Tue, 21 May 2013 13:28:22 -0400
From: Jung-uk Kim <jkim@FreeBSD.org>
User-Agent: Mozilla/5.0 (X11; FreeBSD amd64;
 rv:17.0) Gecko/20130517 Thunderbird/17.0.6
MIME-Version: 1.0
To: Bruce Evans <brde@optusnet.com.au>
Subject: Re: svn commit: r247086 - head/sys/x86/isa
References: <201302210638.r1L6cOVx006678@svn.freebsd.org>
 <20130221064912.GA20360@FreeBSD.org> <20130520022100.GA82181@FreeBSD.org>
 <73F6A8D9-3366-47AB-9DE9-E570006E3A3F@bsdimp.com>
 <20130521061051.GA51568@FreeBSD.org> <20130521172744.U1667@besplex.bde.org>
In-Reply-To: <20130521172744.U1667@besplex.bde.org>
X-Enigmail-Version: 1.5.1
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Cc: Alexey Dokuchaev <danfe@freebsd.org>, src-committers@freebsd.org,
 svn-src-all@freebsd.org, svn-src-head@freebsd.org,
 Warner Losh <imp@freebsd.org>, Warner Losh <imp@bsdimp.com>
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 21 May 2013 17:31:52 -0000

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 2013-05-21 05:18:46 -0400, Bruce Evans wrote:
...
> So it seems that pmtimer has a lot of dead code and reduces to not 
> much more than an inittodr(0) and usually acts like the
> inittodr(!0) in the acpi amd64 resume code.

When I wrote the resume code for amd64, I studied the pmtimer and
realized the same problems, of course.  So, I simply reduced the code
and embedded it in acpi.c (without fixing underlying problems) because
a) I didn't want the dead code in pmtimer, b) acpi is mandatory for
amd64, and c) I didn't want to disturb i386 (or ia64) code paths.

In fact, my evil plan was a) deprecate apm and pmtimer for i386, b)
improve the code in acpi.c, and c) fix "calcru: runtime went
backwards" problem for good.  But I never found free time to do that. :-(

> acpi amd64 resume also seems to be missing the verbose logging of
> the suspension time, but maybe acpi utilities give that.

/etc/rc.{suspend,resume} already do that.  So, yes, the "feature" is
redundant.

Jung-uk Kim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.20 (FreeBSD)

iQEcBAEBAgAGBQJRm662AAoJECXpabHZMqHO+fgH/iEGdzAXeZuG6F4+fGBk9wCi
nEyz/RoJeQ7OwdR8vVIcs3z9TwZT58QVOipt5Tgh6cr77fRWfTrBhdZEWDSInXcb
wpnFXeXNe0MVNHduuq3QIpmpHxJLHUMLKTdaBJhaVixATcQovYhMZUljq0AxVKDs
VJw346YrdXI3IaAzXbm1OonGNgV48N2ACXCJvOKv1qwtYhx9UMIUkbXoWxGkT6rb
i5hWj3VsVPBG/E4vvKPvZAwmieU7TaUY8/BOjyyWh4kAsRW5N4ZmRY9W0v5EyRid
5qDEfUrW6liVFh+4O4HOL3G0NDY6KCQpoBue91Dp5cOaHl3lzGlBUGQ9/9932nM=
=MldY
-----END PGP SIGNATURE-----