/*
* Copyright (C) 2001, 2002 Sistina Software (UK) Limited.
- * Copyright (C) 2004 Red Hat, Inc. All rights reserved.
*
* This file is released under the GPL.
*/
#include <asm/uaccess.h>
-#define DM_DRIVER_EMAIL "dm-devel@redhat.com"
+#define DM_DRIVER_EMAIL "dm@uk.sistina.com"
/*-----------------------------------------------------------------
* The ioctl interface needs to be able to look up devices by
}
register_with_devfs(cell);
dm_get(md);
- dm_set_mdptr(md, cell);
up_write(&_hash_lock);
return 0;
list_del(&hc->uuid_list);
list_del(&hc->name_list);
unregister_with_devfs(hc);
- dm_set_mdptr(hc->md, NULL);
dm_put(hc->md);
if (hc->new_map)
dm_table_put(hc->new_map);
}
/*
- * Always use UUID for lookups if it's present, otherwise use name or dev.
+ * Always use UUID for lookups if it's present, otherwise use name.
*/
static inline struct hash_cell *__find_device_hash_cell(struct dm_ioctl *param)
{
- if (*param->uuid)
- return __get_uuid_cell(param->uuid);
- else if (*param->name)
- return __get_name_cell(param->name);
- else
- return dm_get_mdptr(huge_decode_dev(param->dev));
+ return *param->uuid ?
+ __get_uuid_cell(param->uuid) : __get_name_cell(param->name);
}
static inline struct mapped_device *find_device(struct dm_ioctl *param)
hc = __find_device_hash_cell(param);
if (hc) {
md = hc->md;
- dm_get(md);
/*
* Sneakily write in both the name and the uuid
param->flags |= DM_INACTIVE_PRESENT_FLAG;
else
param->flags &= ~DM_INACTIVE_PRESENT_FLAG;
+
+ dm_get(md);
}
up_read(&_hash_lock);
return r;
}
-/*
- * Pass a message to the target that's at the supplied device offset.
- */
-static int target_message(struct dm_ioctl *param, size_t param_size)
-{
- int r, argc;
- char **argv;
- struct mapped_device *md;
- struct dm_table *table;
- struct dm_target *ti;
- struct dm_target_msg *tmsg = (void *) param + param->data_start;
-
- md = find_device(param);
- if (!md)
- return -ENXIO;
-
- r = __dev_status(md, param);
- if (r)
- goto out;
-
- if (tmsg < (struct dm_target_msg *) (param + 1) ||
- invalid_str(tmsg->message, (void *) param + param_size)) {
- DMWARN("Invalid target message parameters.");
- r = -EINVAL;
- goto out;
- }
-
- r = dm_split_args(&argc, &argv, tmsg->message);
- if (r) {
- DMWARN("Failed to split target message parameters");
- goto out;
- }
-
- table = dm_get_table(md);
- if (!table)
- goto out_argv;
-
- if (tmsg->sector >= dm_table_get_size(table)) {
- DMWARN("Target message sector outside device.");
- r = -EINVAL;
- goto out_table;
- }
-
- ti = dm_table_find_target(table, tmsg->sector);
- if (ti->type->message)
- r = ti->type->message(ti, argc, argv);
- else {
- DMWARN("Target type does not support messages");
- r = -EINVAL;
- }
-
- out_table:
- dm_table_put(table);
- out_argv:
- kfree(argv);
- out:
- param->data_size = 0;
- dm_put(md);
- return r;
-}
-
/*-----------------------------------------------------------------
* Implementation of open/close/ioctl on the special char
* device.
{DM_TABLE_DEPS_CMD, table_deps},
{DM_TABLE_STATUS_CMD, table_status},
- {DM_LIST_VERSIONS_CMD, list_versions},
-
- {DM_TARGET_MSG_CMD, target_message}
+ {DM_LIST_VERSIONS_CMD, list_versions}
};
return (cmd >= ARRAY_SIZE(_ioctls)) ? NULL : _ioctls[cmd].fn;
cmd == DM_LIST_VERSIONS_CMD)
return 0;
- if ((cmd == DM_DEV_CREATE_CMD)) {
- if (!*param->name) {
- DMWARN("name not supplied when creating device");
+ /* Unless creating, either name or uuid but not both */
+ if (cmd != DM_DEV_CREATE_CMD) {
+ if ((!*param->uuid && !*param->name) ||
+ (*param->uuid && *param->name)) {
+ DMWARN("one of name or uuid must be supplied, cmd(%u)",
+ cmd);
return -EINVAL;
}
- } else if ((*param->uuid && *param->name)) {
- DMWARN("only supply one of name or uuid, cmd(%u)", cmd);
- return -EINVAL;
}
/* Ensure strings are terminated */