Re: [systemd-devel] fstab-generator: why nofail=true removes Before=$post in .mount

2013-08-08 Thread Tom Gundersen
On Thu, Aug 8, 2013 at 7:35 AM, WANG Chao chaow...@redhat.com wrote:
 In add_mount(), if nofail=true, Before=$post won't be added to
 xxx.mount:

 [..]
 if (post  !noauto  !nofail  !automount)
 fprintf(f,
 Before=%s\n,
 post);

 why doing this?

The way things currently work you can use 'nofail' when you want your
boot to be unaffected by a missing device. If we change this as you
suggest, then a missing device will lead to a long timeout (which
delays boot by a couple of minutes). That's not really wrong (it
doesn't contradict the fstab manpage), but would obviously limit the
usefulness of 'nofail'.

Cheers,

Tom
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


[systemd-devel] fstab-generator: why nofail=true removes Before=$post in .mount

2013-08-07 Thread WANG Chao
Hi, all

I have a question after looking fstab-generator.c. I hope someone can
help me.

In add_mount(), if nofail=true, Before=$post won't be added to
xxx.mount:

[..]
if (post  !noauto  !nofail  !automount)
fprintf(f,
Before=%s\n,
post);

why doing this?

If nofail=true, that means $post Wants this xxx.mount. And no matter
xxx.mount fails or not, $post will be reached (if post is a target
unit).

I think it's more reasonable to try do xxx.mount before $post. So I'd
like to keep Before=$post in the nofail=true case.

I coud post a patch, but I think it's best to consult to you guys first
in case I'm missing something.


Thanks
WANG Chao
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel