Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
From:   Andreas Dilger <adilger@dilger.ca>
Mime-Version: 1.0 (1.0)
Subject: Re: Is labelling a mounted ext2/3/4 file system safe and supported?
Date:   Tue, 27 Jul 2021 01:49:39 -0600
Message-Id: <8A4E4147-0D89-4B2B-A118-F5EDABF9ABD5@dilger.ca>
References: <CAMU1PDgJAadK21H_-u3vg0NujKRzBegH0SHL2+54+23ZppFDgQ@mail.gmail.com>
Cc:     Reindl Harald <h.reindl@thelounge.net>, linux-ext4@vger.kernel.org,
        "Darrick J. Wong" <djwong@kernel.org>
In-Reply-To: <CAMU1PDgJAadK21H_-u3vg0NujKRzBegH0SHL2+54+23ZppFDgQ@mail.gmail.com>
To:     Mike Fleetwood <mike.fleetwood@googlemail.com>
Precedence: bulk

On Jul 27, 2021, at 01:33, Mike Fleetwood <mike.fleetwood@googlemail.com> wr=
ote:
>=20
> =EF=BB=BFOn Mon, 26 Jul 2021 at 21:50, Reindl Harald <h.reindl@thelounge.n=
et> wrote:
>>> Am 26.07.21 um 20:45 schrieb Mike Fleetwood:
>>> Hi,
>>>=20
>>> Using e2label to set a new label for a mounted ext4 seems to work, but
>>> is it a safe and supported thing to do?
>>=20
>> it is
>=20
> Is there some documentation which states it's safe to write to the label
> while mounted?
>=20
> I ask because 1) I am looking at adding such support into GParted and
> 2) I don't understand how it can be safe.
>=20
> Looking at the e2label source code, it just reads the superblock,
> updates the label and writes the super block.  How is that safe and
> persistent when presumably the linux kernel has an in-memory copy of the
> superblock will be written at unmount and presumable sync.

Currently, the in-memory superblock references the device buffer cache,
which is the same cache that is accessed when reading the block
device from userspace, so they are always consistent.

There has been some discussion about adding ioctl() calls to update
the filesystem label, UUID, and other fields from userspace in a safer way,
but nothing has been implemented in that direction yet (possibly Darrick
had some RFC patches, but they are not landed yet).

Cheers, Andreas=