From fe3ed2b871ce732af83efa6ac63c65e861890f3c Mon Sep 17 00:00:00 2001 From: Alasdair Kergon Date: Thu, 25 Mar 2010 02:40:09 +0000 Subject: [PATCH] improve a few comments in last check-in --- lib/metadata/lv_manip.c | 2 +- lib/metadata/pv_map.h | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/metadata/lv_manip.c b/lib/metadata/lv_manip.c index e41e3279b..008a2cf7e 100644 --- a/lib/metadata/lv_manip.c +++ b/lib/metadata/lv_manip.c @@ -929,7 +929,7 @@ static int _is_condition(struct cmd_context *cmd __attribute((unused)), return 1; /* - * Only used for cling and contiguous policies so its safe to say all + * Only used for cling and contiguous policies so it's safe to say all * the available space is used. */ pvmatch->areas[s].pva = pvmatch->pva; diff --git a/lib/metadata/pv_map.h b/lib/metadata/pv_map.h index 6d6cfff23..bc5919252 100644 --- a/lib/metadata/pv_map.h +++ b/lib/metadata/pv_map.h @@ -39,7 +39,9 @@ struct pv_area { * an allocation attempt, track the maximum number of extents that may * need to be used as a particular parallel area. Several of these * structs may reference the same pv_area, but 'used' may differ between - * them. + * them. The sum of all the 'used' variables referring to the same + * pv_area may not exceed that area's count, so we cannot allocate the + * same extents twice. */ struct pv_area_used { struct pv_area *pva; -- 2.43.5