linux-lvm.redhat.com archive mirror
 help / color / mirror / Atom feed
From: Zdenek Kabelac <zdenek.kabelac@gmail.com>
To: matthew patton <pattonme@yahoo.com>,
	"Brian J. Murrell" <brian@interlinx.bc.ca>,
	"linux-lvm@lists.linux.dev" <linux-lvm@lists.linux.dev>
Subject: Re: pvmove thin volume doesn't move
Date: Wed, 19 Nov 2025 18:31:00 +0100	[thread overview]
Message-ID: <42516759-f09a-407a-9c2d-1f82d0db19af@gmail.com> (raw)
In-Reply-To: <94679191.5310372.1763572932545@mail.yahoo.com>

Dne 19. 11. 25 v 18:22 matthew patton napsal(a):
> Right,  so LVMThin(1) conveniently omits "Oh BTW standard functionality like pvmove() is not implemented."
> 
> lvmthin is a sub-set or sub-class of LVM, therefore any reasonable person would assume all functionality cascades down. Because it pretty much does for everything else.
> 
>> lvm2 has 'no idea'  which disk space is in-use for any individual thin LV.
>> (there are tools like 'thin_ls/thin_rmap' for that)
> 
> And any reasonable person would assume lvmthin would have implemented the necessary hooks so that pvmove() would work by lvmthin handing lvm back a linked-list of all of the pieces in the correct order so they could be written to lvm(thick) or to another lvmthin pool. This is not that big of a cognitive leap.
> 
> That Linux LVM thin is missing "obvious" functionality is fine, it just needs to be stated very clearly.


Yep  thanks this is a  legit complain, we will improve our doc to be way more 
clear about this.

I've been also thinking - that adding support of 'dumb' mirroring of thinLV 
into pvmove - isn't a very difficult request - so we will see - although 
obviously copying  10TiB empty thin volume isn't going to be 'a fast 
operation' this way - but it's not any worse then a 'dd'.


Regards

Zdenek


  reply	other threads:[~2025-11-19 17:31 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-16 13:27 pvmove thin volume doesn't move Brian J. Murrell
2025-11-17 15:01 ` Zdenek Kabelac
2025-11-17 23:30   ` Brian J. Murrell
2025-11-17 23:37     ` Zdenek Kabelac
2025-11-18 23:14       ` Brian J. Murrell
2025-11-19  9:16         ` Zdenek Kabelac
2025-11-19 14:07           ` Matthew Patton
2025-11-19 15:46             ` Brian J. Murrell
2025-11-19 16:34               ` Zdenek Kabelac
2025-11-19 16:06             ` Zdenek Kabelac
2025-11-19 16:36               ` Brian J. Murrell
2025-11-19 16:59                 ` Zdenek Kabelac
2025-11-19 17:22               ` matthew patton
2025-11-19 17:31                 ` Zdenek Kabelac [this message]
2025-11-19 17:38                   ` Brian J. Murrell
2025-11-19 18:11                     ` Zdenek Kabelac
2025-11-19 19:41                       ` matthew patton
2025-11-19 20:38                         ` Zdenek Kabelac
2025-11-19 16:10 ` David Teigland

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=42516759-f09a-407a-9c2d-1f82d0db19af@gmail.com \
    --to=zdenek.kabelac@gmail.com \
    --cc=brian@interlinx.bc.ca \
    --cc=linux-lvm@lists.linux.dev \
    --cc=pattonme@yahoo.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;
as well as URLs for NNTP newsgroup(s).