Bug #1732

Various changes for diskless

Added by rumcic about 4 years ago. Updated almost 4 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:-
Target version:-

Description

Attached patches fix a bug (by default, the mfs mount for /etc is too small for
our /etc due to the big blacklists in etc/ssh/), remove a check for devfs (we
have had devfs for quite a while now, so we probably do not need to check for
it's existance anymore) and replace the usage of mfs with tmpfs (due to the
ability of tmpfs to use swap when available, I believe it is a better choice).
--
Regards,
Rumko

0001-Increase-the-default-md-size-for-etc.patch Magnifier (867 Bytes) rumcic, 04/14/2010 09:31 PM

0002-Remove-the-check-for-devfs-in-initdiskless.patch Magnifier (1.38 KB) rumcic, 04/14/2010 09:31 PM

0003-Use-tmpfs-instead-of-mfs.patch Magnifier (1.05 KB) rumcic, 04/14/2010 09:31 PM

0005-md_size-can-now-contain-more-flexible-definitions-of.patch Magnifier (2.33 KB) rumcic, 04/18/2010 04:21 PM

History

#1 Updated by rumcic about 4 years ago

Rumko wrote:
> Attached patches fix a bug (by default, the mfs mount for /etc is too small
> for our /etc due to the big blacklists in etc/ssh/), remove a check for devfs
> (we have had devfs for quite a while now, so we probably do not need to check
> for it's existance anymore) and replace the usage of mfs with tmpfs (due to
> the ability of tmpfs to use swap when available, I believe it is a better
> choice).

Another patch ... now if md_size contains only a number (without a suffix), it
is assumed it represents the number of sectors (and is multiplied by 512
before passing to mount_tmpfs), otherwise it is passed to mount_tmpfs as is
(so now, you can specify "5M" in md_size and it works).
--
Regards,
Rumko

#2 Updated by swildner almost 4 years ago

Thanks, committed! -> 86de41e6561c1bb93c982537c87446371f8279b7

I was about to just push your patches, but honestly, the commit messages looked
too ugly. :) Next time, please give it some kind of "subsystem" indication in
the first line (such as my "diskless: "; "diskless rc scripts: " would fit the
bill too). And please try to keep lines <= ~70 chars or so. Maybe less for the
first line (committer(7) talks about <50, which I personally find a bit small).

Also available in: Atom PDF