[systemd-devel] [PATCH] debian: Fix fsck/checkroot/checkfs/syinit order

[Santi Béjar]

On Mon, Sep 13, 2010 at 10:44 AM, Santi Béjar <santi at agolina.net> wrote:
> On Mon, Sep 13, 2010 at 2:19 AM, Michael Biebl <mbiebl at gmail.com> wrote:
>> 2010/9/13 Santi Béjar <santi at agolina.net>:
>>> (with and w/o your patch) and no changes. I've just found other
>>> "interesting things", some boot scripts are failing with "read-only
>>> filesystem". Quite a few (or a lot) of boot scripts are run before /
>>> is remounted read-write. The ones failing with this error are:
>>> bootmisc.service , mountnfs-bootclean.service, screen-cleanup.service
>>> and x11-common.service.
>>
>> Are you using an initrd (initial ramdisk)?
>>
>
> Normally no, but I've also tested with it.
>
> I beginning to think that the problem is not with your patch, it just
> makes another symptom more evident. The order of the sysinit.target's
> units seems broken with respect of local_fs and with you patch also
> checkroot and checkfs.
>

I've just tested systemd_8-2 from debian experiemental and it works. I
think this patch is what makes a difference:

commit 8e28f1296d382873afc494bfcbb8dc821b0b1380
Author: Tollef Fog Heen <tfheen at err.no>
Date:   Sun Aug 29 12:20:22 2010

    Add dependencies to local-fs.target and remote-fs.target

    Grab dependencies from insserv.conf and add those to local-fs.target.

diff --git a/units/local-fs.target b/units/local-fs.target
index 52d0e68..3c6d11a 100644
--- a/units/local-fs.target
+++ b/units/local-fs.target
@@ -9,3 +9,5 @@

 [Unit]
 Description=Local File Systems
+Wants=mountall.service mountoverflowtmp.service umountfs.service
+After=mountall.service mountoverflowtmp.service umountfs.service
diff --git a/units/remote-fs.target.m4 b/units/remote-fs.target.m4
index f70fda4..586efe6 100644
--- a/units/remote-fs.target.m4
+++ b/units/remote-fs.target.m4
@@ -13,7 +13,7 @@ m4_dnl
 m4_ifdef(`FOR_SYSTEM',
 m4_dnl When running in system mode we need the network up
 Requires=network.target
-After=network.target
+After=network.target local-fs.target mountnfs.service
mountnfs-bootclean.service
 )m4_dnl

 [Install]

Santi