Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip remount sync when systemd is used (issue1097) #1099

Merged

Conversation

jsmeix
Copy link
Member

@jsmeix jsmeix commented Dec 2, 2016

Remounting all what is mounted below /mnt/local
with sync option is no longer needed when systemd is used
because when systemd is used reboot, halt, poweroff,
and shutdown are replaced by scripts that do umount
plus sync to safely shut down the recovery system,
see #1011
and #1097

Furthermore remounting with sync option could become
in practice a major annoyance because it makes writing
anything below /mnt/local basically unusable slow,
see #1097

@jsmeix jsmeix added cleanup enhancement Adaptions and new features labels Dec 2, 2016
@jsmeix jsmeix added this to the Rear v2.00 milestone Dec 2, 2016
@jsmeix jsmeix self-assigned this Dec 2, 2016
@jsmeix
Copy link
Member Author

jsmeix commented Dec 2, 2016

If there are no furious objections I will merge it soon...

@gdha
Copy link
Member

gdha commented Dec 2, 2016

@jsmeix looks clean - thx

@jsmeix jsmeix merged commit af21361 into rear:master Dec 2, 2016
@jsmeix jsmeix deleted the Skip_remount_sync_when_systemd_is_used_issue1097 branch December 2, 2016 12:48
jsmeix added a commit that referenced this pull request Dec 5, 2016
…is_used_issue1097

Skip remount async when systemd is used.
Remounting with async option is not needed
when systemd is used because when systemd
is used remounting with sync option is skipped and
to aviod needless operations remounting with async
option is also skipped, see
#1097
and
#1099
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants