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

Bug#764676: marked as done (debian-cd: Please use "---" not "--" on installer's kernel command line)

Your message dated Mon, 20 Apr 2015 12:03:33 +0000
with message-id <E1YkAQH-00060H-KO@franck.debian.org>
and subject line Bug#764676: fixed in debian-cd 3.1.17
has caused the Debian Bug report #764676,
regarding debian-cd: Please use "---" not "--" on installer's kernel command line
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

764676: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=764676
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: debian-installer-utils
X-Debbugs-CC: debian-kernel@lists.debian.org

(CCing debian-kernel just FYI, since I don't think this can/should be
fixed with a kernel change, likewise filing against
debian-installer-utils and not the kernel even though a kernel change
introduced the breakage)

A recent change to the kernel[0] (from v3.15 onwards) has changed the
way the kernel handles its command line, such that it now ignores
anything passed after a "--" marker. This has broken d-i's own use of
that marker which was to separate options intended for the installer
only (before the marker) from those which are intended to be both
consumed by the d-i kernel and propagated to the final installation
(after the marker, returned by the user-params utility).

It used to be that you could do:
        vmlinuz some/preseed=value -- console=ttyFOO
which would have the dual affect of having the kernel console (and hence
installer UI) run on ttyFOO and also, via grub-installer's use of
user-params, propagate the console=ttyFOO into the final grub config
(similarly for other bootloaders).

With the kernel change this no longer works -- the kernel doesn't put
its console on ttyFOO since it stops parsing at the --. So you get
silence on boot.

To get the old behaviour you need
        vmlinuz some/preseed=value console=ttyFOO -- console=ttyFOO
which is pretty tedious.

Just using
        vmlinuz some/preseed=value console=ttyFOO
doesn't propagate the console=ttyFOO to the installed system.

Since the kernel change was related to the "systemd abusing kernel
cmdline" debacle I'm not overly keen on raising this upstream and I
don't think that changing the kernel in a way which diverges from
upstream would not be the right approach here.

I don't know how widely used/documented/Supported this ability was, but
I thought e.g. the pxe and isolinux cfgs made use of it.

Not sure what we can do about this. Perhaps choose another separator
("=="?) and make user-params support both?


[0] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=51e158c12aca3c9ac63988611a97c05109b14dc9

--- End Message ---
--- Begin Message ---
Source: debian-cd
Source-Version: 3.1.17

We believe that the bug you reported is fixed in the latest version of
debian-cd, which is due to be installed in the Debian FTP archive.

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

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

Debian distribution maintenance software
Steve McIntyre <93sam@debian.org> (supplier of updated debian-cd 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@ftp-master.debian.org)

Hash: SHA256

Format: 1.8
Date: Mon, 20 Apr 2015 12:36:57 +0100
Source: debian-cd
Binary: debian-cd
Architecture: source all
Version: 3.1.17
Distribution: unstable
Urgency: medium
Maintainer: Debian CD Group <debian-cd@lists.debian.org>
Changed-By: Steve McIntyre <93sam@debian.org>
 debian-cd  - Tools for building (Official) Debian CD set
Closes: 764676 767253 772691 777711 782299
 debian-cd (3.1.17) unstable; urgency=medium
   [ Steve McIntyre ]
   * Provide hashes for uncompressed Translation-* files, and remove the
     uncompressed versions from the output disc. Closes: #767253. Initial
     patch from Michael Vogt - thanks!
   * Add 586 kernels to replace 486 for i386 CDs
   * More tweaks for arm64 CD building
   * Change from -- to --- for kernel command line handling.
     Closes: #764676
   * Add minor support / bug fixes for i386 EFI.
   * Add fine-grained control over usage of BIOS and EFI boot on x86 CDs,
     needed for extra Mac CD builds.
   * update-cd: if we're doing torrents, make sure we include update CD
     checksums in bt-$type too.
   * Improve autorun.inf, make sure the disc icon displays. Closes: #772691
     Thanks to jnqnfe for initial patch, slightly tweaked.
   * Add rdnssd on disc 1 by default. Closes: #782299
   * Update included files from the setup repo in the contrib dir
   [ Ben Hutchings ]
   * generate_di+k_list: Update kernel and related package lists for mips*
     Closes: #777711
   [ Helge Deller ]
   * [alpha] fix unpacking aboot-base, copy versioned vmlinuz file correctly.
 aa9004762ddfb90497cd97fb16b533a2f5485868 1711 debian-cd_3.1.17.dsc
 87003665f4f92014b88def6c99e236254013831e 725652 debian-cd_3.1.17.tar.xz
 f7fe516a6244e272962516fd6ae102fad5673bee 747282 debian-cd_3.1.17_all.deb
 63b0eb9316c4fcbad28d018a1bdb3228f1992a0c375009cfcc739caa6565e731 1711 debian-cd_3.1.17.dsc
 ac9d3fd87994e8ca98524bed0d0703d01b9c03df10c153e8db721ab4e3530446 725652 debian-cd_3.1.17.tar.xz
 cd3cd7199531e9e99771a87292e72e233c9d4b6c4dd78fdbe26241e7ebacf1f5 747282 debian-cd_3.1.17_all.deb
 2293af02e816a84a200d7c7cf9e37ccc 1711 admin extra debian-cd_3.1.17.dsc
 3c6085450403aca472cfae712fd9a4cf 725652 admin extra debian-cd_3.1.17.tar.xz
 0f48f4b5fdd3446af96df796fa3e97c0 747282 admin extra debian-cd_3.1.17_all.deb

Version: GnuPG v1.4.12 (GNU/Linux)


--- End Message ---

Reply to: