spa: Let spa_taskq_param_get()'s addition of a newline be optional

For FreeBSD sysctls, we don't want the extra newline, since the
sysctl(8) utility will format strings appropriately.

Reviewed-by: Rob Norris <robn@despairlabs.com>
Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reported-by: Peter Holm <pho@FreeBSD.org>
Signed-off-by: Mark Johnston <markj@FreeBSD.org>
Closes #15719

diff --git a/module/zfs/spa.c b/module/zfs/spa.c
index dc8a99e7bd25688c2762c5a82aa9478fae007193..b144d065293077e830eaef5684eafba1397e7f6b 100644 (file)
--- a/module/zfs/spa.c
+++ b/module/zfs/spa.c
@@ -1392,7 +1392,7 @@ spa_taskq_param_set(zio_type_t t, char *cfg)
 }
 
 static int
-spa_taskq_param_get(zio_type_t t, char *buf)
+spa_taskq_param_get(zio_type_t t, char *buf, boolean_t add_newline)
 {
        int pos = 0;
 
@@ -1410,7 +1410,8 @@ spa_taskq_param_get(zio_type_t t, char *buf)
                sep = " ";
        }
 
-       buf[pos++] = '\n';
+       if (add_newline)
+               buf[pos++] = '\n';
        buf[pos] = '\0';
 
        return (pos);
@@ -1428,7 +1429,7 @@ spa_taskq_read_param_set(const char *val, zfs_kernel_param_t *kp)
 static int
 spa_taskq_read_param_get(char *buf, zfs_kernel_param_t *kp)
 {
-       return (spa_taskq_param_get(ZIO_TYPE_READ, buf));
+       return (spa_taskq_param_get(ZIO_TYPE_READ, buf, TRUE));
 }
 
 static int
@@ -1442,7 +1443,7 @@ spa_taskq_write_param_set(const char *val, zfs_kernel_param_t *kp)
 static int
 spa_taskq_write_param_get(char *buf, zfs_kernel_param_t *kp)
 {
-       return (spa_taskq_param_get(ZIO_TYPE_WRITE, buf));
+       return (spa_taskq_param_get(ZIO_TYPE_WRITE, buf, TRUE));
 }
 #else
 /*
@@ -1457,7 +1458,7 @@ spa_taskq_read_param(ZFS_MODULE_PARAM_ARGS)
        char buf[SPA_TASKQ_PARAM_MAX];
        int err;
 
-       (void) spa_taskq_param_get(ZIO_TYPE_READ, buf);
+       (void) spa_taskq_param_get(ZIO_TYPE_READ, buf, FALSE);
        err = sysctl_handle_string(oidp, buf, sizeof (buf), req);
        if (err || req->newptr == NULL)
                return (err);
@@ -1470,7 +1471,7 @@ spa_taskq_write_param(ZFS_MODULE_PARAM_ARGS)
        char buf[SPA_TASKQ_PARAM_MAX];
        int err;
 
-       (void) spa_taskq_param_get(ZIO_TYPE_WRITE, buf);
+       (void) spa_taskq_param_get(ZIO_TYPE_WRITE, buf, FALSE);
        err = sysctl_handle_string(oidp, buf, sizeof (buf), req);
        if (err || req->newptr == NULL)
                return (err);