shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-01-02 01:18:26 +03:00
Code Releases Activity

libdm: print params only for ioctls using them

Browse Source 
When preparing ioctl buffer and flatting all parameters,
add table parameters only to ioctl that do process them.

Note: list of ioctl should be kept in sync with kernel code.
This commit is contained in:
Zdenek Kabelac 2018-11-16 15:57:15 +01:00
parent 1ae5bf2b83
commit 43f8da7699
3 changed files with 59 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
WHATS_NEW_DM
View File

@ -1,5 +1,6 @@
Version 1.02.155 - Version 1.02.155 -
==================================== ====================================
Enhance ioctl flattening and add parameters only when needed.
Add DM_DEVICE_ARM_POLL for API completness matching kernel. Add DM_DEVICE_ARM_POLL for API completness matching kernel.
Do not add parameters for RESUME with DM_DEVICE_CREATE dm task. Do not add parameters for RESUME with DM_DEVICE_CREATE dm task.

21
device_mapper/ioctl/libdm-iface.c
View File

@ -1085,6 +1085,22 @@ static int _lookup_dev_name(uint64_t dev, char *buf, size_t len)
return r; return r;
} }
static int _add_params(int type)
{
switch (type) {
case DM_DEVICE_REMOVE_ALL:
case DM_DEVICE_CREATE:
case DM_DEVICE_REMOVE:
case DM_DEVICE_SUSPEND:
case DM_DEVICE_STATUS:
case DM_DEVICE_CLEAR:
case DM_DEVICE_ARM_POLL:
return 0; /* IOCTL_FLAGS_NO_PARAMS in drivers/md/dm-ioctl.c */
default:
return 1;
}
}
static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count) static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
{ {
const size_t min_size = 16 * 1024; const size_t min_size = 16 * 1024;
@ -1097,11 +1113,15 @@ static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
char *b, *e; char *b, *e;
int count = 0; int count = 0;
if (_add_params(dmt->type))
for (t = dmt->head; t; t = t->next) { for (t = dmt->head; t; t = t->next) {
len += sizeof(struct dm_target_spec); len += sizeof(struct dm_target_spec);
len += strlen(t->params) + 1 + ALIGNMENT; len += strlen(t->params) + 1 + ALIGNMENT;
count++; count++;
} }
else if (dmt->head)
log_debug_activation(INTERNAL_ERROR "dm '%s' ioctl should not define parameters.",
_cmd_data_v4[dmt->type].name);
if (count && (dmt->sector || dmt->message)) { if (count && (dmt->sector || dmt->message)) {
log_error("targets and message are incompatible"); log_error("targets and message are incompatible");
@ -1251,6 +1271,7 @@ static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
b = (char *) (dmi + 1); b = (char *) (dmi + 1);
e = (char *) dmi + len; e = (char *) dmi + len;
if (_add_params(dmt->type))
for (t = dmt->head; t; t = t->next) for (t = dmt->head; t; t = t->next)
if (!(b = _add_target(t, b, e))) if (!(b = _add_target(t, b, e)))
goto_bad; goto_bad;

21
libdm/ioctl/libdm-iface.c
View File

@ -1097,6 +1097,22 @@ static int _lookup_dev_name(uint64_t dev, char *buf, size_t len)
return r; return r;
} }
static int _add_params(int type)
{
switch (type) {
case DM_DEVICE_REMOVE_ALL:
case DM_DEVICE_CREATE:
case DM_DEVICE_REMOVE:
case DM_DEVICE_SUSPEND:
case DM_DEVICE_STATUS:
case DM_DEVICE_CLEAR:
case DM_DEVICE_ARM_POLL:
return 0; /* IOCTL_FLAGS_NO_PARAMS in drivers/md/dm-ioctl.c */
default:
return 1;
}
}
static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count) static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
{ {
const size_t min_size = 16 * 1024; const size_t min_size = 16 * 1024;
@ -1109,11 +1125,15 @@ static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
char *b, *e; char *b, *e;
int count = 0; int count = 0;
if (_add_params(dmt->type))
for (t = dmt->head; t; t = t->next) { for (t = dmt->head; t; t = t->next) {
len += sizeof(struct dm_target_spec); len += sizeof(struct dm_target_spec);
len += strlen(t->params) + 1 + ALIGNMENT; len += strlen(t->params) + 1 + ALIGNMENT;
count++; count++;
} }
else if (dmt->head)
log_debug_activation(INTERNAL_ERROR "dm '%s' ioctl should not define parameters.",
_cmd_data_v4[dmt->type].name);
if (count && (dmt->sector || dmt->message)) { if (count && (dmt->sector || dmt->message)) {
log_error("targets and message are incompatible"); log_error("targets and message are incompatible");
@ -1263,6 +1283,7 @@ static struct dm_ioctl *_flatten(struct dm_task *dmt, unsigned repeat_count)
b = (char *) (dmi + 1); b = (char *) (dmi + 1);
e = (char *) dmi + len; e = (char *) dmi + len;
if (_add_params(dmt->type))
for (t = dmt->head; t; t = t->next) for (t = dmt->head; t; t = t->next)
if (!(b = _add_target(t, b, e))) if (!(b = _add_target(t, b, e)))
goto_bad; goto_bad;