From owner-svn-src-head@FreeBSD.ORG Tue Nov 26 13:24:12 2013 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 3CC99442; Tue, 26 Nov 2013 13:24:12 +0000 (UTC) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 0DD4F2DC2; Tue, 26 Nov 2013 13:24:10 +0000 (UTC) Received: from porto.starpoint.kiev.ua (porto-e.starpoint.kiev.ua [212.40.38.100]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id PAA13294; Tue, 26 Nov 2013 15:24:02 +0200 (EET) (envelope-from avg@FreeBSD.org) Received: from localhost ([127.0.0.1]) by porto.starpoint.kiev.ua with esmtp (Exim 4.34 (FreeBSD)) id 1VlIcT-00016M-RB; Tue, 26 Nov 2013 15:24:01 +0200 Message-ID: <5294A0CD.8030000@FreeBSD.org> Date: Tue, 26 Nov 2013 15:23:25 +0200 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 MIME-Version: 1.0 To: Steven Hartland , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org Subject: Re: svn commit: r258632 - in head: cddl/contrib/opensolaris/cmd/ztest cddl/contrib/opensolaris/lib/libzpool/common/sys sys/cddl/compat/opensolaris/sys sys/cddl/contrib/opensolaris/uts/common/fs/zfs sys... References: <201311260957.rAQ9vF6d004168@svn.freebsd.org> In-Reply-To: X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.16 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, 26 Nov 2013 13:24:12 -0000 on 26/11/2013 14:34 Steven Hartland said the following: > Can I ask why choose priority now for TRIM, does this not change the flow of > free requests quite significantly? Sorry, I couldn't grok the question... Also, please take into account that I know nothing about TRIM nor TRIM + ZFS. BTW, I appreciate the review but a timely review[*] would be appreciated even more :-) > Also as we have rotational information available should we apply > the following change which makes the active queue timestamp sorted > instead of offset sorted? > > An AVL as a FIFO queue seems like quite an expensive option. I've just merged the upstream change, perhaps badly. Any improvement are appreciated. [*] Message <5242F5AE.6090407@FreeBSD.org> to zfs-devel@FreeBSD.org. > Index: sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c > =================================================================== > --- sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c (revision 258644) > +++ sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c (working copy) > @@ -262,7 +263,9 @@ > mutex_init(&vq->vq_lock, NULL, MUTEX_DEFAULT, NULL); > vq->vq_vdev = vd; > > - avl_create(&vq->vq_active_tree, vdev_queue_offset_compare, > + avl_create(&vq->vq_active_tree, > + vd->vdev_rotation_rate == VDEV_RATE_NON_ROTATING ? > + vdev_queue_timestamp_compare : vdev_queue_offset_compare, > sizeof (zio_t), offsetof(struct zio, io_queue_node)); > > > ----- Original Message ----- From: "Andriy Gapon" > To: ; ; > > Sent: Tuesday, November 26, 2013 9:57 AM > Subject: svn commit: r258632 - in head: cddl/contrib/opensolaris/cmd/ztest > cddl/contrib/opensolaris/lib/libzpool/common/sys sys/cddl/compat/opensolaris/sys > sys/cddl/contrib/opensolaris/uts/common/fs/zfs sys... > > >> Author: avg >> Date: Tue Nov 26 09:57:14 2013 >> New Revision: 258632 >> URL: http://svnweb.freebsd.org/changeset/base/258632 >> >> Log: >> MFV r255255: 4045 zfs write throttle & i/o scheduler performance work >> >> illumos/illumos-gate@69962b5647e4a8b9b14998733b765925381b727e >> >> Please note the following changes: >> - zio_ioctl has lost its priority parameter and now TRIM is executed >> with 'now' priority >> - some knobs are gone and some new knobs are added; not all of them are >> exposed as tunables / sysctls yet > ... > > ================================================ > This e.mail is private and confidential between Multiplay (UK) Ltd. and the > person or entity to whom it is addressed. In the event of misdirection, the > recipient is prohibited from using, copying, printing or otherwise disseminating > it or any information contained in it. > In the event of misdirection, illegible or incomplete transmission please > telephone +44 845 868 1337 > or return the E.mail to postmaster@multiplay.co.uk. > -- Andriy Gapon