Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/dev/dm Adapt to proplib API chanages.



details:   https://anonhg.NetBSD.org/src/rev/a48303413fd3
branches:  trunk
changeset: 973711:a48303413fd3
user:      thorpej <thorpej%NetBSD.org@localhost>
date:      Wed Jul 08 15:07:13 2020 +0000

description:
Adapt to proplib API chanages.

diffstat:

 sys/dev/dm/device-mapper.c |   4 +-
 sys/dev/dm/dm_dev.c        |   6 +-
 sys/dev/dm/dm_ioctl.c      |  78 +++++++++++++++++++++++++--------------------
 sys/dev/dm/dm_target.c     |   6 +-
 4 files changed, 52 insertions(+), 42 deletions(-)

diffs (truncated from 304 to 300 lines):

diff -r 36d8b7f24f84 -r a48303413fd3 sys/dev/dm/device-mapper.c
--- a/sys/dev/dm/device-mapper.c        Wed Jul 08 13:41:53 2020 +0000
+++ b/sys/dev/dm/device-mapper.c        Wed Jul 08 15:07:13 2020 +0000
@@ -1,4 +1,4 @@
-/*        $NetBSD: device-mapper.c,v 1.60 2020/01/16 07:18:08 maya Exp $ */
+/*        $NetBSD: device-mapper.c,v 1.61 2020/07/08 15:07:13 thorpej Exp $ */
 
 /*
  * Copyright (c) 2010 The NetBSD Foundation, Inc.
@@ -402,7 +402,7 @@
                return EINVAL;
 
        for (i = 0; cmd_fn[i].cmd != NULL; i++)
-               if (prop_string_equals_cstring(command, cmd_fn[i].cmd))
+               if (prop_string_equals_string(command, cmd_fn[i].cmd))
                        break;
 
        if (!cmd_fn[i].allowed &&
diff -r 36d8b7f24f84 -r a48303413fd3 sys/dev/dm/dm_dev.c
--- a/sys/dev/dm/dm_dev.c       Wed Jul 08 13:41:53 2020 +0000
+++ b/sys/dev/dm/dm_dev.c       Wed Jul 08 15:07:13 2020 +0000
@@ -1,4 +1,4 @@
-/*        $NetBSD: dm_dev.c,v 1.17 2019/12/31 10:30:30 tkusumi Exp $      */
+/*        $NetBSD: dm_dev.c,v 1.18 2020/07/08 15:07:13 thorpej Exp $      */
 
 /*
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
@@ -29,7 +29,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dm_dev.c,v 1.17 2019/12/31 10:30:30 tkusumi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dm_dev.c,v 1.18 2020/07/08 15:07:13 thorpej Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -381,7 +381,7 @@
        TAILQ_FOREACH(dmv, &dm_dev_list, next_devlist) {
                dev_dict = prop_dictionary_create();
 
-               prop_dictionary_set_cstring(dev_dict, DM_DEV_NAME, dmv->name);
+               prop_dictionary_set_string(dev_dict, DM_DEV_NAME, dmv->name);
                prop_dictionary_set_uint32(dev_dict, DM_DEV_DEV, dmv->minor);
 
                prop_array_add(dev_array, dev_dict);
diff -r 36d8b7f24f84 -r a48303413fd3 sys/dev/dm/dm_ioctl.c
--- a/sys/dev/dm/dm_ioctl.c     Wed Jul 08 13:41:53 2020 +0000
+++ b/sys/dev/dm/dm_ioctl.c     Wed Jul 08 15:07:13 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: dm_ioctl.c,v 1.49 2019/12/23 16:17:35 tkusumi Exp $      */
+/* $NetBSD: dm_ioctl.c,v 1.50 2020/07/08 15:07:13 thorpej Exp $      */
 
 /*
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
@@ -29,7 +29,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dm_ioctl.c,v 1.49 2019/12/23 16:17:35 tkusumi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dm_ioctl.c,v 1.50 2020/07/08 15:07:13 thorpej Exp $");
 
 /*
  * Locking is used to synchronise between ioctl calls and between dm_table's
@@ -202,8 +202,8 @@
        uuid = NULL;
 
        /* Get needed values from dictionary. */
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
 
        dm_dbg_print_flags(flags);
@@ -323,8 +323,8 @@
        minor = 0;
 
        /* Get needed values from dictionary. */
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -332,7 +332,7 @@
 
        cmd_array = prop_dictionary_get(dm_dict, DM_IOCTL_CMD_DATA);
 
-       prop_array_get_cstring_nocopy(cmd_array, 0, &n_name);
+       prop_array_get_string(cmd_array, 0, &n_name);
 
        if (strlen(n_name) + 1 > DM_NAME_LEN)
                return EINVAL;
@@ -351,7 +351,7 @@
 
        prop_dictionary_set_uint32(dm_dict, DM_IOCTL_OPEN, dmv->table_head.io_cnt);
        prop_dictionary_set_uint32(dm_dict, DM_IOCTL_MINOR, dmv->minor);
-       prop_dictionary_set_cstring(dm_dict, DM_IOCTL_UUID, dmv->uuid);
+       prop_dictionary_set_string(dm_dict, DM_IOCTL_UUID, dmv->uuid);
 
        dm_dev_insert(dmv);
 
@@ -375,8 +375,8 @@
        uuid = NULL;
 
        /* Get needed values from dictionary. */
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -415,8 +415,8 @@
        uuid = NULL;
        flags = 0;
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -428,7 +428,7 @@
 
        prop_dictionary_set_uint32(dm_dict, DM_IOCTL_OPEN, dmv->table_head.io_cnt);
        prop_dictionary_set_uint32(dm_dict, DM_IOCTL_MINOR, dmv->minor);
-       prop_dictionary_set_cstring(dm_dict, DM_IOCTL_UUID, dmv->uuid);
+       prop_dictionary_set_string(dm_dict, DM_IOCTL_UUID, dmv->uuid);
 
        if (dmv->flags & DM_SUSPEND_FLAG)
                DM_ADD_FLAG(flags, DM_SUSPEND_FLAG);
@@ -469,8 +469,8 @@
        uuid = NULL;
        flags = 0;
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -514,8 +514,8 @@
         * printf("%s\n",xml);
         */
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -570,8 +570,8 @@
        flags = 0;
        minor = 0;
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -615,8 +615,8 @@
        uuid = NULL;
        flags = 0;
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -628,8 +628,8 @@
                return ENOENT;
        }
        prop_dictionary_set_uint32(dm_dict, DM_IOCTL_MINOR, dmv->minor);
-       prop_dictionary_set_cstring(dm_dict, DM_IOCTL_NAME, dmv->name);
-       prop_dictionary_set_cstring(dm_dict, DM_IOCTL_UUID, dmv->uuid);
+       prop_dictionary_set_string(dm_dict, DM_IOCTL_NAME, dmv->name);
+       prop_dictionary_set_string(dm_dict, DM_IOCTL_UUID, dmv->uuid);
 
        aprint_debug("Getting table deps for device: %s\n", dmv->name);
 
@@ -720,8 +720,8 @@
         * printf("%s\n",xml);
         */
 
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_NAME, &name);
-       prop_dictionary_get_cstring_nocopy(dm_dict, DM_IOCTL_UUID, &uuid);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_NAME, &name);
+       prop_dictionary_get_string(dm_dict, DM_IOCTL_UUID, &uuid);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_FLAGS, &flags);
        prop_dictionary_get_uint32(dm_dict, DM_IOCTL_MINOR, &minor);
 
@@ -753,9 +753,11 @@
 
        while ((target_dict = prop_object_iterator_next(iter)) != NULL) {
                int ret;
-               char *str = NULL;
+               const char *cp;
+               char *str;
+               size_t strsz;
 
-               prop_dictionary_get_cstring_nocopy(target_dict,
+               prop_dictionary_get_string(target_dict,
                    DM_TABLE_TYPE, &type);
                /*
                 * If we want to deny table with 2 or more different
@@ -786,8 +788,13 @@
                 * null and therefore it should be checked before we try to
                 * use it.
                 */
-               prop_dictionary_get_cstring(target_dict,
-                   DM_TABLE_PARAMS, (char **) &str);
+               cp = NULL;
+               prop_dictionary_get_string(target_dict,
+                   DM_TABLE_PARAMS, &cp);
+               if (cp == NULL)
+                       str = NULL;
+               else
+                       str = kmem_strdupsize(cp, &strsz, KM_SLEEP);
 
                if (SLIST_EMPTY(tbl) || last_table == NULL)
                        /* insert this table to head */
@@ -798,14 +805,17 @@
                if ((ret = dm_table_init(target, table_en, str)) != 0) {
                        dm_table_release(&dmv->table_head, DM_TABLE_INACTIVE);
                        dm_table_destroy(&dmv->table_head, DM_TABLE_INACTIVE);
-                       free(str, M_TEMP);
+
+                       if (str != NULL)
+                               kmem_free(str, strsz);
 
                        dm_dev_unbusy(dmv);
                        prop_object_iterator_release(iter);
                        return ret;
                }
                last_table = table_en;
-               free(str, M_TEMP);
+               if (str != NULL)
+                       kmem_free(str, strsz);
        }
        prop_object_iterator_release(iter);
 
@@ -948,7 +958,7 @@
                prop_dictionary_set_uint64(target_dict, DM_TABLE_LENGTH,
                    table_en->length);
 
-               prop_dictionary_set_cstring(target_dict, DM_TABLE_TYPE,
+               prop_dictionary_set_string(target_dict, DM_TABLE_TYPE,
                    table_en->target->name);
 
                /* dm_table_get_cur_actv.table ?? */
@@ -959,7 +969,7 @@
                 * Explicitly clear DM_TABLE_PARAMS to prevent dmsetup(8) from
                 * printing junk when DM_TABLE_PARAMS was never initialized.
                 */
-               prop_dictionary_set_cstring(target_dict, DM_TABLE_PARAMS, "");
+               prop_dictionary_set_string(target_dict, DM_TABLE_PARAMS, "");
 
                is_table = (flags & DM_STATUS_TABLE_FLAG) ? 1 : 0;
                if (is_table && table_en->target->table)
@@ -972,7 +982,7 @@
                        params = NULL;
 
                if (params != NULL) {
-                       prop_dictionary_set_cstring(target_dict,
+                       prop_dictionary_set_string(target_dict,
                            DM_TABLE_PARAMS, params);
                        kmem_free(params, DM_MAX_PARAMS_SIZE);
                }
diff -r 36d8b7f24f84 -r a48303413fd3 sys/dev/dm/dm_target.c
--- a/sys/dev/dm/dm_target.c    Wed Jul 08 13:41:53 2020 +0000
+++ b/sys/dev/dm/dm_target.c    Wed Jul 08 15:07:13 2020 +0000
@@ -1,4 +1,4 @@
-/*        $NetBSD: dm_target.c,v 1.40 2020/01/21 16:27:53 tkusumi Exp $      */
+/*        $NetBSD: dm_target.c,v 1.41 2020/07/08 15:07:13 thorpej Exp $      */
 
 /*
  * Copyright (c) 2008 The NetBSD Foundation, Inc.
@@ -29,7 +29,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dm_target.c,v 1.40 2020/01/21 16:27:53 tkusumi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dm_target.c,v 1.41 2020/07/08 15:07:13 thorpej Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -285,7 +285,7 @@
 
                target_dict = prop_dictionary_create();
                ver = prop_array_create();
-               prop_dictionary_set_cstring(target_dict, DM_TARGETS_NAME,



Home | Main Index | Thread Index | Old Index