From: Maxim Patlasov <mpatlasov@parallels.com>
To: Mike Snitzer <snitzer@redhat.com>
Cc: dm-devel@redhat.com
Subject: Re: dm thin: How to shrink think pool device?
Date: Wed, 18 Mar 2015 18:42:44 -0700 [thread overview]
Message-ID: <550A2994.5000601@parallels.com> (raw)
In-Reply-To: <20150319005144.GA18305@redhat.com>
On 03/18/2015 05:51 PM, Mike Snitzer wrote:
> On Wed, Mar 18 2015 at 7:46pm -0400,
> Maxim Patlasov <mpatlasov@parallels.com> wrote:
>
>> Hi,
>>
>> It seems that a "trim" pool message was planned long time ago, but never
>> implemented. Could someone update about current status of that? The
>> question came from the following use-case:
>>
>> Suppose we have a thin pool on top of LVM volume group. The group comprises
>> of several storage drives. Now I need to (physically) remove one of them.
>> The intention is to use pvmove and vgreduce, but obviously thin pool must
>> be requested to shrink itself at first.
> We only support growing the pool. If a drive used by the pool goes
> missing then thin_repair will need to fix things.
Do you think a feature allowing to shrink pool data device on-line
smoothly will be useful (w.r.t all code complications needed to relocate
data blocks inside dm-thin while handling i/o at the same time)?
>
>> Btw, Documentation/device-mapper/thin-provisioning.txt still has a leftover
>> about it:
>>
>>> If you wish to reduce the size of your thin device and potentially
>>> regain some space then send the 'trim' message to the pool.
> I'll remove that stale documentation (commit 5ec02084f6 should've done
> that).
>
> The 'trim' message was never about reducing pool space. The usage for
> the trim message (sent to the pool device) was:
> trim <dev_id> <new_size_in_sectors>
>
> It was to reduce the size of the thin volume; don't think it was ever
> wired up because it wasn't needed.
Hm... "dmset load <thin-volume>" seems to do exactly that. But, yes, it
has nothing to do with reclaiming storage space back to the system.
Thanks,
Maxim
prev parent reply other threads:[~2015-03-19 1:42 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-18 23:46 dm thin: How to shrink think pool device? Maxim Patlasov
2015-03-19 0:51 ` Mike Snitzer
2015-03-19 1:42 ` Maxim Patlasov [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=550A2994.5000601@parallels.com \
--to=mpatlasov@parallels.com \
--cc=dm-devel@redhat.com \
--cc=snitzer@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox