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

metadata: Use a consistent format for callback fn parameters

Browse Source
This commit is contained in:
Alasdair G Kergon 2018-01-05 14:24:10 +00:00
parent a0ddfad94b
commit 946f07af3e
7 changed files with 38 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/config/config.c
View File

@ -503,8 +503,10 @@ struct process_config_file_params {
int ret;
};
static void _process_config_file_buffer(struct process_config_file_params *pcfp, char *buffer)
static void _process_config_file_buffer(int failed, void *context, void *data)
{
struct process_config_file_params *pcfp = context;
char *buffer = data;
char *fb, *fe;
fb = buffer;
@ -598,7 +600,7 @@ int config_file_read_fd(struct dm_pool *mem, struct dm_config_tree *cft, struct
fb = buf;
}
_process_config_file_buffer(pcfp, fb);
_process_config_file_buffer(0, pcfp, fb);
r = pcfp->ret;
out:

5
lib/device/device.h
View File

@ -32,6 +32,11 @@
#define DEV_ASSUMED_FOR_LV 0x00000200 /* Is device assumed for an LV */
#define DEV_NOT_O_NOATIME 0x00000400 /* Don't use O_NOATIME */
/*
* Standard format for callback functions
*/
typedef void (*lvm_callback_fn_t)(int failed, void *context, void *data);
/*
* Support for external device info.
* Any new external device info source needs to be

16
lib/format_text/format-text.c
View File

@ -332,8 +332,9 @@ struct process_raw_mda_header_params {
int ret;
};
static void _process_raw_mda_header(struct process_raw_mda_header_params *prmp)
static void _process_raw_mda_header(int failed, void *context, void *data)
{
struct process_raw_mda_header_params *prmp = context;
struct mda_header *mdah = prmp->mdah;
struct device_area *dev_area = &prmp->dev_area;
@ -413,7 +414,7 @@ static struct mda_header *_raw_read_mda_header(struct dm_pool *mem, struct devic
return_NULL;
}
_process_raw_mda_header(prmp);
_process_raw_mda_header(0, prmp, NULL);
if (!prmp->ret)
return_NULL;
@ -1347,8 +1348,9 @@ struct vgname_from_mda_params{
int ret;
};
static void _vgname_from_mda_process(struct vgname_from_mda_params *vfmp)
static void _vgname_from_mda_process(int failed, void *context, void *data)
{
struct vgname_from_mda_params *vfmp = context;
struct mda_header *mdah = vfmp->mdah;
struct device_area *dev_area = vfmp->dev_area;
struct lvmcache_vgsummary *vgsummary = vfmp->vgsummary;
@ -1386,8 +1388,10 @@ out:
;
}
static void _vgname_from_mda_validate(struct vgname_from_mda_params *vfmp, char *buffer)
static void _vgname_from_mda_validate(int failed, void *context, void *data)
{
struct vgname_from_mda_params *vfmp = context;
char *buffer = data;
const struct format_type *fmt = vfmp->fmt;
struct mda_header *mdah = vfmp->mdah;
struct device_area *dev_area = vfmp->dev_area;
@ -1439,7 +1443,7 @@ static void _vgname_from_mda_validate(struct vgname_from_mda_params *vfmp, char
goto_out;
}
_vgname_from_mda_process(vfmp);
_vgname_from_mda_process(0, vfmp, NULL);
out:
;
@ -1492,7 +1496,7 @@ int vgname_from_mda(const struct format_type *fmt,
NAME_LEN, MDA_CONTENT_REASON(primary_mda), buf))
return_0;
_vgname_from_mda_validate(vfmp, buf);
_vgname_from_mda_validate(0, vfmp, buf);
return vfmp->ret;
}

10
lib/format_text/import.c
View File

@ -41,8 +41,9 @@ struct import_vgsummary_params {
int ret;
};
static void _import_vgsummary(struct import_vgsummary_params *ivsp)
static void _import_vgsummary(int failed, void *context, void *data)
{
struct import_vgsummary_params *ivsp = context;
struct text_vg_version_ops **vsn;
if (ivsp->checksum_only)
@ -114,7 +115,7 @@ int text_vgsummary_import(const struct format_type *fmt,
return 0;
}
_import_vgsummary(ivsp);
_import_vgsummary(0, ivsp, NULL);
return ivsp->ret;
}
@ -138,8 +139,9 @@ struct import_vg_params {
char **desc;
};
static void _import_vg(struct import_vg_params *ivp)
static void _import_vg(int failed, void *context, void *data)
{
struct import_vg_params *ivp = context;
struct text_vg_version_ops **vsn;
ivp->vg = NULL;
@ -234,7 +236,7 @@ struct volume_group *text_vg_import_fd(struct format_instance *fid,
return_NULL;
}
_import_vg(ivp);
_import_vg(0, ivp, NULL);
return ivp->vg;
}

13
lib/format_text/text_label.c
View File

@ -333,8 +333,11 @@ struct process_mda_header_params {
int ret;
};
static void _process_vgsummary(struct process_mda_header_params *pmp, struct lvmcache_vgsummary *vgsummary)
static void _process_vgsummary(int failed, void *context, void *data)
{
struct process_mda_header_params *pmp = context;
struct lvmcache_vgsummary *vgsummary = data;
if (!lvmcache_update_vgname_and_id(pmp->umb->info, vgsummary)) {
pmp->umb->ret = 0;
pmp->ret = 0;
@ -346,8 +349,10 @@ out:
stack;
}
static void _process_mda_header(struct process_mda_header_params *pmp, struct mda_header *mdah)
static void _process_mda_header(int failed, void *context, void *data)
{
struct process_mda_header_params *pmp = context;
struct mda_header *mdah = data;
struct update_mda_baton *umb = pmp->umb;
const struct format_type *fmt = umb->label->labeller->fmt;
struct metadata_area *mda = pmp->mda;
@ -373,7 +378,7 @@ static void _process_mda_header(struct process_mda_header_params *pmp, struct md
return;
}
_process_vgsummary(pmp, &pmp->vgsummary);
_process_vgsummary(0, pmp, &pmp->vgsummary);
}
static int _update_mda(struct metadata_area *mda, void *baton)
@ -416,7 +421,7 @@ static int _update_mda(struct metadata_area *mda, void *baton)
return 1;
}
_process_mda_header(pmp, mdah);
_process_mda_header(0, pmp, mdah);
return pmp->ret;
}

8
lib/label/label.c
View File

@ -146,8 +146,10 @@ out:
stack;
}
static void _find_labeller(struct find_labeller_params *flp, char *readbuf)
static void _find_labeller(int failed, void *context, void *data)
{
struct find_labeller_params *flp = context;
char *readbuf = data;
struct device *dev = flp->dev;
uint64_t scan_sector = flp->scan_sector;
struct label **result = flp->result;
@ -158,8 +160,6 @@ static void _find_labeller(struct find_labeller_params *flp, char *readbuf)
struct lvmcache_info *info;
uint64_t sector;
int r = 0;
/* Scan a few sectors for a valid label */
for (sector = 0; sector < LABEL_SCAN_SECTORS;
sector += LABEL_SIZE >> SECTOR_SHIFT) {
@ -343,7 +343,7 @@ static int _label_read(struct device *dev, uint64_t scan_sector, struct label **
return 0;
}
_find_labeller(flp, readbuf);
_find_labeller(0, flp, readbuf);
return flp->ret;
}

1
lib/metadata/metadata.h
View File

@ -48,7 +48,6 @@
*/
#define dm_round_up(n, sz) (dm_div_up((n), (sz)) * (sz))
/* Various flags */
/* See metadata-exported.h for the complete list. */
/* Note that the bits no longer necessarily correspond to LVM1 disk format */