On Sun, Oct 24, 2010 at 7:32 PM, Gustavo Sverzut Barbieri <[email protected]> wrote: >> +static int mount_points_list_umount(MountPoint **mount_point_list_head) { >> + MountPoint *mp, *mp_next; >> + int failed = 0; >> + >> + LIST_FOREACH_SAFE(mount_point, mp, mp_next, *mount_point_list_head) >> { >> + if (streq(mp->path, "/")) >> + continue; >> ^ add failed++ to make sure / is remounted-ro? > > it would be a bit misleading. >
Actually it is only name "failed" that is misleading. Otherwise it is quite logical - it makes it return number of mount points that had not been unmounted. Change name to n_still_mounted and it should not be misleading anymore. _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
