From 3bb0dc03afee5825400ea2f4cd69498b00d1b849 Mon Sep 17 00:00:00 2001 From: Dave Wysochanski Date: Tue, 6 Oct 2009 16:00:38 +0000 Subject: [PATCH] Cleanup comment and some whitespace. --- lib/metadata/metadata.c | 11 ++++++----- tools/toollib.c | 6 +++--- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/lib/metadata/metadata.c b/lib/metadata/metadata.c index 1bee7d0dc..82690a595 100644 --- a/lib/metadata/metadata.c +++ b/lib/metadata/metadata.c @@ -67,7 +67,7 @@ unsigned long set_pe_align(struct physical_volume *pv, unsigned long data_alignm if (data_alignment) pv->pe_align = data_alignment; - else + else pv->pe_align = MAX(65536UL, lvm_getpagesize()) >> SECTOR_SHIFT; if (!pv->dev) @@ -1292,18 +1292,19 @@ void fill_default_pvcreate_params(struct pvcreate_params *pp) } /* - * pvcreate_single() - initialize a device with PV label and metadata + * pvcreate_single() - initialize a device with PV label and metadata area * * Parameters: * - pv_name: device path to initialize - * - handle: options to pass to pv_create; NULL indicates use defaults + * - pp: parameters to pass to pv_create; if NULL, use default values * * Returns: * NULL: error * struct physical_volume * (non-NULL): handle to physical volume created */ -struct physical_volume * pvcreate_single(struct cmd_context *cmd, const char *pv_name, - struct pvcreate_params *pp) +struct physical_volume * pvcreate_single(struct cmd_context *cmd, + const char *pv_name, + struct pvcreate_params *pp) { void *pv; struct device *dev; diff --git a/tools/toollib.c b/tools/toollib.c index c48314fac..d14dc771d 100644 --- a/tools/toollib.c +++ b/tools/toollib.c @@ -1021,7 +1021,7 @@ static int _create_pv_entry(struct dm_pool *mem, struct pv_list *pvl, new_pvl = pvl2; break; } - + if (!new_pvl) { if (!(new_pvl = dm_pool_alloc(mem, sizeof(*new_pvl)))) { log_error("Unable to allocate physical volume list."); @@ -1239,12 +1239,12 @@ int vg_refresh_visible(struct cmd_context *cmd, struct volume_group *vg) { struct lv_list *lvl; int r = 1; - + dm_list_iterate_items(lvl, &vg->lvs) if (lv_is_visible(lvl->lv)) if (!lv_refresh(cmd, lvl->lv)) r = 0; - + return r; }