Actions
Bug #12051
closedre-enable ZFS trim by default
Start date:
Due date:
% Done:
100%
Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
External Bug:
Description
I have tested a variety of SSDs with the fix from commit:
d0562c105d 11754 ZoL trim port used wrong offset for sd unmap
and everything seems to be working correctly. At this point I think it is safe to re-enable trim support.
Related issues
Updated by Jerry Jelinek over 3 years ago
I have tested the following different SSDs
ATA INTEL SSDSC2BX01 1490.42 GiB HGST HUSMH8010BSS204 93.16 GiB HGST HUSMR1619ASS204 1788.50 GiB HGST HUSMR3240ASS204 372.61 GiB SAMSUNG MZILS3T8HCJM/003 3576.98 GiB TOSHIBA PX04SRB192 1788.50 GiB WDC WUSTR6440ASS204 372.61 GiB
For testing, I created a zpool, or used the existing zpool, on the device, wrote a bunch of files into the dataset, then deleted most of those. I ran a trim and verified that 'zpool status' showed the device was trimming for some period of time. After the trim was complete, I scrubbed the zpool to verify there were no errors, and then exported/imported the zpool to confirm there were no problems. For the system's main pool (rpool or zones), I could not export the live pool so I rebooted to confirm things were ok.
There is no way we can test every SSD out there, but our code is consistent with the upstream philosophy from OpenZFS. They do not have a blacklist either, and recommend that people simply buy good quality SSDs if they want to use trim.
Updated by Electric Monk over 3 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
git commit fc5c75cf5edb072564020725faa0c4313714f09f
commit fc5c75cf5edb072564020725faa0c4313714f09f Author: Jerry Jelinek <jerry.jelinek@joyent.com> Date: 2019-12-18T18:02:41.000Z 12051 re-enable ZFS trim by default Reviewed by: Andrew Stormont <andyjstormont@gmail.com> Approved by: Robert Mustacchi <rm@fingolfin.org>
Updated by Marcel Telka about 3 years ago
- Related to Bug #11754: ZoL trim port used wrong offset for sd unmap added
Updated by Marcel Telka about 3 years ago
- Related to Bug #11714: temporarily disable ZFS TRIM support added
Actions