search for: ext2_initrd

From: Chen Hanxiao <chenhanxiao@gmail.com> If we failed to add something to initrd, just error out. Signed-off-by: Chen Hanxiao <chenhanxiao@gmail.com> --- src/ext2_initrd.ml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml index d4a4e2f..d9a3a99 100644 --- a/src/ext2_initrd.ml +++ b/src/ext2_initrd.ml @@ -151,8 +151,12 @@ let rec build_initrd debug tmpdir modpath initrd = visit topset; close_out c...

Various patches to make supermin quieter. By outputting fewer messages on the fast path, we use the slow emulated UART less, and this improves boot times. Also remove some kernel modules that we cannot or should not be using, which also improves boot times. Rich.

On 03/17/2016 12:07, Richard W.M. Jones wrote: > Also drop the sr_mod module used by the deprecated guestfs_add_cdrom > interface. > --- > src/ext2_initrd.ml | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml > index 730f806..5e841ea 100644 > --- a/src/ext2_initrd.ml > +++ b/src/ext2_initrd.ml > @@ -40,12 +40,7 @@ let kmods = [ > - "sd_mod.ko*"; sd_mod.ko is requir...

On Fri, Jul 22, 2016 at 03:59:58PM +0800, Chen Hanxiao wrote: > From: Chen Hanxiao <chenhanxiao@gmail.com> > > If we failed to add something to initrd, just error out. > > Signed-off-by: Chen Hanxiao <chenhanxiao@gmail.com> > --- > src/ext2_initrd.ml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml > index d4a4e2f..d9a3a99 100644 > --- a/src/ext2_initrd.ml > +++ b/src/ext2_initrd.ml > @@ -151,8 +151,12 @@ let rec build_initrd debug tmpdir modpath initr...

...tio-gpu since it's irrelevant, large, and pulls in other unwanted dependencies (modeswitching, drm). Add a second list of kmods which is a blacklist, applied after the first. This reduces the libguestfs initrd size from 39M down to 17M, with concomitant small reductions in boot time. --- src/ext2_initrd.ml | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml index 63ed493..a177018 100644 --- a/src/ext2_initrd.ml +++ b/src/ext2_initrd.ml @@ -54,6 +54,13 @@ let kmods = [ "megaraid*.ko*"; ] +(* A blacklist of kmods which...

Also drop the sr_mod module used by the deprecated guestfs_add_cdrom interface. --- src/ext2_initrd.ml | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml index 730f806..5e841ea 100644 --- a/src/ext2_initrd.ml +++ b/src/ext2_initrd.ml @@ -40,12 +40,7 @@ let kmods = [ "virtio*.ko*"; "libata*.ko*"; "piix*.ko*"; - &q...

On Sun, Mar 20, 2016 at 01:30:17PM +0200, Török Edwin wrote: > On 03/17/2016 12:07, Richard W.M. Jones wrote: > > Also drop the sr_mod module used by the deprecated guestfs_add_cdrom > > interface. > > --- > > src/ext2_initrd.ml | 5 ----- > > 1 file changed, 5 deletions(-) > > > > diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml > > index 730f806..5e841ea 100644 > > --- a/src/ext2_initrd.ml > > +++ b/src/ext2_initrd.ml > > @@ -40,12 +40,7 @@ let kmods = [ > > - &quo...

Document what each module does, using *.mli files. Remove the --dtb option, it's obsolete. Rename modules according to their purpose. Rich.

DAX is explained in detail here: https://lwn.net/Articles/610174/ This patch adds support to supermin for using a DAX root filesystem. The corresponding libguestfs patches will be posted shortly once I've tested them a bit more. This requires qemu >= 2.6. Unfortunately it's not really a win for a few reasons: - Requires enabling ACPI, which slows everything down by hundreds of

.... - Removes the dependency on static zlib & lzma. - We can use an alternate libc to make a much smaller init binary (see following commits). --- README | 4 +- configure.ac | 74 ++------------------------- src/Makefile.am | 1 - src/config.ml.in | 2 + src/ext2_initrd.ml | 28 ++++++++++- src/init.c | 145 ----------------------------------------------------- 6 files changed, 34 insertions(+), 220 deletions(-) diff --git a/README b/README index 34949a2..56a28f6 100644 --- a/README +++ b/README @@ -97,9 +97,9 @@ are building: qemu >= 0.13 kerne...

Allow an alternate libc, such as dietlibc, to be used for the init binary in the supermin appliance. Rich.

Hi, Rich: I met a supermin issue on CentOS 7.1: supermin: version: 5.1.10 supermin: rpm: detected RPM version 4.11 supermin: package handler: fedora/rpm supermin: acquiring lock on /root/test/aaa/lock supermin: build: /usr/lib64/guestfs/supermin.d supermin: build: visiting /usr/lib64/guestfs/supermin.d/base.tar.gz type gzip base image (tar) supermin: build: visiting

...in the supermin codebase; no actual functional change. Thanks, -- Pino Toscano (3): ext2: simplify tracking of visited modules utils: remove unused run_python function Add and use an helper error function src/build.ml | 20 +++++----------- src/dpkg.ml | 4 +--- src/ext2_initrd.ml | 10 ++++---- src/kernel.ml | 27 ++++++++++------------ src/package_handler.ml | 7 +++--- src/pacman.ml | 6 ++--- src/prepare.ml | 12 ++++------ src/rpm.ml | 33 ++++++++------------------- src/supermin.ml | 62 +++++++++++++++++++----------...

...upermin failed to write modules into initrd. That doesn't look good! What is supposed to happen here is that the modules path (/lib/modules/3.10.0-229.el7.x86_64) is searched in order to find any modules matching one of these patterns: https://github.com/libguestfs/supermin/blob/master/src/ext2_initrd.ml#L33-L52 You could try doing for example: find /lib/modules/3.10.0-229.el7.x86_64/ -name 'ext4.ko*' BTW this isn't always a problem. It could be that you are using a self-compiled CentOS kernel which doesn't have any modules (eg. everything is compiled into the kernel). If...

v1 -> v2: - If we split out the init program into a separate init/ directory, that makes it much easier to build against an alternate libc. I tried to build against uClibc, but uClibc requires an entire build chain, which looked like it was going to be a massive ballache. Rich.

...into initrd. > >That doesn't look good! > >What is supposed to happen here is that the modules path >(/lib/modules/3.10.0-229.el7.x86_64) is searched in order to find any >modules matching one of these patterns: > > https://github.com/libguestfs/supermin/blob/master/src/ext2_initrd.ml#L33-L52 > >You could try doing for example: > > find /lib/modules/3.10.0-229.el7.x86_64/ -name 'ext4.ko*' > >BTW this isn't always a problem. It could be that you are using a >self-compiled CentOS kernel which doesn't have any modules >(eg. everything is...

At 2016-07-21 19:52:11, "Pino Toscano" <ptoscano@redhat.com> wrote: >On Thursday, 21 July 2016 18:50:52 CEST Chen Hanxiao wrote: >> >> At 2016-07-21 18:43:04, "Richard W.M. Jones" <rjones@redhat.com> wrote: >> >On Thu, Jul 21, 2016 at 06:36:20PM +0800, Chen Hanxiao wrote: >> >> the virtio modules in kmods did existed in this

...138,7 +138,7 @@ main () /* XXX Because of the way we construct the module list, the * "modules" file can contain non-existent modules. Ignore those * for now. Really we should add them as missing dependencies. - * See ext2initrd.c:ext2_make_initrd(). + * See src/ext2_initrd.ml. */ if (access (line, R_OK) == 0) insmod (line); -- 1.8.3.1

...hine works fine. > > It works fine now. > > But why we make initrd only depends on modules.dep? modules.dep is used to work out the module dependencies, so we know which minimal set of modules should be copied into the initrd: https://github.com/libguestfs/supermin/blob/master/src/ext2_initrd.ml#L68-L152 If you copy too many modules into the initrd, then it will be huge and boot times will be very slow, so knowing the exact set of modules which are required is important. Of course if modules.dep is broken then that won't work. Do you understand who broke the file and/or why it wa...

--- src/utils.ml | 21 +++++++++++++++++++++ src/utils.mli | 3 +++ 2 files changed, 24 insertions(+) diff --git a/src/utils.ml b/src/utils.ml index 3e81c21..7ae24bd 100644 --- a/src/utils.ml +++ b/src/utils.ml @@ -204,3 +204,24 @@ let compare_architecture a1 a2 = exit 1 in compare (index_of_architecture a1) (index_of_architecture a2) + +(* Parse a size field, eg. "10G".