Jump to content

Talk:Filesystems: Difference between revisions

From NixOS Wiki
Pigeon (talk | contribs)
Pigs (talk | contribs)
Line 18: Line 18:
:::Exactly. A typo on my side.  
:::Exactly. A typo on my side.  
:::Also it seems that <code>discard=sync</code> is [https://btrfs.readthedocs.io/en/latest/Trim.html#trim-discard BTRFS-specific]. From what i can see after doing a bit of googling continuous trim is commonly enabled with <code>discard</code> option so we should just not mention <code>discard=sync</code> in that section. [[User:Pigeon|Pigeon]] ([[User talk:Pigeon|talk]]) 17:24, 6 June 2025 (UTC)
:::Also it seems that <code>discard=sync</code> is [https://btrfs.readthedocs.io/en/latest/Trim.html#trim-discard BTRFS-specific]. From what i can see after doing a bit of googling continuous trim is commonly enabled with <code>discard</code> option so we should just not mention <code>discard=sync</code> in that section. [[User:Pigeon|Pigeon]] ([[User talk:Pigeon|talk]]) 17:24, 6 June 2025 (UTC)
::::I think it is reasonable to leave `discard=sync` as a user might see `discard=async` and think they should remove it. [[User:Pigs|Pigs]] ([[User talk:Pigs|talk]]) 17:43, 6 June 2025 (UTC)

Revision as of 17:43, 6 June 2025

#SSD_TRIM_support definitely needs improvement

1) Periodic trim is already enabled by default

2) No mention of differences between periodic and continuous trim. Continuous trim is often not recommended.

3) On btrfs, discard=sync is enabled by default and using discard (or discard=sync) is discouraged

Pigeon (talk) 15:54, 6 June 2025 (UTC)Reply

Also,  from man mount(8):
noatime
Do not update inode access times on this filesystem (e.g. for faster access on the news spool to speed up news servers). This works for all inode types (directories too), so it implies nodiratime.
Hence everything stated above renders the whole Filesystems#SSD TRIM support section mostly redundant. I am a wiki noob so i hesitate take any actions myself. What do you think? Pigeon (talk) 16:07, 6 June 2025 (UTC)Reply
Good catch, I updated the section.
On point number 3, I am seeing that on btrfs `discard=async` is what is set to as default. Pigs (talk) 16:44, 6 June 2025 (UTC)Reply
> On point number 3, I am seeing that on btrfs `discard=async` is what is set to as default.
Exactly. A typo on my side.
Also it seems that discard=sync is BTRFS-specific. From what i can see after doing a bit of googling continuous trim is commonly enabled with discard option so we should just not mention discard=sync in that section. Pigeon (talk) 17:24, 6 June 2025 (UTC)Reply
I think it is reasonable to leave `discard=sync` as a user might see `discard=async` and think they should remove it. Pigs (talk) 17:43, 6 June 2025 (UTC)Reply