[lvm-devel] [PATCH V2 0/2] Add thin lv and thin pool creation support
Tony Asleson
tasleson at redhat.com
Thu Jan 31 21:04:59 UTC 2013
On 01/31/2013 08:25 AM, M. Mohan Kumar wrote:
> From: "M. Mohan Kumar" <mohan at in.ibm.com>
>
> Add thin lv and thin pool creation support to lvm2app.
>
> Changes from previous version:
> * Add support to specify data block size and low water mark thresold for
> newly created thin pool.
> * Added support for specifying lowwatermark parameter to thin pool
> creation in lvcreate command
OK, so the block size if specified as 0 results in a default value being
selected as before, which is what I believe we want. The default low
water mark was 0, which is what we believe disables event creation.
This doesn't appear to be a safe default behavior?
One option would be to change the type to a signed integer and change
the meaning from number of blocks free to percentage free and then treat
-1 as pick a sane default. Thus 0 would be no events and 30 would
indicate when 30% blocks free is remaining to generate an event. We
could also use an unsigned value and use 100 as please pick a sane
default as this value and values close to it really wouldn't make much
sense.
Once we have defined what the default values are, I can modify my python
bindings patch to use these default values.
Thanks,
Tony
More information about the lvm-devel
mailing list