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>
index | next in thread | previous in thread | raw e-mail
----- 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
home |
help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7B840D2D10124A4FAC40C69E91E6C20D>