From owner-svn-src-head@FreeBSD.ORG Tue Nov 18 22:57:55 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 07365A18; Tue, 18 Nov 2014 22:57:55 +0000 (UTC) Received: from anubis.delphij.net (anubis.delphij.net [IPv6:2001:470:1:117::25]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "anubis.delphij.net", Issuer "StartCom Class 1 Primary Intermediate Server CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id DB689F8F; Tue, 18 Nov 2014 22:57:54 +0000 (UTC) Received: from zeta.ixsystems.com (unknown [12.229.62.2]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by anubis.delphij.net (Postfix) with ESMTPSA id 5B628221D5; Tue, 18 Nov 2014 14:57:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=delphij.net; s=anubis; t=1416351474; x=1416365874; bh=sEEA2TvypbiI1Ye7Qg4RBhcKldfwXw5hCKXrdQe9Iy0=; h=Date:From:Reply-To:To:CC:Subject:References:In-Reply-To; b=K+9FrWjA1o4qMZ8ygEe6NerSQxYOtmGEc2M39uFlIJoyu3KKUsZjKzpjTqfcRr5ZZ WftwMLMqYu7VqwEyNHUkw7q+roz0seS0h3tkvMNBJ5g7YmIl4NF8ST3GY6SSqibOPG W5YNA8kRpEU37L8PIWJNoBUrvdYI2rthHtQLnQuo= Message-ID: <546BCEF1.7080009@delphij.net> Date: Tue, 18 Nov 2014 14:57:53 -0800 From: Xin Li Reply-To: d@delphij.net Organization: The FreeBSD Project MIME-Version: 1.0 To: Slawa Olhovchenkov , Xin LI Subject: Re: svn commit: r274673 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs References: <201411181840.sAIIe2C7094558@svn.freebsd.org> <20141118220030.GA31347@zxy.spb.ru> In-Reply-To: <20141118220030.GA31347@zxy.spb.ru> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Nov 2014 22:57:55 -0000 -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 On 11/18/14 14:00, Slawa Olhovchenkov wrote: > On Tue, Nov 18, 2014 at 06:40:02PM +0000, Xin LI wrote: > >> Author: delphij Date: Tue Nov 18 18:40:01 2014 New Revision: >> 274673 URL: https://svnweb.freebsd.org/changeset/base/274673 >> >> Log: Allow tuning zfs_max_recordsize via loader tunable. Tuning >> is NOT recommended. > > This is not tuning, this is simple guard. And can be safe changed > (incrased, at the least) in any time. FWIW: It can be safely decreased: the ability to read such pool is maintained. > For some workload optimal is maximum, i.e. 16MB. > > May be better comment: increasing this value required strong > understaning after-effects of increasing memory consumption and > bulk data transfers. (sorry for english) That's true, but not free. I don't think we have carefully evaluated the situation (and possible mitigation measures, like, do we need to make some improvements to the slab allocator? It's never exposed to workload like this before, etc.), and people tend to make uninformed changes all the times (a lot of FreeBSD tuning guide gives wrong instructions). >> Requested by: Slawa Olhovchenkov MFC after: 2 >> weeks > > Thanks, can you allow RW? Sure, done in 274681. I didn't do it at the beginning because I don't think it's a good idea to make it easy for users to fiddle with it, and power users who actually need to tweak do not change it often. But you are right there is no real technical reason not to allow read-write. >> Modified: >> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_dataset.c >> >> >> Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_dataset.c >> ============================================================================== >> >> - --- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_dataset.c Tue Nov 18 18:03:40 2014 (r274672) >> +++ >> head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/dsl_dataset.c >> Tue Nov 18 18:40:01 2014 (r274673) @@ -51,6 +51,8 @@ #include >> #include >> >> +SYSCTL_DECL(_vfs_zfs); + /* * The SPA supports block sizes up to >> 16MB. However, very large blocks * can have an impact on i/o >> latency (e.g. tying up a spinning disk for @@ -61,6 +63,9 @@ * of >> this setting. */ int zfs_max_recordsize = 1 * 1024 * 1024; >> +SYSCTL_INT(_vfs_zfs, OID_AUTO, max_recordsize, CTLFLAG_RDTUN, + >> &zfs_max_recordsize, 0, + "Maximum block size. Expect dragons >> when tuning this."); >> >> #define SWITCH64(x, y) \ { \ >> _______________________________________________ >> svn-src-all@freebsd.org mailing list >> http://lists.freebsd.org/mailman/listinfo/svn-src-all To >> unsubscribe, send any mail to >> "svn-src-all-unsubscribe@freebsd.org" - -- Xin LI https://www.delphij.net/ FreeBSD - The Power to Serve! Live free or die -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0 iQIcBAEBCgAGBQJUa87xAAoJEJW2GBstM+nsPrgP/0o9O/AiiFJZhCezZc29oMQe Za4m5U3sWCPTb1x5fc3A5pKq1frT2GUvbUAjh21yuN5A8GfMt8A4v+nQ4i/asZ3C GkyU7Zkv0sW/14DR3nhRrLTNCum6p2Yk0XkxDBnUlBVzoJzbbbqin9PtTbe6i330 mav6E17V0AelCk8BoWpnhCcQbLObP9zvklzqknjlPHl5d81ERhfyp1p+BjPexdML ykOb3ZsjC0+8MfNmlKEyUga6Saz6vQXVqtXhkMuvqE8VNSfedXO+EpOweOID2SDU 7S4bwkFfu0GuBdqHSXTWsp3ncNiN/KfuX5fAUjb8Aoqp2RCHbft+DS2mnxSw/S7/ cUpOoC1ga+cUU4KLEgm3DG8y1LZH3KPyrzwFzXfY31lgOttCosQi9IeWgigGW975 3jhU7Oz+CX6oMPoAWhujAD3yGKVp2Qp2lePl12pl+F73jrwJ7UaTbKWZlNkofX0T o9/u/tqtna7NY3w9/z7WRDQFZvFEp2laARNAlr8QcakIuCXsDnG26LxFHbtLld0M R/88EYYJgWLPWD7TzapkrKhqbaf6sn6Z7m4M0Oo+MbFRNSII59/rTZsAbDI9PgiZ ScnaQldbnJdi0yMp8VyHrPDFqJtByhL/dQvmcTYf2QItnFEu1Hf3Os1nD+p3+8Z+ sdEBf8XuZdlMIcRIzFR2 =Gccl -----END PGP SIGNATURE-----