From: Maxim Levitsky <maximlevitsky@gmail.com>
To: linux-hotplug@vger.kernel.org
Subject: Re: [BUG] udev doesn't run blkid on cdroms with one session
Date: Sun, 20 Sep 2009 21:28:29 +0000 [thread overview]
Message-ID: <1253482109.4941.7.camel@maxim-laptop> (raw)
In-Reply-To: <1253411352.3791.14.camel@maxim-laptop>
On Sun, 2009-09-20 at 06:00 +0200, Kay Sievers wrote:
> On Sun, Sep 20, 2009 at 03:49, Maxim Levitsky <maximlevitsky@gmail.com> wrote:
> > A line in 60-persistent-storage.rules states:
> >
> > KERNEL="sr*", ENV{ID_CDROM_MEDIA}="?*", IMPORT{program}="/sbin/blkid -o udev -p -u noraid -O $env{ID_CDROM_MEDIA_SESSION_LAST_OFFSET} $tempnode"
> >
> > It seems to assume that ID_CDROM_MEDIA_SESSION_LAST_OFFSET is always
> > set, since otherwise blkid fails:
> >
> > maxim@maxim-laptop:~$ /sbin/blkid -o udev -p -u noraid -O /dev/sr0
> > The low-probe option requires a device
>
> In vol_id it was an optional parameter, and worked fine with an empty
> option, at least when long options are used:
> { "offset", optional_argument, NULL, 'o' },
>
> We could also make cdrom_id print the "0", that might be the simplest.
Indeed,
I have reported this bug at
https://bugs.launchpad.net/ubuntu/+source/udev/+bug/431055
Martin Pitt, has commited now a fix for this problem by splitting the
rule in two according to value existence of
ID_CDROM_MEDIA_SESSION_LAST_OFFSET
>
> > maxim@maxim-laptop:~$ /sbin/blkid -o udev -p -u noraid /dev/sr0
> > ID_FS_LABELÍROM
> > ID_FS_LABEL_ENCÍROM\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20\x20
> > ID_FS_VERSION=Joliet\x20Extension
> > ID_FS_TYPE=iso9660
> > ID_FS_USAGE=filesystem
> >
> > Maybe this is a change in blkid
> > (btw, you notice obvious bug, the label is 'padded' with spaces, I will
> > report that to util-linux package maintainers)
>
> Maybe this is expected in the *_ENC version of the string, I'm not sure.
>
Do you know where to report the util-linux bugs?
There was a fork, util-linux-ng, but ubuntu seems to use util-linux
naming, but it seems to be based on -ng.
As usual, filling a lanchpad bug didn't really help.
I think that _ENC means that non printable chars are printed as \xNNN,
and indeed, here I see the long row of spaces after the volume name.
I am sure this is trivial to fix, but I want to know where to report
that (launchpad sadly doesn't count...)
Best regards,
Maxim Levitsky
prev parent reply other threads:[~2009-09-20 21:28 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-20 1:49 [BUG] udev doesn't run blkid on cdroms with one session Maxim Levitsky
2009-09-20 4:00 ` Kay Sievers
2009-09-20 21:28 ` Maxim Levitsky [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=1253482109.4941.7.camel@maxim-laptop \
--to=maximlevitsky@gmail.com \
--cc=linux-hotplug@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