From owner-freebsd-geom@freebsd.org Fri Jul 10 20:00:09 2015 Return-Path: Delivered-To: freebsd-geom@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id ED6DA997CC1 for ; Fri, 10 Jul 2015 20:00:09 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from mail.dawidek.net (garage.dawidek.net [91.121.88.72]) by mx1.freebsd.org (Postfix) with ESMTP id B94791295 for ; Fri, 10 Jul 2015 20:00:09 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from localhost (unknown [91.206.210.19]) by mail.dawidek.net (Postfix) with ESMTPSA id D85A9FC4; Fri, 10 Jul 2015 21:59:49 +0200 (CEST) Date: Fri, 10 Jul 2015 22:01:01 +0200 From: Pawel Jakub Dawidek To: "Matthew D. Fuller" Cc: freebsd-geom@freebsd.org Subject: Re: RFC: Pass TRIM through GELI Message-ID: <20150710200055.GB1270@garage.freebsd.pl> References: <20150308000131.GP1742@over-yonder.net> <20150324021924.GQ52331@over-yonder.net> <20150502125220.GS78376@over-yonder.net> <20150629013841.GO50491@over-yonder.net> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="O5XBE6gyVG5Rl6Rj" Content-Disposition: inline In-Reply-To: <20150629013841.GO50491@over-yonder.net> X-OS: FreeBSD 11.0-CURRENT amd64 User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: freebsd-geom@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: GEOM-specific discussions and implementations List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 10 Jul 2015 20:00:10 -0000 --O5XBE6gyVG5Rl6Rj Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Jun 28, 2015 at 08:38:41PM -0500, Matthew D. Fuller wrote: > > > Stuffed into bugzilla as > > > > > [...] > > > After last round, everybody seems happy enough with this, so I've > > > filed it as > > > . > >=20 > > Does anybody have outstanding concerns on these? Or, if not, what > > else do we need to move them along? They're working fine for me > > here... >=20 > Ping... still working fine here, and I'm pretty sure I've addressed > every concern anybody's raised. Matthew, I'm sorry that it took me so long to get to your patch. The good news is that I like the patch - it looks clean and complete. The bad news is that I like it a bit too much:) I think I'd prefer that BIO_DELETE is passed through by default and there is an option to turn it off. This would mean changing -t option to -T for init and onetime and renaming the G_ELI_FLAG_DELETE flag to G_ELI_FLAG_IGNORE_DELETE. OR... just removing the ability to ignore BIO_DELETEs. The latter is appealing especially if some days we will implement BIO_DELETEs as overwrites. Then we should have an option to turn that on, which would turn off TRIM/UNMAP. Thinking about it some more, I believe that if someone doesn't want TRIM/UNMAP to hit his SSDs it should be configurable on per-SSD basis and not on every layer above SSD. So at the end I'd change my preference to just passing BIO_DELETEs always. What do you think? --=20 Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://mobter.com --O5XBE6gyVG5Rl6Rj Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCgAGBQJVoCR3AAoJEJVLhSuxKFt13fUP/juY4IFaXVXMFtgDZYVTWzsn TPkPslFj0jpW+qWYE4aH21WyrXuVe2r6tBaE8c4ao1MZ9EtJv3qWjFf1P6nELf7R RHY+8boW3fuTp7e/R1ijaMkScD5tWs5cHSkvrxrdXsh9LtcPEyEO5djxG+29oZ+9 t1/cSNW40quiTZthP0df7ljwRYT8uW/buO9n1EOLU1sPf8rhlqkoKxfV2FshUcbm 8TK3mAfUC3TIPxqcuMJUDAc+Nguo1FwW0qYwvIbHIHAg4UDEhlxy3QpYJbY9vhAe iX6+lpSAqUxfgs5NiEAiWeH340a81z/QVM3nG8HVPZaA0DTimmp++Lqa9pwv8p1U xFsKd3R/YdYmBbbepCyYS0G2qKmTUysvtns6RVDwLzZWXoctSdjXvAYoXDi9WG3G wVwSUWQGRad2j0QhffzWAC/ViXrsfFd3e3JyI1qWIq8XWMcXtPJeew01Ewox0PEQ 5mUvQYIpwdrbk5MB3V5jK/IGlVzMB0VOAcSIWwQYRwvlygjP5G37lyU0xkRc5u5f Abi0baZLUk79Wos8PyGbvMMAgee0H4Vk9PwZKQRbgl5yV+4LooUnYKeDrHecRDU4 baqnJzMMtxKavUCTpIJIXTASR22aqoMSXhX6K3EX/o+yG13qFHezkzMCeSrlUQaz jOd3m8wHgQgdZL7q2CW6 =WZrH -----END PGP SIGNATURE----- --O5XBE6gyVG5Rl6Rj--