From: Lennart Poettering <email@example.com>
To: Matthew Wilcox <firstname.lastname@example.org>
Cc: "Matteo Croce" <email@example.com>,
firstname.lastname@example.org, "Luca Boccassi" <email@example.com>,
"Jens Axboe" <firstname.lastname@example.org>,
"Alexander Viro" <email@example.com>,
"Damien Le Moal" <firstname.lastname@example.org>,
"Tejun Heo" <email@example.com>,
"Javier González" <firstname.lastname@example.org>,
"Niklas Cassel" <email@example.com>,
"Johannes Thumshirn" <firstname.lastname@example.org>,
"Hannes Reinecke" <email@example.com>
Subject: Re: [PATCH -next 1/5] block: add disk sequence number
Date: Thu, 25 Mar 2021 21:58:42 +0100 [thread overview]
Message-ID: <YFz5gjRDtrW6RxFk@gardel-login> (raw)
On Mo, 15.03.21 21:04, Matthew Wilcox (firstname.lastname@example.org) wrote:
> On Mon, Mar 15, 2021 at 08:18:24PM +0000, Matthew Wilcox wrote:
> > On Mon, Mar 15, 2021 at 09:02:38PM +0100, Matteo Croce wrote:
> > > From: Matteo Croce <email@example.com>
> > >
> > > Add a sequence number to the disk devices. This number is put in the
> > > uevent so userspace can correlate events when a driver reuses a device,
> > > like the loop one.
> > Should this be documented as monotonically increasing? I think this
> > is actually a media identifier. Consider (if you will) a floppy disc.
> > Back when such things were common, it was possible with personal computers
> > of the era to have multiple floppy discs "in play" and be prompted to
> > insert them as needed. So shouldn't it be possible to support something
> > similar here -- you're really removing the media from the loop device.
> > With a monotonically increasing number, you're always destroying the
> > media when you remove it, but in principle, it should be possible to
> > reinsert the same media and have the same media identifier number.
> So ... a lot of devices have UUIDs or similar. eg:
> $ cat /sys/block/nvme0n1/uuid
> https://linux.die.net/man/8/scsi_id (for scsi)
> how about making this way more generic; create an xattr on a file to
> store the uuid (if one doesn't already exist) whenever it's used as the
> base for a loop device. then sysfs (or whatever) can report the contents
> of that xattr as the unique id.
> That can be mostly in userspace -- losetup can create it, and read it.
> It can be passed in as the first two current-reserved __u64 entries in
> loop_config. The only kernel change should be creating the sysfs
> entry /sys/block/loopN/uuid from those two array entries.
I prefer seqnos over uuids because we can order them when we see a
bunch of uevents for the same loopback device with their seqnos, as
mentioned in that other mail.
But beggars can't be choosers. If we could propagate some uuid from
the loopback setup ioctl into the device so that that appears via
sysfs that would work too for me, but not as robustly, since we lack
the ordering to detect whether it's worth waiting for more uevents or
if already somebody else took possesion of the device.
TLDR: seqnos FTW! but uuids assigned at attachment time is better than
Lennart Poettering, Berlin
next prev parent reply other threads:[~2021-03-25 20:59 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-15 20:02 [PATCH -next 0/5] block: add a sequence number to disks Matteo Croce
2021-03-15 20:02 ` [PATCH -next 1/5] block: add disk sequence number Matteo Croce
2021-03-15 20:18 ` Matthew Wilcox
2021-03-15 21:04 ` Matthew Wilcox
2021-03-15 21:32 ` Lennart Poettering
2021-03-25 17:29 ` Matteo Croce
2021-03-26 8:00 ` Hannes Reinecke
2021-03-25 20:58 ` Lennart Poettering [this message]
2021-03-16 14:13 ` Christoph Hellwig
2021-04-20 20:12 ` Lennart Poettering
2021-03-25 20:52 ` Lennart Poettering
2021-03-16 1:44 ` JeffleXu
2021-03-23 17:43 ` Matteo Croce
2021-03-15 20:02 ` [PATCH -next 2/5] block: add ioctl to read the " Matteo Croce
2021-03-15 20:13 ` Matthew Wilcox
2021-03-15 20:17 ` Damien Le Moal
2021-03-15 20:34 ` Matteo Croce
2021-03-15 20:02 ` [PATCH -next 3/5] block: refactor sysfs code Matteo Croce
2021-03-15 20:02 ` [PATCH -next 4/5] block: export diskseq in sysfs Matteo Croce
2021-03-15 20:02 ` [PATCH -next 5/5] loop: increment sequence number Matteo Croce
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:
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
* 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).