Date: Fri, 3 May 2024 09:25:00 -0600 From: Warner Losh <imp@bsdimp.com> To: Ronald Klop <ronald-lists@klop.ws> Cc: Warner Losh <imp@freebsd.org>, dev-commits-src-all@freebsd.org, src-committers@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 12117d0e9314 - main - da: Update trim stats for WRITE SAME and ATA TRIM Message-ID: <CANCZdfomJiBfP1njZVLQWfaZGoA1fDqgvWMOB7EXJLk6UX_LQQ@mail.gmail.com> In-Reply-To: <1010899452.10421.1714749653185@localhost> References: <202405031447.443ElgZj051266@gitrepo.freebsd.org> <1010899452.10421.1714749653185@localhost>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Fri, May 3, 2024 at 9:21 AM Ronald Klop <ronald-lists@klop.ws> wrote: > Are you sure you used the right PR? > > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277637 > "[NEW PORT] security/enc: Modern and friendly CLI alternative to GnuPG" > Doh! Should have used 277673. </doh!> Warner > > > Regards, > Ronald. > > > > *Van:* Warner Losh <imp@FreeBSD.org> > *Datum:* vrijdag, 3 mei 2024 16:47 > *Aan:* src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, > dev-commits-src-main@FreeBSD.org > *Onderwerp:* git: 12117d0e9314 - main - da: Update trim stats for WRITE > SAME and ATA TRIM > > The branch main has been updated by imp: > > URL: > https://cgit.FreeBSD.org/src/commit/?id=12117d0e9314d1706508386c1c49ace2de578b31 > > commit 12117d0e9314d1706508386c1c49ace2de578b31 > Author: Matthew Grooms <mgrooms@shrew.net> > AuthorDate: 2024-05-03 14:41:05 +0000 > Commit: Warner Losh <imp@FreeBSD.org> > CommitDate: 2024-05-03 14:46:59 +0000 > > da: Update trim stats for WRITE SAME and ATA TRIM > > The scsi UNMAP path updated trim stats in the da sysctl, but the ATA > TRIM passthru and WRITE SAME paths did not. Add code so they do. > > PR: 277637 > Reviewed by: imp (tweaked WS path to update ranges) > --- > sys/cam/scsi/scsi_da.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/sys/cam/scsi/scsi_da.c b/sys/cam/scsi/scsi_da.c > index 9b3d706d6168..0daaff9229b0 100644 > --- a/sys/cam/scsi/scsi_da.c > +++ b/sys/cam/scsi/scsi_da.c > @@ -4208,6 +4208,9 @@ da_delete_trim(struct cam_periph *periph, union ccb > *ccb, struct bio *bp) > da_default_timeout * 1000); > ccb->ccb_h.ccb_state = DA_CCB_DELETE; > ccb->ccb_h.flags |= CAM_UNLOCKED; > + softc->trim_count++; > + softc->trim_ranges += ranges; > + softc->trim_lbas += block_count; > cam_iosched_submit_trim(softc->cam_iosched); > } > > @@ -4268,6 +4271,9 @@ da_delete_ws(struct cam_periph *periph, union ccb > *ccb, struct bio *bp) > da_default_timeout * 1000); > ccb->ccb_h.ccb_state = DA_CCB_DELETE; > ccb->ccb_h.flags |= CAM_UNLOCKED; > + softc->trim_count++; > + softc->trim_ranges++; > + softc->trim_lbas += count; > cam_iosched_submit_trim(softc->cam_iosched); > } > > > ------------------------------ > > > > [-- Attachment #2 --] <div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 3, 2024 at 9:21 AM Ronald Klop <<a href="mailto:ronald-lists@klop.ws">ronald-lists@klop.ws</a>> 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"><div>Are you sure you used the right PR?<br> <br> <a href="https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277637" target="_blank">https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277637</a><br>; "<span id="m_-1572156723909711205summary_container"><span id="m_-1572156723909711205short_desc_nonedit_display">[NEW PORT] security/enc: Modern and friendly CLI alternative to GnuPG"<br></span></span></div></blockquote><div><br></div><div>Doh! Should have used 277673. </doh!></div><div><br></div><div>Warner <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span id="m_-1572156723909711205summary_container"><span id="m_-1572156723909711205short_desc_nonedit_display"> <br> <br> Regards,<br> Ronald.</span></span><br> <br> <p><b>Van:</b> Warner Losh <imp@FreeBSD.org><br> <b>Datum:</b> vrijdag, 3 mei 2024 16:47<br> <b>Aan:</b> src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org<br> <b>Onderwerp:</b> git: 12117d0e9314 - main - da: Update trim stats for WRITE SAME and ATA TRIM</p> <blockquote style="padding-right:0px;padding-left:5px;margin-left:5px;border-left:2px solid rgb(0,0,0);margin-right:0px"> <div id="m_-1572156723909711205P"> <div id="m_-1572156723909711205P.P">The branch main has been updated by imp:<br> <br> URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=12117d0e9314d1706508386c1c49ace2de578b31" target="_blank">https://cgit.FreeBSD.org/src/commit/?id=12117d0e9314d1706508386c1c49ace2de578b31</a><br>; <br> commit 12117d0e9314d1706508386c1c49ace2de578b31<br> Author: Matthew Grooms <<a href="mailto:mgrooms@shrew.net" target="_blank">mgrooms@shrew.net</a>><br> AuthorDate: 2024-05-03 14:41:05 +0000<br> Commit: Warner Losh <imp@FreeBSD.org><br> CommitDate: 2024-05-03 14:46:59 +0000<br> <br> da: Update trim stats for WRITE SAME and ATA TRIM<br> <br> The scsi UNMAP path updated trim stats in the da sysctl, but the ATA<br> TRIM passthru and WRITE SAME paths did not. Add code so they do.<br> <br> PR: 277637<br> Reviewed by: imp (tweaked WS path to update ranges)<br> ---<br> sys/cam/scsi/scsi_da.c | 6 ++++++<br> 1 file changed, 6 insertions(+)<br> <br> diff --git a/sys/cam/scsi/scsi_da.c b/sys/cam/scsi/scsi_da.c<br> index 9b3d706d6168..0daaff9229b0 100644<br> --- a/sys/cam/scsi/scsi_da.c<br> +++ b/sys/cam/scsi/scsi_da.c<br> @@ -4208,6 +4208,9 @@ da_delete_trim(struct cam_periph *periph, union ccb *ccb, struct bio *bp)<br> da_default_timeout * 1000);<br> ccb->ccb_h.ccb_state = DA_CCB_DELETE;<br> ccb->ccb_h.flags |= CAM_UNLOCKED;<br> + softc->trim_count++;<br> + softc->trim_ranges += ranges;<br> + softc->trim_lbas += block_count;<br> cam_iosched_submit_trim(softc->cam_iosched);<br> }<br> <br> @@ -4268,6 +4271,9 @@ da_delete_ws(struct cam_periph *periph, union ccb *ccb, struct bio *bp)<br> da_default_timeout * 1000);<br> ccb->ccb_h.ccb_state = DA_CCB_DELETE;<br> ccb->ccb_h.flags |= CAM_UNLOCKED;<br> + softc->trim_count++;<br> + softc->trim_ranges++;<br> + softc->trim_lbas += count;<br> cam_iosched_submit_trim(softc->cam_iosched);<br> }<br> <br> </div> <hr></div> </blockquote> <br> </div></blockquote></div></div>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfomJiBfP1njZVLQWfaZGoA1fDqgvWMOB7EXJLk6UX_LQQ>