linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Pali Rohár" <pali.rohar@gmail.com>
To: "OGAWA Hirofumi" <hirofumi@mail.parknet.co.jp>,
	"Andrius Štikonas" <andrius@stikonas.eu>,
	"Curtis Gedak" <gedakc@gmail.com>,
	"Andreas Bombe" <aeb@debian.org>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	Karel Zak <kzak@redhat.com>,
	util-linux@vger.kernel.org
Subject: Re: Changing FAT label of mounted file system
Date: Mon, 30 Oct 2017 16:35:22 +0100	[thread overview]
Message-ID: <20171030153522.gaf22di5r5c4hjfo@pali> (raw)
In-Reply-To: <87a8295fbl.fsf@devron>

On Wednesday 06 September 2017 00:35:10 OGAWA Hirofumi wrote:
> Pali Rohár <pali.rohar@gmail.com> writes:
> 
> > Hello!
> 
> Hi,
> 
> > On most file system it is possible to change its label at anytime, when 
> > mounted or unmounted. But FAT is such special file system on which is 
> > label stored in two location, plus one is in directory structure (which 
> > cannot be modified when file system is mounted).
> >
> > I would like to ask what about providing FAT specific IOCTL for vfat 
> > kernel driver which would allow to change volume label?
> 
> I have no objection to add this ioctl though, there is really needs of
> apps?

Lets ask other people, maintainers of the GUI & command line tools which
are already used by lot of users.

Andrius, Curtis and Andreas, what do you think?

> For example, ext4's volume name is sb->s_volume_name, right? And no
> ioctl to change it, so I guess the app does read-modify-write
> superblock. If so, there is theoretical race with umount and super block
> update of ext4 (yes, although updating superblock is rare timing).  So
> ext4 is unsafe actually to rewrite volume_name on the mounted fs.

You are right. But IIRC in the worst case label of ext4 would not be
changed (overwritten by the old label when kernel ext4 driver changes
superblock).

> Well, anyway, if there is the patch to add ioctl, again, I have no
> objection.
> 
> Thanks.

-- 
Pali Rohár
pali.rohar@gmail.com

      reply	other threads:[~2017-10-30 15:35 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-05 14:14 Changing FAT label of mounted file system Pali Rohár
2017-09-05 15:35 ` OGAWA Hirofumi
2017-10-30 15:35   ` Pali Rohár [this message]

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=20171030153522.gaf22di5r5c4hjfo@pali \
    --to=pali.rohar@gmail.com \
    --cc=aeb@debian.org \
    --cc=andrius@stikonas.eu \
    --cc=gedakc@gmail.com \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=kzak@redhat.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=util-linux@vger.kernel.org \
    /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).