From tobias.rehbein@web.de  Sat May 29 10:28:28 2010
Return-Path: <tobias.rehbein@web.de>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 2C0961065674;
	Sat, 29 May 2010 10:28:28 +0000 (UTC)
	(envelope-from tobias.rehbein@web.de)
Received: from fmmailgate03.web.de (fmmailgate03.web.de [217.72.192.234])
	by mx1.freebsd.org (Postfix) with ESMTP id AFA208FC08;
	Sat, 29 May 2010 10:28:27 +0000 (UTC)
Received: from smtp01.web.de  ( [172.20.0.243])
	by fmmailgate03.web.de (Postfix) with ESMTP id 42EEB15377F65;
	Sat, 29 May 2010 12:28:26 +0200 (CEST)
Received: from [95.88.228.74] (helo=sushi.pseudo.local)
	by smtp01.web.de with asmtp (TLSv1:AES256-SHA:256)
	(WEB.DE 4.110 #4)
	id 1OIJHG-0004ov-00; Sat, 29 May 2010 12:28:26 +0200
Received: from sushi.pseudo.local (localhost [127.0.0.1])
	by sushi.pseudo.local (8.14.3/8.14.3) with ESMTP id o4TASPqr006912;
	Sat, 29 May 2010 12:28:25 +0200 (CEST)
	(envelope-from tobi@sushi.pseudo.local)
Received: (from tobi@localhost)
	by sushi.pseudo.local (8.14.3/8.14.3/Submit) id o4TASPmw006911;
	Sat, 29 May 2010 12:28:25 +0200 (CEST)
	(envelope-from tobi)
Message-Id: <201005291028.o4TASPmw006911@sushi.pseudo.local>
Date: Sat, 29 May 2010 12:28:25 +0200 (CEST)
From: Tobias Rehbein <tobias.rehbein@web.de>
Sender: tobias.rehbein@web.de
Reply-To: Tobias Rehbein <tobias.rehbein@web.de>
To: FreeBSD-gnats-submit@freebsd.org
Cc: mav@freebsd.org
Subject: Fix manpage of acpi(4): No need to disable the whole APIC subsystem for CPU idle states C3 and deeper
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         147180
>Category:       docs
>Synopsis:       [patch] Fix manpage of acpi(4): No need to disable the whole APIC subsystem for CPU idle states C3 and deeper
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-doc
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          doc-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sat May 29 10:30:08 UTC 2010
>Closed-Date:    Mon Jun 28 08:12:38 UTC 2010
>Last-Modified:  Mon Jun 28 08:20:01 UTC 2010
>Originator:     Tobias Rehbein
>Release:        FreeBSD 8.0-RELEASE-p2 i386
>Organization:
>Environment:
System: FreeBSD sushi.pseudo.local 8.0-RELEASE-p2 FreeBSD 8.0-RELEASE-p2 #5: Fri Jan 15 19:16:49 CET 2010 tobi@sushi.pseudo.local:/usr/obj/usr/src/sys/SUSHI i386


	
>Description:
	
        Since http://svn.freebsd.org/changeset/base/191720 it's no longer needed to disable the whole APIC
        subsystem to use CPU idle states C3 and deeper, it's sufficient to disable the LAPIC timer. I cc'ed
        mav@ who implemented this feature to veto this PR if my statement is invalid.

        Also I'm quite sure the LAPIC conflicts are not relevant for CPU idle state C2 but only for C3 and
        deeper.
>How-To-Repeat:
	
>Fix:

	

--- share_man_man4_acpi.4.diff begins here ---
--- /usr/src/share/man/man4/acpi.4	2009-11-26 20:09:36.000000000 +0100
+++ share/man/man4/acpi.4	2010-05-29 12:16:21.000000000 +0200
@@ -592,10 +592,8 @@
 .Nm
 CPU idle power management drive conflicts with the local APIC (LAPIC)
 timer.
-Disable APIC mode with
-.Va hint.apic.0.disabled
+Disable the local APIC timer with
+.Va hint.apic.0.clock=0
 or do not use the
-.Li C2
-and
 .Li C3
-states if APIC mode is enabled.
+and deeper states if the local APIC timer is enabled.
--- share_man_man4_acpi.4.diff ends here ---


>Release-Note:
>Audit-Trail:

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/147180: commit references a PR
Date: Wed,  2 Jun 2010 08:43:28 +0000 (UTC)

 Author: mav
 Date: Wed Jun  2 08:43:18 2010
 New Revision: 208729
 URL: http://svn.freebsd.org/changeset/base/208729
 
 Log:
   Recommend disabling LAPIC timer instead whole APIC for fixing C3 state.
   
   PR:		docs/147180
   Submitted by:	Tobias Rehbein
 
 Modified:
   head/share/man/man4/acpi.4
 
 Modified: head/share/man/man4/acpi.4
 ==============================================================================
 --- head/share/man/man4/acpi.4	Wed Jun  2 07:47:29 2010	(r208728)
 +++ head/share/man/man4/acpi.4	Wed Jun  2 08:43:18 2010	(r208729)
 @@ -25,7 +25,7 @@
  .\"
  .\" $FreeBSD$
  .\"
 -.Dd March 9, 2010
 +.Dd June 2, 2010
  .Dt ACPI 4
  .Os
  .Sh NAME
 @@ -599,10 +599,8 @@ The
  .Nm
  CPU idle power management drive conflicts with the local APIC (LAPIC)
  timer.
 -Disable APIC mode with
 -.Va hint.apic.0.disabled
 +Disable the local APIC timer with
 +.Va hint.apic.0.clock=0
  or do not use the
 -.Li C2
 -and
  .Li C3
 -states if APIC mode is enabled.
 +and deeper states if the local APIC timer is enabled.
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 

From: Tobias Rehbein <tobias.rehbein@web.de>
To: bug-followup@FreeBSD.org, freebsd-doc@FreeBSD.org
Cc: mav@FreeBSD.org
Subject: Re: docs/147180: Fix manpage of acpi(4): No need to disable the whole APIC subsystem for CPU idle states C3 and deeper
Date: Fri, 25 Jun 2010 22:59:21 +0200

 --G4iJoqBmSsgzjUCe
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 
 Hi.
 
 Thanks for commiting this stuff. It's the first patch I created which has been
 commited to the FreeBSD base system ;)
 
 Nonetheless I wanted to query the state of this PR. The attached patch has been
 committed but the PR is still open and no one has taken responsibility for this
 PR. I thought it should have been closed or at least set to state 'patched'.
 
 Regards Tobias
 
 
 --G4iJoqBmSsgzjUCe
 Content-Type: application/pgp-signature
 Content-Disposition: inline
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v2.0.15 (FreeBSD)
 
 iEYEARECAAYFAkwlGKgACgkQ6LYxOE8q4xTXsgCgh9pkKyJuDOoS+m55n0ydvlop
 JNQAoKfLIZALsSzCQPbxd98ch6y2wV3J
 =CfJ5
 -----END PGP SIGNATURE-----
 
 --G4iJoqBmSsgzjUCe--
State-Changed-From-To: open->closed 
State-Changed-By: mav 
State-Changed-When: Mon Jun 28 08:11:30 UTC 2010 
State-Changed-Why:  
Patch committed to HEAD and merged to 8-STABLE. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147180 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: docs/147180: commit references a PR
Date: Mon, 28 Jun 2010 08:11:06 +0000 (UTC)

 Author: mav
 Date: Mon Jun 28 08:10:55 2010
 New Revision: 209566
 URL: http://svn.freebsd.org/changeset/base/209566
 
 Log:
   MFC r208729:
   Recommend disabling LAPIC timer instead whole APIC for fixing C3 state.
   
   PR:             docs/147180
   Submitted by:   Tobias Rehbein
 
 Modified:
   stable/8/share/man/man4/acpi.4
 Directory Properties:
   stable/8/share/man/man4/   (props changed)
 
 Modified: stable/8/share/man/man4/acpi.4
 ==============================================================================
 --- stable/8/share/man/man4/acpi.4	Mon Jun 28 08:09:33 2010	(r209565)
 +++ stable/8/share/man/man4/acpi.4	Mon Jun 28 08:10:55 2010	(r209566)
 @@ -25,7 +25,7 @@
  .\"
  .\" $FreeBSD$
  .\"
 -.Dd March 9, 2010
 +.Dd June 2, 2010
  .Dt ACPI 4
  .Os
  .Sh NAME
 @@ -599,10 +599,8 @@ The
  .Nm
  CPU idle power management drive conflicts with the local APIC (LAPIC)
  timer.
 -Disable APIC mode with
 -.Va hint.apic.0.disabled
 +Disable the local APIC timer with
 +.Va hint.apic.0.clock=0
  or do not use the
 -.Li C2
 -and
  .Li C3
 -states if APIC mode is enabled.
 +and deeper states if the local APIC timer is enabled.
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 
>Unformatted:
