Ahci(4) needed for TRIM
-
Thanks jimp! :D
Looks like using rc.conf.local is not really needed, because during update rc.conf is edited, not deleted, so my things added into rc.conf years ago are still working. But to be safe… yes it's not bad to use rc.conf.local instead
-
smartctl 6.1 2013-03-16 r3800 [FreeBSD 8.3-RELEASE-p8 i386] (local build) Copyright (C) 2002-13, Bruce Allen, Christian Franke, www.smartmontools.org /dev/: Unable to detect device type Please specify device type with the -d option. Use smartctl -h to get a usage summaryI am getting this when trying to get S.M.A.R.T., after ahci enabled, any suggestions?
-
@w0w:
I am getting this when trying to get S.M.A.R.T., after ahci enabled, any suggestions?
The device name changed with ahci, ie from ad4 to ada0 in my case, and the extra 'a' is not being picked up by the regexp in diag_smart.php
This change to /usr/local/www/diag_smart.php will pick up the devices with the extra 'a', if it's there.
--- diag_smart.php.orig 2013-06-27 22:25:23.000000000 -0400 +++ diag_smart.php 2013-06-27 22:20:08.000000000 -0400 @@ -277,7 +277,7 @@ default: { // Get all AD* and DA* (IDE and SCSI) devices currently installed and stores them in the $devs array - exec("ls /dev | grep '^[ad][da][0-9]\{1,2\}$'", $devs); + exec("ls /dev | grep '^[ad][da]a*[0-9]\{1,2\}$'", $devs); ?> -
Thank you charliem!
Hmm, may be it should be submitted here - https://github.com/pfsense/pfsense/blob/master/usr/local/www/diag_smart.php ?Or somebody can do the patch that can be applied with "Patches" package. This one posted does not look compatible.
I have patched manually but…
Patch works via "System Patches" package. Sorry ;) -
Use with caution!
I've cleaned up the "patch" a bit and added one extra for the dashboard widget! ;D1st:
Description : "AHCI SMART fix"--- diag_smart.php.orig 2013-06-27 22:25:23.000000000 -0400 +++ diag_smart.php 2013-07-16 22:20:08.000000000 -0400 @@ -280,1 +280,1 @@ - exec("ls /dev | grep '^[ad][da][0-9]\{1,2\}$'", $devs); + exec("ls /dev | grep '^[ad][da]a*[0-9]\{1,2\}$'", $devs);Base Directory: "/usr/local/www/"
(Please test first!!! Line numbers may change or the patch may not be needed in the future!!!)2nd (for dashboard widget) ;)
Description : "ACHI dashboard fix"--- smart_status.widget.php.orig 2013-07-16 22:25:23.000000000 -0400 +++ smart_status.widget.php 2013-07-16 22:20:08.000000000 -0400 @@ -45,1 +45,1 @@ - exec("ls /dev | grep '^[ad][da][0-9]\{1,2\}$'", $devs); ## leant from orginal SMART status screen + exec("ls /dev | grep '^[ad][da]a*[0-9]\{1,2\}$'", $devs); ## leant from orginal SMART status screenBase Directory: "/usr/local/www/widgets/widgets/"
(Please test first!!! Line numbers may change or the patch may not be needed in the future!!!)It works for me..
-
I think this is great, but TRIM on a SSD is so common like dirt by now and required, that it should either be:
1. Something that is just automagically detected and applied (like it does for any other common device)
or
2. Asked at initial install "would you like to enable TRIM on this device."
Maybe next decade.
-
AHCI is great! for TCQ & NCQ on normal HDD's not just for TRIM on SSD's.
"ada0: Command Queueing enabled"And for SATA 6Gbs support.
ada0: 600.000MB/s transfers (SATA 3.x, UDMA5, PIO 8192bytes) -
The AHCI module should be included in snapshots now (I just updated a VM and checked, it's there).
Also look at /usr/local/sbin/ufslabels.sh - that would eliminate the need to make any fstab edits.
Too late for 2.1 to grow an installer option for it, but perhaps for 2.2.
-
Thanks Jimp, just started a fresh installation on my main pf box using a new SSD and tried out the latest snap. Works perfectly.
Couldnt figure out how to run the ufslabels.sh file. Getting a permission denied at a shell, so just edited fstab the old fashioned way.
-
It's just missing exec bits
chmod a+x ufslabels.sh; ./ufslabels.shOr:
sh ufslabels.shI pushed a fix to correct the exec bits, it'll turn up in shapshots eventually
-
Does that printout look normal with the unexpected operator? End result seems to be correct.
[2.1-RC0][root@pfsense@pfsenseuser.net]/root(17): chmod a+x /usr/local/sbin/ufslabels.sh ; /usr/local/sbin/ufslabels.sh FS: / on device ada0s1a with ufsid 51f5bc9c854a3254 FS: Swap on device ada0s1b ==================== Current fstab: # Device Mountpoint FStype Options Dump Pass# /dev/ada0s1a / ufs rw 1 1 /dev/ada0s1b none swap sw 0 0 ==================== New fstab: # Device Mountpoint FStype Options Dump Pass# /dev/ufsid/51f5bc9c854a3254 / ufs rw 11 /dev/label/swap none swap sw 0 0 [: =: unexpected operator Commit changes? (y/n): y Disabling swap to apply label Applying label to swap parition Activating new fstab Re-enabling swap[/code] -
That's not fatal (it was in code to auto-commit the changes if you passed in the right command line parameter), but it isn't relevant to using it interactively, so you can proceed from there.
-
Looks like SMART is also fixed
https://redmine.pfsense.org/projects/pfsense/repository/revisions/ccf479876283082b39ac9a37c1c6308f4857a6f0/diffThanks jimp!
-
For completeness the smart dashboard widget needs the same fix.
-
-
I just forgot about the widget entirely in the previous commit.
Done now though:
https://github.com/pfsense/pfsense/commit/0aa297594cd1882f32d52074ba3546c4f5f97c8e -
Thanks to your guides I think I have enabled TRIM on my SSD:
/root(5): tunefs -p / tunefs: POSIX.1e ACLs: (-a) disabled tunefs: NFSv4 ACLs: (-N) disabled tunefs: MAC multilabel: (-l) disabled tunefs: soft updates: (-n) disabled tunefs: gjournal: (-J) disabled tunefs: trim: (-t) enabled tunefs: maximum blocks per file in a cylinder group: (-e) 2048 tunefs: average file size: (-f) 16384 tunefs: average number of files in a directory: (-s) 64 tunefs: minimum percentage of free space: (-m) 8% tunefs: optimization preference: (-o) time tunefs: volume label: (-L)and from dmesg
ada0 at ahcich0 bus 0 scbus0 target 0 lun 0 ada0: <ts16gssd25s-s v1210=""> ATA-8 SATA 2.x device ada0: 300.000MB/s transfers (SATA 2.x, UDMA5, PIO 512bytes) ada0: Command Queueing enabled ada0: 15272MB (31277056 512 byte sectors: 16H 63S/T 16383C) lapic1: Forcing LINT1 to edge trigger SMP: AP CPU #1 Launched! Trying to mount root from ufs:/dev/ufsid/51fa1b22d7786f5c WARNING: /: TRIM flag on fs but disk does not support TRIM</ts16gssd25s-s>Why is there still that last line that my disk doesn't support it?
I have this one:
=== START OF INFORMATION SECTION ===
Model Family: JMicron based SSDs
Device Model: TS16GSSD25S-S
Serial Number: 00246293003C
Firmware Version: V1210According to data sheet it does support trim command.
-
It appears you have a similar drive to mine, these drives are older and do not support TRIM.
Model Family: JMicron based SSDs
Device Model: TS32GSSD25S-M -
That's weird because on this page:
http://www.transcend-info.com/industry/products_details.asp?ModNo=3&Func1No=1
Transcend says it does support it. Look at the Features tab and under Specifications SSD25S is named. I have the SLC and you have the MLC.
-
I'll be honest, its been years since i've read or done anything with this drive. Last I read (for mine) it didn't support TRIM, but had "Wear-leveling algorithm ensures reliable data transfer", which I had assumed was pre-TRIM. My drive has been happily sitting in my router doing its job. It wasn't until this thread that I started playing with it again.
Mine also doesn't appear to have command queueing either.
ada0 at ahcich0 bus 0 scbus0 target 0 lun 0 ada0: <ts32gssd25s-m 100415=""> ATA-7 SATA 2.x device ada0: 300.000MB/s transfers (SATA 2.x, UDMA6, PIO 512bytes) ada0: 30560MB (62586880 512 byte sectors: 16H 63S/T 16383C) WARNING: /: TRIM flag on fs but disk does not support TRIM</ts32gssd25s-m>I wonder if there is a newer firmware that adds TRIM?
EDIT:
I don't see in the spec sheets where either drive supports TRIM.
TS16GSSD25S-S
http://www.csee.umbc.edu/~squire/images/ssd3.pdf
http://c.master.lv/p/01/53/40/05/F611373.pdf
