On Tue, Oct 11, 2011 at 03:57:43PM -0500, James Bottomley wrote: > Well, when you put things like > > "NOT FOR COMMIT USE; TESTING ONLY; NO WARRANTY; DANGER!" > > and > > "I constructed this patch for my friend to test, completely blind. > I haven't heard back from him at all yet, but I thought that the wider > community might find it useful for testing as well. I think it _should_ > work, just need to figure out what the correct chip variant is." > > On the patch it tends to make maintainers exercise extreme caution. > Thus comments from others (preferably successful test reports) or acks > from Marvell or Vertex would be very helpful. Both of those were on the first version of the patch, when I'd written the code and not run it at all. After that, I got access to a RevoDrive3 remotely, and run about 5TiB of data through it for testing. The second version of the patch removed those warnings, and described some of my testing: http://thread.gmane.org/gmane.linux.kernel/1200731/focus=50312 Some fio performance numbers: http://paste.pocoo.org/show/489716/ dmesg/hdparm/smartctl: http://www.ocztechnologyforum.com/forum/showthread.php?95151-Linux-patch-support-for-RevoDrive3-RevoDrive3-X2-zDrive-R4&p=686054&viewfull=1#post686054 -- Robin Hugh Johnson Gentoo Linux: Developer, Trustee & Infrastructure Lead E-Mail : robbat2@xxxxxxxxxx GnuPG FP : 11AC BA4F 4778 E3F6 E4ED F38E B27B 944E 3488 4E85
Attachment:
signature.asc
Description: Digital signature
