[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#494466: marked as done ([patch, RFC] Allow to select driver inclusion policy for initramfs-tools)

Your message dated Sun, 24 Aug 2008 19:32:02 +0000
with message-id <E1KXLJi-0002mR-W3@ries.debian.org>
and subject line Bug#494466: fixed in base-installer 1.94
has caused the Debian Bug report #494466,
regarding [patch, RFC] Allow to select driver inclusion policy for initramfs-tools
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
494466: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=494466
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: base-installer
Version: 1.93
Severity: wishlist
Tags: patch

With the default initrd now being over 6 GB on amd64 and the fact that 
lilo has structural problems with large initrds, I think that 
implementing something like the attached patch for Lenny is almost 
unavoidable.

It is also needed for e.g. arm where in some cases the initrd is getting 
too big for the flash memory it needs to fit in.

My personal guess is that selecting MODULES=dep for x86 is unlikely to 
result in unbootable systems. As I've no idea about other architectures 
the default can be set per architecture (to be determined by their 
porters).

Note that the patch adds a new dialog (at medium priority).
Note that this will also need to be documented in the installation guide.


Index: debian/bootstrap-base.templates
===================================================================
--- debian/bootstrap-base.templates	(revision 54931)
+++ debian/bootstrap-base.templates	(working copy)
@@ -62,6 +62,23 @@
  The package ${GENERATOR} that was selected to generate the
  initrd is not supported.
 
+Template: base-installer/initramfs-tools/driver-policy
+Type: select
+Choices-C: most, dep
+# :sl3:
+__Choices: generic: include all available drivers, targeted: only include drivers needed for this system
+# :sl3:
+_Description: Drivers to include in the initrd:
+ The primairy function of an initrd is to allow the kernel to mount the
+ root file system. It therefore needs to contain all drivers and supporting
+ programs required to do that.
+ .
+ A generic initrd is much larger than a targeted one and may even be so
+ large that some bootloaders are unable to load it but has the advantage that
+ it can be used to boot the target system on almost any hardware. With the
+ smaller targeted initrd there is a very small chance that not all needed
+ drivers are included.
+
 Template: base-installer/kernel/failed-install
 Type: error
 _Description: Unable to install the selected kernel
Index: debian/templates-arch
===================================================================
--- debian/templates-arch	(revision 54931)
+++ debian/templates-arch	(working copy)
@@ -18,6 +18,14 @@
 Description: for internal use; can be preseeded
  Package to use to generate initramfs
 
+Template: base-installer/kernel/linux/initramfs-tools/driver-policy
+Type: string
+Default: most
+Default[amd64]: dep
+Default[i386]: dep
+Description: for internal use
+ Default driver inclusion policy for initramfs-tools
+
 Template: base-installer/kernel/linux/extra-packages
 Type: string
 Default:
Index: library.sh
===================================================================
--- library.sh	(revision 54931)
+++ library.sh	(working copy)
@@ -512,7 +512,7 @@
 
 		# initramfs-tools needs busybox pre-installed (and only
 		# recommends it)
-		if [ "$rd_generator" = "initramfs-tools" ]; then
+		if [ "$rd_generator" = initramfs-tools ]; then
 			if ! log-output -t base-installer apt-install busybox; then
 				db_subst base-installer/kernel/failed-package-install PACKAGE busybox
 				exit_error base-installer/kernel/failed-package-install
@@ -549,6 +549,29 @@
 			rm $QUEUEFILE
 			FIRSTMODULE=0
 		done
+
+		# Select and set driver inclusion policy for initramfs-tools
+		if [ "$rd_generator" = initramfs-tools ]; then
+			if db_get base-installer/initramfs-tools/driver-policy && \
+			   [ "$RET" = "" ]; then
+				# Get default for architecture
+				db_get base-installer/kernel/linux/initramfs-tools/driver-policy
+				db_set base-installer/initramfs-tools/driver-policy "$RET"
+			fi
+			db_input medium base-installer/initramfs-tools/driver-policy || true
+			if ! db_go; then
+				db_progress stop
+				exit 10
+			fi
+
+			db_get base-installer/initramfs-tools/driver-policy
+			cat > /target/etc/initramfs-tools/conf.d/driver-policy <<EOF
+# Driver inclusion policy selected during installation
+# Note: this setting overrides the value set in the file
+# /etc/initramfs-tools/initramfs.conf
+MODULES=$RET
+EOF
+		fi
 	else
 		info "Not installing an initrd generator."
 	fi

Attachment: signature.asc
Description: This is a digitally signed message part.


--- End Message ---
--- Begin Message --- 
Source: base-installer
Source-Version: 1.94

We believe that the bug you reported is fixed in the latest version of
base-installer, which is due to be installed in the Debian FTP archive:

base-installer_1.94.dsc
  to pool/main/b/base-installer/base-installer_1.94.dsc
base-installer_1.94.tar.gz
  to pool/main/b/base-installer/base-installer_1.94.tar.gz
base-installer_1.94_all.udeb
  to pool/main/b/base-installer/base-installer_1.94_all.udeb
bootstrap-base_1.94_armel.udeb
  to pool/main/b/base-installer/bootstrap-base_1.94_armel.udeb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 494466@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Martin Michlmayr <tbm@cyrius.com> (supplier of updated base-installer package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Sun, 24 Aug 2008 21:47:33 +0300
Source: base-installer
Binary: base-installer bootstrap-base
Architecture: source all armel
Version: 1.94
Distribution: unstable
Urgency: low
Maintainer: Debian Install System Team <debian-boot@lists.debian.org>
Changed-By: Martin Michlmayr <tbm@cyrius.com>
Description: 
 base-installer - base system installation framework (udeb)
 bootstrap-base - Install the base system (udeb)
Closes: 492751 494466
Changes: 
 base-installer (1.94) unstable; urgency=low
 .
   [ Frans Pop ]
   * Allow to select driver inclusion policy for initramfs-tools
     Closes: #494466
 .
   [ Martin Michlmayr ]
   * Use MODULES=dep on arm and armel.
 .
   [ Jérémy Bobbio ]
   * i386: Use -486 flavour for all CentaurHauls processors.  As they lack long
     NOP instructions, -686 kernels since 2.6.22+ fail to boot on these
     processors even if they claim 686 compatibility.  Closes: #492751
 .
   [ Updated translations ]
   * Arabic (ar.po) by Ossama M. Khayat
   * Bulgarian (bg.po) by Damyan Ivanov
   * Czech (cs.po) by Miroslav Kure
   * Greek, Modern (1453-) (el.po)
   * Esperanto (eo.po) by Felipe Castro
   * French (fr.po) by Christian Perrier
   * Galician (gl.po) by Jacobo Tarrio
   * Italian (it.po) by Milo Casagrande
   * Japanese (ja.po) by Kenshi Muto
   * Kurdish (ku.po) by Erdal Ronahi
   * Lithuanian (lt.po) by Kęstutis Biliūnas
   * Portuguese (pt.po) by Miguel Figueiredo
   * Portuguese (Brazil) (pt_BR.po) by Felipe Augusto van de Wiel (faw)
   * Romanian (ro.po) by Eddy Petrișor
   * Russian (ru.po) by Yuri Kozlov
   * Slovak (sk.po) by Ivan Masár
   * Thai (th.po) by Theppitak Karoonboonyanan
   * Turkish (tr.po) by Mert Dirik
   * Traditional Chinese (zh_TW.po) by Tetralet
Checksums-Sha1: 
 def7224f40f25a1dc28660adae8d6c19000ab24d 1091 base-installer_1.94.dsc
 4aa1dfd563643f23ab8569e590a23730055db445 227495 base-installer_1.94.tar.gz
 3e127eaa18f0380e9fc407e7ea42700b0273e4e1 39282 base-installer_1.94_all.udeb
 96c1e74c204c9abfb864f08870c9a65441d74848 140418 bootstrap-base_1.94_armel.udeb
Checksums-Sha256: 
 e82ac080ec6deaca522269c8b9e9de3a1e8f5d0f810555437f9504f5fe2b0aad 1091 base-installer_1.94.dsc
 46cbbb6c910dbebe266308e05955b029eee01ff4d79683523d25ca561ee9b71b 227495 base-installer_1.94.tar.gz
 97e6074913f16a496eadc48edd180b775dfa6b8c38b5728adf5ec1dc96f4bcea 39282 base-installer_1.94_all.udeb
 e2dd4b25e94fe9af480e68c6c7ac1c0573a2052122011f65ecafceb853f93e34 140418 bootstrap-base_1.94_armel.udeb
Files: 
 e7dad758136c5773ccb25db413aa90b3 1091 debian-installer required base-installer_1.94.dsc
 323de1c6150bf80c83920ac6885ca0b4 227495 debian-installer required base-installer_1.94.tar.gz
 06a1f4f60cd0e1aed0a8196021749589 39282 debian-installer required base-installer_1.94_all.udeb
 b1b9ac60b8a695604f6d8206edef9df4 140418 debian-installer required bootstrap-base_1.94_armel.udeb
Package-Type: udeb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkixtH8ACgkQKb5dImj9VJ82igCcDSgCZmUPVuFjHdJIKRka56lW
RJ8An0OdVtoZw5C0pd522MWBded+2c3g
=Hif4
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: