From owner-freebsd-fs@FreeBSD.ORG  Thu May 22 04:56:49 2014
Return-Path: <owner-freebsd-fs@FreeBSD.ORG>
Delivered-To: freebsd-fs@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by hub.freebsd.org (Postfix) with ESMTPS id 72CC04DA;
 Thu, 22 May 2014 04:56:49 +0000 (UTC)
Received: from smtp1.multiplay.co.uk (smtp1.multiplay.co.uk [85.236.96.35])
 by mx1.freebsd.org (Postfix) with ESMTP id 36A85248F;
 Thu, 22 May 2014 04:56:48 +0000 (UTC)
Received: by smtp1.multiplay.co.uk (Postfix, from userid 65534)
 id 495CF20E7088E; Thu, 22 May 2014 04:56:47 +0000 (UTC)
X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on
 smtp1.multiplay.co.uk
X-Spam-Level: **
X-Spam-Status: No, score=2.0 required=8.0 tests=AWL,BAYES_00,DOS_OE_TO_MX,
 FSL_HELO_NON_FQDN_1,HELO_NO_DOMAIN,RDNS_DYNAMIC autolearn=no version=3.3.1
Received: from r2d2 (82-69-141-170.dsl.in-addr.zen.co.uk [82.69.141.170])
 by smtp1.multiplay.co.uk (Postfix) with ESMTPS id D468C20E7088B;
 Thu, 22 May 2014 04:56:42 +0000 (UTC)
Message-ID: <7B840D2D10124A4FAC40C69E91E6C20D@multiplay.co.uk>
From: "Steven Hartland" <killing@multiplay.co.uk>
To: "Warren Block" <wblock@wonkity.com>, "Allan Jude" <allanjude@freebsd.org>
References: <537D7431.4070103@freebsd.org>
 <alpine.BSF.2.00.1405212222170.28790@wonkity.com>
Subject: Re: [patch] zfs sysctl patch
Date: Thu, 22 May 2014 05:56:45 +0100
MIME-Version: 1.0
Content-Type: text/plain; format=flowed; charset="iso-8859-1";
 reply-type=response
Content-Transfer-Encoding: 7bit
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 6.00.2900.5931
X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.6157
Cc: freebsd-fs@freebsd.org, Benedict Reuschling <bcr@freebsd.org>,
 Eitan Adler <eadler@freebsd.org>
X-BeenThere: freebsd-fs@freebsd.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: Filesystems <freebsd-fs.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/freebsd-fs>,
 <mailto:freebsd-fs-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-fs/>
List-Post: <mailto:freebsd-fs@freebsd.org>
List-Help: <mailto:freebsd-fs-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-fs>,
 <mailto:freebsd-fs-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 22 May 2014 04:56:49 -0000

----- Original Message ----- 
From: "Warren Block" <wblock@wonkity.com>
To: "Allan Jude" <allanjude@freebsd.org>
Cc: <freebsd-fs@freebsd.org>; "Benedict Reuschling" <bcr@freebsd.org>; "Eitan Adler" <eadler@freebsd.org>
Sent: Thursday, May 22, 2014 5:25 AM
Subject: Re: [patch] zfs sysctl patch


> On Wed, 21 May 2014, Allan Jude wrote:
> 
>> A recent commit (r266497 by smh) added a number of new sysctls for ZFS
>>
>> Two of these had minor typos, and the phrasing of another was very awkward.
>>
>> ---------------
>>
>> Improve sysctl descriptions for:
>> vfs.zfs.dirty_data_max
>> vfs.zfs.dirty_data_max_max
>> vfs.zfs.dirty_data_sync
> 
> Nice.  Approved for the doc side, but please also get approval from smh.

All good for me, thanks for reviewing and picking these up.

    Regards
    Steve