All of lore.kernel.org
 help / color / mirror / Atom feed
From: Heinz Mauelshagen <mauelshagen@redhat.com>
To: lvm-devel@redhat.com
Subject: [PATCH pvmove 0/6] Refactoring pvmove with generic APIs
Date: Fri, 7 Dec 2007 12:49:54 +0100	[thread overview]
Message-ID: <20071207114954.GA6313@redhat.com> (raw)
In-Reply-To: <475825EF.6000803@ce.jp.nec.com>


Nick,

it would be better to make this more generic.

Ie. convert_segments(struct lvm_conversion_handler *handler, ...)
rather than having a per target conversion function.

Heinz


On Thu, Dec 06, 2007 at 11:40:15AM -0500, Jun'ichi Nomura wrote:
> Hi,
> 
> This patchset is a follow up to the previous post:
> https://www.redhat.com/archives/lvm-devel/2007-November/msg00034.html
> 
> This version introduces 3 APIs and let pvmove use them:
>   - insert_layer_for_segments_on_pv()
>       Insert a layer (a linear lv segment) below each lv segment
>       of the LV, only when the segment is on a specified PV.
>   - convert_segments_mirrored()
>       Convert each segment in the LV to mirrored.
>   - split_parent_segments()
>       For a given layer LV, walk through LVs in the VG and split
>       any LV segment whose underlying layer segment is splitted.
> 
> I'm trying to add functionality for lvconvert, i.e. mirroring
> as a whole LV, not segment by segment.
> However, it involves moving some codes from toollib to lib and
> I think further discussion may be necessary.
> And anyway they will be an addition to this patchset.
> 
> So I'm posting this set first as they are stable and easy to review.
> 
> [1/6] add insert_layer_for_segments_on_pv()
> [2/6] add convert_segments_mirrored()
> [3/6] add split_parent_segments()
> [4/6] change pvmove to use the generalized APIs
> [5/6] remove unused mirrored_pv/mirrored_pe params from internal code
> [6/6] remove unused can_split param from allocate_extents()
> [Appendix] pvmove test script for 'make check'
> 
> Thanks,
> -- 
> Jun'ichi Nomura, NEC Corporation of America
> 
> --
> lvm-devel mailing list
> lvm-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/lvm-devel

-- 

Regards,
Heinz    -- The LVM Guy --

*** Software bugs are stupid.
    Nevertheless it needs not so stupid people to solve them ***

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

Heinz Mauelshagen                                 Red Hat GmbH
Consulting Development Engineer                   Am Sonnenhang 11
Storage Development                               56242 Marienrachdorf
                                                  Germany
Mauelshagen at RedHat.com                            PHONE +49  171 7803392
                                                  FAX   +49 2626 924446
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-



  parent reply	other threads:[~2007-12-07 11:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-06 16:40 [PATCH pvmove 0/6] Refactoring pvmove with generic APIs Jun'ichi Nomura
2007-12-06 16:48 ` [PATCH pvmove 1/6] Add insert_layer_for_segments_on_pv() Jun'ichi Nomura
2007-12-06 16:48 ` [PATCH pvmove 2/6] Add convert_segments_mirrored() Jun'ichi Nomura
2007-12-06 16:49 ` [PATCH pvmove 3/6] Add split_parent_segments() Jun'ichi Nomura
2007-12-06 16:50 ` [PATCH pvmove 4/6] Change pvmove to use the generalized APIs Jun'ichi Nomura
2007-12-06 16:51 ` [PATCH pvmove 5/6] Remove unused mirrored_pv/mirrored_pe params from internal code Jun'ichi Nomura
2007-12-06 16:51 ` [PATCH pvmove 6/6] Remove unused can_split param from allocate_extents() Jun'ichi Nomura
2007-12-06 16:53 ` [PATCH pvmove 0/6] Refactoring pvmove with generic APIs Jun'ichi Nomura
2007-12-07 11:49 ` Heinz Mauelshagen [this message]
2007-12-07 15:48   ` Jun'ichi Nomura

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=20071207114954.GA6313@redhat.com \
    --to=mauelshagen@redhat.com \
    --cc=lvm-devel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.