Re: Updated TRIM patches and a graph

[Tim Bisson <bissont@xxxxxxx>]

On 10/6/11 1:41 AM, Samuel J. Greear wrote:
> On Sat, Aug 20, 2011 at 4:34 PM, Tim Bisson<bissont@mac.com>  wrote:
> > Hi All,
> >
> > I've updated the TRIM patches to include Sam Greear's comments. Thanks Sam.
> >
> > The updated code and graph below can be found here:
> > http://www.box.net/shared/0os5q17q12f68p7k9r89
> >
> > I also created a graph showing the effect of "random small writes" on
> > sequential read performance on an SSD. What I did was TRIM the device, then
> > dirty the device for X minutes, then sequentially read through the entire
> > device. The graph shows the average read throughput of reading the entire
> > device.
> >
> > This implies that if you end up doing a lot of small-write updates and
> > sequential reads, you should use TRIM...
> >
> > I'm trying to get numbers while using online UFS trim, but I'm still trying
> > to debug the "fsstress&  UFS" bug (appears on master w/out TRIM too):
> > http://bugs.dragonflybsd.org/issue2102
> >
> > Thanks,
> > Tim
> Tim,
>
> Is this "whole_thing.diff" ready to commit in your opinion?
>
> Best,
> Sam

Hi Sam,

Yes, I would say it's ready to commit. And all TRIM functionality is off 
by default.

Tim