From: Peter Rajnoha <prajnoha@redhat.com>
To: Lennart Poettering <lennart@poettering.net>
Cc: Andy Kittner <andy.kittner@gmail.com>,
systemd-devel@lists.freedesktop.org, linux-lvm@redhat.com
Subject: Re: [linux-lvm] [systemd-devel] cryptsetup vs. swapon/fsck (some kind of race condition)
Date: Mon, 19 May 2014 10:13:40 +0200 [thread overview]
Message-ID: <5379BD34.5040202@redhat.com> (raw)
In-Reply-To: <20140515213826.GB13171@tango.0pointer.de>
On 05/15/2014 11:38 PM, Lennart Poettering wrote:
> On Thu, 15.05.14 23:15, Andy Kittner (andy.kittner@gmail.com) wrote:
>
>> | May 15 22:06:47 pinky systemd[1]: Mounted Runtime Directory.
>> | May 15 22:06:47 pinky systemd[1]: Mounted Lock Directory.
>
> This looks like remains from a *really* old systemd version. THis does not
> exist anymore. What distribution is this?
>
> This used to bind mount /run to /var/run, and similar for /var/lock.
>
>> | May 15 22:06:47 pinky systemd[1]: Started udev Kernel Device Manager.
>> | May 15 22:06:47 pinky systemd-cryptsetup[2433]: AKI DEBUG udev: open_queue_file failed with errno 2
>> | May 15 22:06:47 pinky systemd-cryptsetup[2433]: AKI DEBUG libdm:_check_udev_is_running: Udev is not running. Not using udev synchronisation code.
>>
>> So the udev queue file does not yet exist, libdm thinks the system is
>> not running udev and things go wrong...
>
> This really should be turned off in libdm. It's really stupid and
> broken. We start these things in parallel, they create these races
> without reason. Since ages we don't support non-devtmpfs kernels
> anymore, so it's *always* wrong to invoke mknod(), since the kernel will
> create the device nodes. Unless you run an early 2.6 kernels this is
> completely wrong. I'd recommend filing a bug against the distribution to
> remove this old crap from libdm.
>
> What does libdm even check there precisely?
It uses libudev's udev_queue_get_udev_is_active. So is there a better
function we should use instead? If not, there should probably be a better
one provided in libudev to check whether udev is ready to serve on the system
or not.
As for devtmpfs, are you sure it's enabled everywhere and always?
It's configurable for the kernel and one does not need to enable it it seems.
I just need to be sure that if we completely turn this fallback node management
in libdm, I won't cut someone off in some distro with settings not exactly
the same as used in Fedora.
--
Peter
next parent reply other threads:[~2014-05-19 8:13 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <lktmlp$8nh$1@ger.gmane.org>
[not found] ` <20140514163304.GG8642@tango.0pointer.de>
[not found] ` <53740A8A.5080604@gmail.com>
[not found] ` <20140515162743.GC27565@tango.0pointer.de>
[not found] ` <53752E6E.2010108@gmail.com>
[not found] ` <20140515213826.GB13171@tango.0pointer.de>
2014-05-19 8:13 ` Peter Rajnoha [this message]
2014-05-20 16:16 ` [linux-lvm] [systemd-devel] cryptsetup vs. swapon/fsck (some kind of race condition) Lennart Poettering
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=5379BD34.5040202@redhat.com \
--to=prajnoha@redhat.com \
--cc=andy.kittner@gmail.com \
--cc=lennart@poettering.net \
--cc=linux-lvm@redhat.com \
--cc=systemd-devel@lists.freedesktop.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).