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