All of lore.kernel.org
 help / color / mirror / Atom feed
* Please revert commit fa0115c3c06176bd0fdfcd8966021979ec84f3d7
@ 2009-06-23  7:25 Seewer Philippe
       [not found] ` <4A408358.7050706-omB+W0Dpw2o@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Seewer Philippe @ 2009-06-23  7:25 UTC (permalink / raw)
  To: <initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>

Commit fa0115c3c06176bd0fdfcd8966021979ec84f3d7 moved calling
cryptsetup from the mount hook to inside a udev rule. While 
this is a good idea it causes more problems than it solves:

cryptsetup version 1.6 (and possibly earlier versions) calls
'udevadm settle' on it's own. This must not be done from inside
udev rules as it potentially results in a deadlock depending
on udev version.

Thank you,
Philippe
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Please revert commit fa0115c3c06176bd0fdfcd8966021979ec84f3d7
       [not found] ` <4A408358.7050706-omB+W0Dpw2o@public.gmane.org>
@ 2009-06-24  6:59   ` Seewer Philippe
  0 siblings, 0 replies; 2+ messages in thread
From: Seewer Philippe @ 2009-06-24  6:59 UTC (permalink / raw)
  To: <initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>

Seewer Philippe wrote:
> Commit fa0115c3c06176bd0fdfcd8966021979ec84f3d7 moved calling
> cryptsetup from the mount hook to inside a udev rule. While this is a 
> good idea it causes more problems than it solves:
> 
> cryptsetup version 1.6 (and possibly earlier versions) calls
> 'udevadm settle' on it's own. This must not be done from inside
> udev rules as it potentially results in a deadlock depending
> on udev version.

A short amendment to this:

- Fedora cryptsetup is patched, removing the udevadm settle altogether.
That's why it was working for most of you. (Thanks to Warren for helping
to debug this)

- In addition I don't think its a good idea to include anything in a
udev-rule that does user interaction, since udev has an built-in
event-timeout.

> 
> Thank you,
> Philippe
> -- 
> To unsubscribe from this list: send the line "unsubscribe initramfs" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-06-24  6:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-06-23  7:25 Please revert commit fa0115c3c06176bd0fdfcd8966021979ec84f3d7 Seewer Philippe
     [not found] ` <4A408358.7050706-omB+W0Dpw2o@public.gmane.org>
2009-06-24  6:59   ` Seewer Philippe

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.