Hi, On 9/19/21 6:27 PM, Tor Vic wrote: > > > On 19.09.21 15:27, Hans de Goede wrote: >> Hi Tor, >> >> On 9/19/21 4:24 PM, Tor Vic wrote: >>> Hi, >>> >>> I saw that v2 (?) of this patch has made it into stable, which >>> is quite reasonable given the number of bug reports. >>> Are there any plans to "enhance" this patch once sufficient data >>> on controller support/drive combinations has been collected? >> >> ATM there are no plans to limit these quirks, we have bug >> reports of queued trims being an issue over all usual chip-vendors >> of sata controllers (including more recent AMD models). >> >> Note that unless you have immediate "discard" enabled as an option >> on all layers of your storage stack (dmcrypt, device-mapper/raid, >> filesystem) then this change will not impact you at all. > > Is that the "discard" mount option? > I added this to one of the partitions residing on my 860 Evo, > reverted the patch, and it still seems to work just fine. > > $ mount | grep sdb > > > /dev/sdb1 on /mnt/vbox type ext4 (rw,nosuid,nodev,noatime,discard) > > Is there another place where discard has to be enabled? No since you do not seem to be using dmcrypt/raid/lvm that should do the trick. Except that it sounds like this is a partition carrying vm images. Those never delete storage, they only grow, to effectively trim you need to either punch holes in files, or remove files. Discard only comes in to play when used diskspace becomes unused. To test preferably remove several large files at once while also generating a whole bunch of other diskio (e.g. compile the kernel while also deleting several large files from the same disk, with discard enable). But even if that works for you, that is 1 report that this works in some cases, vs many that it does not work; and also note that you had to manually enable this, it was not enabled before. So this really is going to impact the performance of very few users, while looking at the amount of bugreports about hangs / disk-corruption the problem of having queued-trim support enabled is much much bigger, so I see very little reason to re-enable this even if it happens to work in your case. > Or is there a way to check that discard is effectively enabled? There probably is, but I don't know where / how to check this. Regards, Hans