Linux Device Mapper development
 help / color / mirror / Atom feed
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

      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