Date: Wed, 14 Mar 2018 10:08:13 -0700 From: Ravi Pokala <rpokala@mac.com> To: Warner Losh <imp@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r330932 - in head/sys: cam/nvme dev/nvme Message-ID: <22796282-7844-4E9A-AC32-10B09822FD25@mac.com> In-Reply-To: <201803141644.w2EGioti046140@repo.freebsd.org> References: <201803141644.w2EGioti046140@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Warner,
The TRIM command takes a buffer of range structures, and this change consol=
idates multiple range structures into the buffer for a single TRIM command, =
correct? Whereas the old functionality was to populate the buffer with a sin=
gle range structure?
So if we wanted to trim ranges [P..T], [A..E], [K..O], the old functionalit=
y would be:
TRIM:[P..T] ; TRIM:[A..E] ; TRIM:[K..O]
The new functionality would be:
TRIM:[P..T], [A..E], [K..O]
Right?
> + /* XXX -- Could collapse adjacent ranges, but we don't for now */
> + /* XXX -- Could limit based on total payload size */
And that future enhancement would make it:
TRIM:[A..E], [K..T]
Is that correct?
Thanks,
Ravi (rpokala@)
=EF=BB=BF-----Original Message-----
From: <owner-src-committers@freebsd.org> on behalf of Warner Losh <imp@Free=
BSD.org>
Date: 2018-03-14, Wednesday at 09:44
To: <src-committers@freebsd.org>, <svn-src-all@freebsd.org>, <svn-src-head@=
freebsd.org>
Subject: svn commit: r330932 - in head/sys: cam/nvme dev/nvme
> Author: imp
> Date: Wed Mar 14 16:44:50 2018
> New Revision: 330932
> URL: https://svnweb.freebsd.org/changeset/base/330932
>=20
> Log:
> Implement trim collapsing in nda
> =20
> When multiple trims are in the queue, collapse them as much as
> possible. At present, this usually results in only a few trims being
> collapsed together, but more work on that will make it possible to do
> hundreds (up to some configurable max).
> =20
> Sponsored by: Netflix
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?22796282-7844-4E9A-AC32-10B09822FD25>