[lvm-devel] LVM2 liblvm/lvm2app.h liblvm/lvm_lv.c test/api ...
wysochanski at sourceware.org
wysochanski at sourceware.org
Thu Aug 13 12:17:32 UTC 2009
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: wysochanski at sourceware.org 2009-08-13 12:17:32
Modified files:
liblvm : lvm2app.h lvm_lv.c
test/api : test.c
Log message:
Make lvm2app lv_t handle definition consistent with lvm_t.
This patch update lv_t handle to be consistent with lvm_t - define as a pointer
to internal struct logical_volume.
Author: Dave Wysochanski <dwysocha at redhat.com>
Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/liblvm/lvm2app.h.diff?cvsroot=lvm2&r1=1.4&r2=1.5
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/liblvm/lvm_lv.c.diff?cvsroot=lvm2&r1=1.15&r2=1.16
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/api/test.c.diff?cvsroot=lvm2&r1=1.26&r2=1.27
--- LVM2/liblvm/lvm2app.h 2009/08/13 12:16:45 1.4
+++ LVM2/liblvm/lvm2app.h 2009/08/13 12:17:32 1.5
@@ -121,7 +121,7 @@
* group. Changes will be written to disk when the volume group gets
* committed to disk.
*/
-typedef struct logical_volume lv_t;
+typedef struct logical_volume *lv_t;
/**
* Physical volume object.
@@ -139,7 +139,7 @@
*/
typedef struct lvm_lv_list {
struct dm_list list;
- lv_t *lv;
+ lv_t lv;
} lv_list_t;
/**
@@ -623,7 +623,7 @@
* non-NULL handle to an LV object created, or NULL if creation fails.
*
*/
-lv_t *lvm_vg_create_lv_linear(vg_t vg, const char *name, uint64_t size);
+lv_t lvm_vg_create_lv_linear(vg_t vg, const char *name, uint64_t size);
/**
* Activate a logical volume.
@@ -639,7 +639,7 @@
* \return
* 0 (success) or -1 (failure).
*/
-int lvm_lv_activate(lv_t *lv);
+int lvm_lv_activate(lv_t lv);
/**
* Deactivate a logical volume.
@@ -652,7 +652,7 @@
* \return
* 0 (success) or -1 (failure).
*/
-int lvm_lv_deactivate(lv_t *lv);
+int lvm_lv_deactivate(lv_t lv);
/**
* Remove a logical volume from a volume group.
@@ -669,7 +669,7 @@
* \return
* 0 (success) or -1 (failure).
*/
-int lvm_vg_remove_lv(lv_t *lv);
+int lvm_vg_remove_lv(lv_t lv);
/**
* Get the current name of a logical volume.
@@ -683,7 +683,7 @@
* \return
* Copy of the uuid string.
*/
-char *lvm_lv_get_uuid(const lv_t *lv);
+char *lvm_lv_get_uuid(const lv_t lv);
/**
* Get the current uuid of a logical volume.
@@ -697,7 +697,7 @@
* \return
* Copy of the name.
*/
-char *lvm_lv_get_name(const lv_t *lv);
+char *lvm_lv_get_name(const lv_t lv);
/**
* Get the current size in bytes of a logical volume.
@@ -708,7 +708,7 @@
* \return
* Size in bytes.
*/
-uint64_t lvm_lv_get_size(const lv_t *lv);
+uint64_t lvm_lv_get_size(const lv_t lv);
/**
* Get the current activation state of a logical volume.
@@ -719,7 +719,7 @@
* \return
* 1 if the LV is active in the kernel, 0 if not
*/
-uint64_t lvm_lv_is_active(const lv_t *lv);
+uint64_t lvm_lv_is_active(const lv_t lv);
/**
* Get the current suspended state of a logical volume.
@@ -730,7 +730,7 @@
* \return
* 1 if the LV is suspended in the kernel, 0 if not
*/
-uint64_t lvm_lv_is_suspended(const lv_t *lv);
+uint64_t lvm_lv_is_suspended(const lv_t lv);
/**
* Resize logical volume to new_size bytes.
@@ -747,7 +747,7 @@
* 0 (success) or -1 (failure).
*
*/
-int lvm_lv_resize(const lv_t *lv, uint64_t new_size);
+int lvm_lv_resize(const lv_t lv, uint64_t new_size);
/************************** physical volume handling ************************/
--- LVM2/liblvm/lvm_lv.c 2009/08/13 12:16:45 1.15
+++ LVM2/liblvm/lvm_lv.c 2009/08/13 12:17:32 1.16
@@ -24,12 +24,12 @@
#include <string.h>
/* FIXME: have lib/report/report.c _disp function call lv_size()? */
-uint64_t lvm_lv_get_size(const lv_t *lv)
+uint64_t lvm_lv_get_size(const lv_t lv)
{
return lv_size(lv);
}
-char *lvm_lv_get_uuid(const lv_t *lv)
+char *lvm_lv_get_uuid(const lv_t lv)
{
char uuid[64] __attribute((aligned(8)));
@@ -40,7 +40,7 @@
return strndup((const char *)uuid, 64);
}
-char *lvm_lv_get_name(const lv_t *lv)
+char *lvm_lv_get_name(const lv_t lv)
{
char *name;
@@ -50,7 +50,7 @@
return name;
}
-uint64_t lvm_lv_is_active(const lv_t *lv)
+uint64_t lvm_lv_is_active(const lv_t lv)
{
struct lvinfo info;
if (lv_info(lv->vg->cmd, lv, &info, 1, 0) &&
@@ -59,7 +59,7 @@
return 0;
}
-uint64_t lvm_lv_is_suspended(const lv_t *lv)
+uint64_t lvm_lv_is_suspended(const lv_t lv)
{
struct lvinfo info;
if (lv_info(lv->vg->cmd, lv, &info, 1, 0) &&
@@ -101,7 +101,7 @@
* lvm_vg_write. However, this appears to be non-trivial change until
* lv_create_single is refactored by segtype.
*/
-lv_t *lvm_vg_create_lv_linear(vg_t vg, const char *name, uint64_t size)
+lv_t lvm_vg_create_lv_linear(vg_t vg, const char *name, uint64_t size)
{
struct lvcreate_params lp;
uint64_t extents;
@@ -120,14 +120,14 @@
lvl = find_lv_in_vg(vg, name);
if (!lvl)
return NULL;
- return (lv_t *) lvl->lv;
+ return (lv_t) lvl->lv;
}
/*
* FIXME: This function should probably not commit to disk but require calling
* lvm_vg_write.
*/
-int lvm_vg_remove_lv(lv_t *lv)
+int lvm_vg_remove_lv(lv_t lv)
{
if (!lv || !lv->vg || vg_read_error(lv->vg))
return -1;
@@ -138,7 +138,7 @@
return 0;
}
-int lvm_lv_activate(lv_t *lv)
+int lvm_lv_activate(lv_t lv)
{
if (!lv || !lv->vg || vg_read_error(lv->vg) || !lv->vg->cmd)
return -1;
@@ -173,7 +173,7 @@
return 0;
}
-int lvm_lv_deactivate(lv_t *lv)
+int lvm_lv_deactivate(lv_t lv)
{
if (!lv || !lv->vg || vg_read_error(lv->vg) || !lv->vg->cmd)
return -1;
@@ -186,7 +186,7 @@
return 0;
}
-int lvm_lv_resize(const lv_t *lv, uint64_t new_size)
+int lvm_lv_resize(const lv_t lv, uint64_t new_size)
{
/* FIXME: add lv resize code here */
log_error("NOT IMPLEMENTED YET");
--- LVM2/test/api/test.c 2009/08/13 12:16:45 1.26
+++ LVM2/test/api/test.c 2009/08/13 12:17:32 1.27
@@ -136,9 +136,9 @@
}
/* FIXME: this should be per vg */
-static lv_t *_lookup_lv_by_name(const char *name)
+static lv_t _lookup_lv_by_name(const char *name)
{
- lv_t *lv;
+ lv_t lv;
if (!name) {
printf ("Invalid LV name\n");
@@ -516,7 +516,7 @@
static void _lv_deactivate(char **argv, int argc)
{
- lv_t *lv;
+ lv_t lv;
int rc=0;
if (argc < 3) {
@@ -532,7 +532,7 @@
}
static void _lv_activate(char **argv, int argc)
{
- lv_t *lv;
+ lv_t lv;
int rc=0;
if (argc < 3) {
@@ -548,7 +548,7 @@
}
static void _vg_remove_lv(char **argv, int argc)
{
- lv_t *lv;
+ lv_t lv;
if (argc < 3) {
printf("Please enter vgname, lvname\n");
@@ -569,7 +569,7 @@
static void _vg_create_lv_linear(char **argv, int argc)
{
vg_t vg;
- lv_t *lv;
+ lv_t lv;
if (argc < 4) {
printf("Please enter vgname, lvname, and size\n");
More information about the lvm-devel
mailing list