[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freebsd-hackers
Subject:    Re: SSD - trim fails
From:       Warner Losh <imp () bsdimp ! com>
Date:       2022-11-07 20:55:40
Message-ID: CANCZdfpWJude0-q=z2ZVFPuDv81p6dxd1Uo4115+6an9q_-f9A () mail ! gmail ! com
[Download RAW message or body]

On Sun, Nov 6, 2022 at 3:44 PM Wojciech Puchar <wojtek@puchar.net> wrote:

> > Next, maybe the drive is lying the size of the DSM it will support, but
> again, this is a weird
> > message to report a request that's too long with.
> >
> > Maybe it doesn't support queued DSM, despite all appearances to the
> contrary from its
> > identify tables. Try setting the trem method to DSM_TRIM:
> > # sysctl kern.cam.ada.0.delete_method=DSM_TRIM
> > should do the trick. At the very least, that will change the command we
> send so if it can't
>
>
> this works. but not from /boot/loader.conf
> i put it in /etc/rc at beginning so it for sure is done before doing first
> trim.
>

Can you file a bug for this?

Also, I'm thinking that maybe this should be the default for these drives,
so if you could
send me a camcontrol identify for  it?

Warner

[Attachment #3 (text/html)]

<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" \
class="gmail_attr">On Sun, Nov 6, 2022 at 3:44 PM Wojciech Puchar &lt;<a \
href="mailto:wojtek@puchar.net">wojtek@puchar.net</a>&gt; wrote:<br></div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">&gt; Next, maybe the drive is lying the size of \
the DSM it will support, but again, this is a weird<br> &gt; message to report a \
request that&#39;s too long with.<br> &gt; <br>
&gt; Maybe it doesn&#39;t support queued DSM, despite all appearances to the contrary \
from its<br> &gt; identify tables. Try setting the trem method to DSM_TRIM:<br>
&gt; # sysctl kern.cam.ada.0.delete_method=DSM_TRIM<br>
&gt; should do the trick. At the very least, that will change the command we send so \
if it can&#39;t<br> <br>
<br>
this works. but not from /boot/loader.conf<br>
i put it in /etc/rc at beginning so it for sure is done before doing first <br>
trim.<br></blockquote><div><br></div><div>Can you file a bug for \
this?</div><div><br></div><div>Also, I&#39;m thinking that maybe this should be the \
default for these drives, so if you could</div><div>send me a camcontrol identify for \
it?</div><div><br></div><div>Warner  </div></div></div>



[prev in list] [next in list] [prev in thread] [next in thread]