Author: scottl
Date: Mon Feb 10 00:23:20 2020
New Revision: 357710
URL: https://svnweb.freebsd.org/changeset/base/357710

Log:
  Add rudamentary support for UFS to probe whether a block device supports the
  BIO_SPEEDUP command.  Add complimentary support to the CAM periphs that
  support it.

Modified:
  head/sys/cam/ata/ata_da.c
  head/sys/cam/nvme/nvme_da.c
  head/sys/cam/scsi/scsi_da.c
  head/sys/ufs/ffs/ffs_softdep.c
  head/sys/ufs/ffs/ffs_vfsops.c

Modified: head/sys/cam/ata/ata_da.c
==============================================================================
--- head/sys/cam/ata/ata_da.c   Mon Feb 10 00:05:04 2020        (r357709)
+++ head/sys/cam/ata/ata_da.c   Mon Feb 10 00:23:20 2020        (r357710)
@@ -50,6 +50,7 @@ __FBSDID("$FreeBSD$");
 #include <sys/proc.h>
 #include <sys/reboot.h>
 #include <sys/sbuf.h>
+#include <geom/geom.h>
 #include <geom/geom_disk.h>
 #endif /* _KERNEL */
 
@@ -1565,6 +1566,10 @@ adagetattr(struct bio *bp)
 {
        int ret;
        struct cam_periph *periph;
+
+       /* TODO: tunable knob */
+       if (g_handleattr_int(bp, "GEOM::canspeedup", 1))
+               return (0);
 
        periph = (struct cam_periph *)bp->bio_disk->d_drv1;
        cam_periph_lock(periph);

Modified: head/sys/cam/nvme/nvme_da.c
==============================================================================
--- head/sys/cam/nvme/nvme_da.c Mon Feb 10 00:05:04 2020        (r357709)
+++ head/sys/cam/nvme/nvme_da.c Mon Feb 10 00:23:20 2020        (r357710)
@@ -48,6 +48,7 @@ __FBSDID("$FreeBSD$");
 #include <sys/cons.h>
 #include <sys/proc.h>
 #include <sys/reboot.h>
+#include <geom/geom.h>
 #include <geom/geom_disk.h>
 #endif /* _KERNEL */
 
@@ -699,6 +700,10 @@ ndagetattr(struct bio *bp)
 {
        int ret;
        struct cam_periph *periph;
+
+       /* TODO: tunable knob */
+       if (g_handleattr_int(bp, "GEOM::canspeedup", 1))
+               return (0);
 
        periph = (struct cam_periph *)bp->bio_disk->d_drv1;
        cam_periph_lock(periph);

Modified: head/sys/cam/scsi/scsi_da.c
==============================================================================
--- head/sys/cam/scsi/scsi_da.c Mon Feb 10 00:05:04 2020        (r357709)
+++ head/sys/cam/scsi/scsi_da.c Mon Feb 10 00:23:20 2020        (r357710)
@@ -1942,6 +1942,10 @@ dagetattr(struct bio *bp)
        int ret;
        struct cam_periph *periph;
 
+       /* TODO: tunable knob for this */
+       if (g_handleattr_int(bp, "GEOM::canspeedup", 1))
+               return (0);
+
        periph = (struct cam_periph *)bp->bio_disk->d_drv1;
        cam_periph_lock(periph);
        ret = xpt_getattr(bp->bio_data, bp->bio_length, bp->bio_attribute,

Modified: head/sys/ufs/ffs/ffs_softdep.c
==============================================================================
--- head/sys/ufs/ffs/ffs_softdep.c      Mon Feb 10 00:05:04 2020        
(r357709)
+++ head/sys/ufs/ffs/ffs_softdep.c      Mon Feb 10 00:23:20 2020        
(r357710)
@@ -1464,6 +1464,9 @@ softdep_send_speedup(struct ufsmount *ump, size_t shor
 {
        struct buf *bp;
 
+       if ((ump->um_flags & UM_CANSPEEDUP) == 0)
+               return;
+
        bp = malloc(sizeof(*bp), M_TRIM, M_WAITOK | M_ZERO);
        bp->b_iocmd = BIO_SPEEDUP;
        bp->b_ioflags = flags;

Modified: head/sys/ufs/ffs/ffs_vfsops.c
==============================================================================
--- head/sys/ufs/ffs/ffs_vfsops.c       Mon Feb 10 00:05:04 2020        
(r357709)
+++ head/sys/ufs/ffs/ffs_vfsops.c       Mon Feb 10 00:23:20 2020        
(r357710)
@@ -794,7 +794,7 @@ ffs_mountfs(devvp, mp, td)
        struct ucred *cred;
        struct g_consumer *cp;
        struct mount *nmp;
-       int candelete;
+       int candelete, canspeedup;
        off_t loc;
 
        fs = NULL;
@@ -1009,6 +1009,13 @@ ffs_mountfs(devvp, mp, td)
                        ump->um_trimhash = hashinit(MAXTRIMIO, M_TRIM,
                            &ump->um_trimlisthashsize);
                }
+       }
+
+       /* TODO: sysctl tunables, runtime modification */
+       len = sizeof(int);
+       if (g_io_getattr("GEOM::canspeedup", cp, &len, &canspeedup) == 0) {
+               if (canspeedup)
+                       ump->um_flags |= UM_CANSPEEDUP;
        }
 
        ump->um_mountp = mp;
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to