Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block

From	Amir Goldstein <amir73il@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block
Date	2017-05-09 10:00 +0200
Message-ID	<tF8dQ-7bU-19@gated-at.bofh.it> (permalink)
References	(10 earlier) <tF6lH-5Sx-9@gated-at.bofh.it> <tF6lH-5Sx-7@gated-at.bofh.it> <tF7rr-6Pv-5@gated-at.bofh.it> <tF7rr-6Pv-3@gated-at.bofh.it> <tF7Uu-72H-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, May 9, 2017 at 10:35 AM, Richard Weinberger <richard@nod.at> wrote:
> Amir,
>
> Am 09.05.2017 um 09:08 schrieb Amir Goldstein:
>>> Then we can queue this patch for 4.13.
>>> Please resend and make sure it addresses everything what was also
>>> suggested for the xfs s_uuid patch.
>>>
>>
>> Just to be clear, the xfs s_uuid patch is just a memcpy,
>> no different from Oleksij's patch.

See upstream commit
8f720d9 xfs: publish UUID in struct super_block

>
> Wasn't there a huge discussion about LE/BE/uniqueness and more details
> on UUID that hurt my brain.
>

LE/BE discussions are more about which variants of uuid helpers should be
created, among other things, for consumers of s_uuid to check that s_uuid
was filled by fs.
Converting s_uuid type to uuid_t or whatever is for the far future.

uniqueness of s_uuid does not exist with current filesystems,
so no reason whatsoever to act differently with ubifs.
Fixing uniqueness of s_uuid (if at all is needed) is a future VFS task.

Bottom line, for Oleksij's original patch:

Reviewed-by: Amir Goldstein <amir73il@gmail.com>

If it's not too late for 4.12 that could be nice, because then
ubifs+overlayfs would gain a new feature (constant inode numbers)

Thread

Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Richard Weinberger <richard@nod.at> - 2017-04-28 00:10 +0200
  Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Amir Goldstein <amir73il@gmail.com> - 2017-04-28 11:00 +0200
    Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Oleksij Rempel <ore@pengutronix.de> - 2017-05-02 07:40 +0200
  Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Amir Goldstein <amir73il@gmail.com> - 2017-05-02 09:30 +0200
    Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Richard Weinberger <richard@nod.at> - 2017-05-02 09:40 +0200
      Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Oleksij Rempel <ore@pengutronix.de> - 2017-05-09 06:20 +0200
        Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Oleksij Rempel <ore@pengutronix.de> - 2017-05-09 08:00 +0200
          Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Amir Goldstein <amir73il@gmail.com> - 2017-05-09 09:10 +0200
            Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Oleksij Rempel <ore@pengutronix.de> - 2017-05-09 09:40 +0200
            Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Richard Weinberger <richard@nod.at> - 2017-05-09 09:40 +0200
              Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Amir Goldstein <amir73il@gmail.com> - 2017-05-09 10:00 +0200
          Re: [PATCH v2 3/3] fs: ubifs: set s_uuid in super block Richard Weinberger <richard@nod.at> - 2017-05-09 09:10 +0200

csiph-web