Date: Thu, 22 May 2014 05:56:45 +0100 From: "Steven Hartland" <killing@multiplay.co.uk> To: "Warren Block" <wblock@wonkity.com>, "Allan Jude" <allanjude@freebsd.org> Cc: freebsd-fs@freebsd.org, Benedict Reuschling <bcr@freebsd.org>, Eitan Adler <eadler@freebsd.org> Subject: Re: [patch] zfs sysctl patch Message-ID: <7B840D2D10124A4FAC40C69E91E6C20D@multiplay.co.uk> References: <537D7431.4070103@freebsd.org> <alpine.BSF.2.00.1405212222170.28790@wonkity.com>
next in thread | previous in thread | raw e-mail | index | archive | help
----- 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7B840D2D10124A4FAC40C69E91E6C20D>