Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Feb 2014 07:51:15 -0700 (MST)
From:      Warren Block <wblock@wonkity.com>
To:        Hiroki Sato <hrs@FreeBSD.org>
Cc:        freebsd-hackers@FreeBSD.org, ivoras@FreeBSD.org
Subject:   Re: ata.4 changes
Message-ID:  <alpine.BSF.2.00.1402210748480.9125@wonkity.com>
In-Reply-To: <20140221.211026.115752007914804800.hrs@allbsd.org>
References:  <le7em3$qiv$1@ger.gmane.org> <20140221.211026.115752007914804800.hrs@allbsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.1402210748480.9125>