Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 98405 - [PATCH] checkroot shouldn't try to remount a nfs-mounted root filesystem readonly
Summary: [PATCH] checkroot shouldn't try to remount a nfs-mounted root filesystem read...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] baselayout (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-08 15:04 UTC by Patrick McLean
Modified: 2005-07-09 15:25 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for checkroot to make it not attempt to remount a network-mounted root filesystem (checkroot-dont-remout-network-fs.patch,417 bytes, patch)
2005-07-08 15:05 UTC, Patrick McLean
Details | Diff
checkroot.patch (checkroot.patch,1.77 KB, patch)
2005-07-08 17:14 UTC, SpanKY
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick McLean gentoo-dev 2005-07-08 15:04:51 UTC
There is no reason to try to remount a nfs-mounted root filesystem readonly,
since fsck can't be run on a nfs filesystem, and it just makes an ugly failure
message appear in the startup of a gentoo system with a read-only nfs-mounted
root filesystem.

Reproducible: Always
Steps to Reproduce:
Comment 1 Patrick McLean gentoo-dev 2005-07-08 15:05:27 UTC
Created attachment 62965 [details, diff]
patch for checkroot to make it not attempt to remount a network-mounted root filesystem
Comment 2 Martin Schlemmer (RETIRED) gentoo-dev 2005-07-08 16:58:03 UTC
Hmm, I wonder in any case if we should not make it more generic (check if its
already ro, and do not remount if so), as currently it will remount it anyhow,
upping the mount count, and causing journaled filesystems to be checked in half
the reboot counts that they should have been checked ....
Comment 3 SpanKY gentoo-dev 2005-07-08 17:14:12 UTC
Created attachment 62968 [details, diff]
checkroot.patch

i think we should do both ... how does this look to everyone ?
Comment 4 Patrick McLean gentoo-dev 2005-07-08 20:11:36 UTC
That looks pretty good to me, I thought about doing something like that, but
decided to just check for a network FS in my patch, assuming there was some
reason in particular the script wasn't checking for already ro filesystems,
other than not being implemented yet.
Comment 5 Martin Schlemmer (RETIRED) gentoo-dev 2005-07-09 03:55:39 UTC
Yup, ok here as well.  Just remember to fix that inverse if ! touch ... logic.
Comment 6 SpanKY gentoo-dev 2005-07-09 15:25:57 UTC
fixed touch logic and committed