From owner-freebsd-hackers@FreeBSD.ORG  Fri Feb 21 14:51:24 2014
Return-Path: <owner-freebsd-hackers@FreeBSD.ORG>
Delivered-To: freebsd-hackers@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115])
 (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id 520CD606;
 Fri, 21 Feb 2014 14:51:24 +0000 (UTC)
Received: from wonkity.com (wonkity.com [67.158.26.137])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (No client certificate requested)
 by mx1.freebsd.org (Postfix) with ESMTPS id 00C0016FA;
 Fri, 21 Feb 2014 14:51:23 +0000 (UTC)
Received: from wonkity.com (localhost [127.0.0.1])
 by wonkity.com (8.14.8/8.14.8) with ESMTP id s1LEpFNj009376
 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO);
 Fri, 21 Feb 2014 07:51:15 -0700 (MST)
 (envelope-from wblock@wonkity.com)
Received: from localhost (wblock@localhost)
 by wonkity.com (8.14.8/8.14.8/Submit) with ESMTP id s1LEpFt6009373;
 Fri, 21 Feb 2014 07:51:15 -0700 (MST)
 (envelope-from wblock@wonkity.com)
Date: Fri, 21 Feb 2014 07:51:15 -0700 (MST)
From: Warren Block <wblock@wonkity.com>
To: Hiroki Sato <hrs@FreeBSD.org>
Subject: Re: ata.4 changes
In-Reply-To: <20140221.211026.115752007914804800.hrs@allbsd.org>
Message-ID: <alpine.BSF.2.00.1402210748480.9125@wonkity.com>
References: <le7em3$qiv$1@ger.gmane.org>
 <20140221.211026.115752007914804800.hrs@allbsd.org>
User-Agent: Alpine 2.00 (BSF 1167 2008-08-23)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.4.3
 (wonkity.com [127.0.0.1]); Fri, 21 Feb 2014 07:51:15 -0700 (MST)
Cc: freebsd-hackers@FreeBSD.org, ivoras@FreeBSD.org
X-BeenThere: freebsd-hackers@freebsd.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Technical Discussions relating to FreeBSD
 <freebsd-hackers.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-hackers>, 
 <mailto:freebsd-hackers-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-hackers/>
List-Post: <mailto:freebsd-hackers@freebsd.org>
List-Help: <mailto:freebsd-hackers-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-hackers>,
 <mailto:freebsd-hackers-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 21 Feb 2014 14:51:24 -0000

On Fri, 21 Feb 2014, Hiroki Sato wrote:

> Ivan Voras <ivoras@freebsd.org> wrote
>  in <le7em3$qiv$1@ger.gmane.org>:
>
> iv> Hello,
> iv>
> iv> Can someone who knows more about man pages take a look at this patch to
> iv> ata.4:
> iv>
> iv> Index: ada.4
> iv> ===================================================================
> iv> --- ada.4    (revision 262293)
> iv> +++ ada.4    (working copy)
> iv> @@ -25,7 +25,7 @@
> iv>  .\"
> iv>  .\" $FreeBSD$
> iv>  .\"
> iv> -.Dd February 8, 2012
> iv> +.Dd February 21, 2014
> iv>  .Dt ADA 4
> iv>  .Os
> iv>  .Sh NAME
> iv> @@ -129,8 +129,14 @@
> iv>  These variables determines whether device write cache should be enabled

Not part of the original change, but since we are here: 
s/determines/determine/

> iv>  globally or per-device or disabled.
> iv>  Set to 1 to enable write cache, 0 to disable, -1 to leave it as-is.
> iv> -Values modified in runtime take effect only after device reset.
> iv> -The global default is currently enabled.
> iv> +Values modified in runtime take effect only after device reset (using
> iv> +.Xr camcontrol 8
> iv> +reset).
>
> .Pq should be used for ():
>
>  Values modified in runtime take effect only after device reset
>  .Pq using reset subcommand of Xr camcontrol 8 .
>
> iv> +Because of that, this setting should be changed in a
> iv> +.Xr loader 8
> iv> +tunable, not in
> iv> +.Pa /etc/sysctl.conf .
>
> I think /boot/loader.conf is easier to understand:
>
>  Because of that, this setting should be changed in
>  .Pa /boot/loader.conf ,
>  not in
>  .Pa /etc/sysctl.conf .

Agreed.