[Libguestfs] [PATCH v3.1 0/9] uuid: add btrfs uuid change support and set_uuid_random
Chen, Hanxiao
chenhanxiao at cn.fujitsu.com
Mon Jun 29 03:48:58 UTC 2015
> -----Original Message-----
> From: libguestfs-bounces at redhat.com [mailto:libguestfs-bounces at redhat.com] On
> Behalf Of Pino Toscano
> Sent: Friday, June 26, 2015 9:11 PM
> To: libguestfs at redhat.com
> Subject: Re: [Libguestfs] [PATCH v3.1 0/9] uuid: add btrfs uuid change support and
> set_uuid_random
>
> In data venerdì 26 giugno 2015 17:35:35, Chen Hanxiao ha scritto:
> > - Btrfs-progs v4.1 introduced new feature of changing
> > uuid of btrfs partition.
> > This patch add support of this.
> >
> > - Introduce set_uuid_random
> >
> > - uuids.c did a lot of deplicated work for changing uuid
> > of fs. Use existing functions.
> >
> > v3.1: fix typos
> > v3: set errno if feature is not available.
> >
> > Chen Hanxiao (9):
> > [...]
> > uuid: use existing function of ext2
> > uuid: use newly introduced xfs_set_uuid of xfs
> > uuid: use newly introduced swap_set_uuid
>
> These commits look good.
>
> > New API: btrfs_set_uuid_random
> > New API: set_e2uuid_random
> > New API: swap_set_uuid_random
> > New API: xfs_set_uuid_random
> > New API: set_uuid_random
>
> New internal functions are not "new API" -- that is used for user-facing
> APIs, such as set_uuid_random.
>
> Could you also please squash them together, since they are not really
> different changes from the set_uuid_random addition?
>
> Commits 5,6,8 look good as well.
I'll squash all internal api into one patch and resend.
Regards,
- Chen
More information about the Libguestfs
mailing list