Ubuntu
kdump-tools package

Remove obsolete "nousb" option in kdump command-line for newer kernels

  1. Xenial (16.04)
  2. Bug #1800562
Bug #1800562 reported by Guilherme G. Piccoli
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
kdump-tools (Ubuntu)
Xenial
New
Undecided
Unassigned
Bionic
New
Undecided
Unassigned
Eoan
New
Undecided
Unassigned
Focal
New
Undecided
Unassigned
Groovy
New
Undecided
Unassigned
makedumpfile (Ubuntu)
Confirmed
Medium
Guilherme G. Piccoli
Xenial
Won't Fix
Medium
Guilherme G. Piccoli
Bionic
Confirmed
Medium
Guilherme G. Piccoli
Eoan
Won't Fix
Medium
Guilherme G. Piccoli
Focal
Confirmed
Medium
Guilherme G. Piccoli
Groovy
Won't Fix
Medium
Guilherme G. Piccoli

Bug Description

[Impact]
* Kdump command-line include an obsolete "nousb" parameter by default, which can cause a misimpression: users will think they are not booting with USB, but they are.

* Since kernel v4.5, the correct parameter to disable USB subsystem initialization is "usbcore.nousb" always (instead of "nousb" in case the subsystem is built-in). This was changed by commit 097a9ea0e48 ("usb: make "nousb" a clear module parameter").

* USB may be pretty essential in case for example kdump users need to decrypt a disk under LUKS, and there's only an USB keyboard connected to the system. Given the option is innocuous since Bionic, we should just drop it to prevent confusion.

[Test Case]

1) Deploy a Disco VM e.g. with uvt-kvm
2) Install the kdump-tools package
3) Run `kdump-config test`and check for the 'nousb' parameter:

$ kdump-config test
...
kexec command to be used:
  /sbin/kexec -p --command-line="BOOT_IMAGE=/boot/vmlinuz-4.15.0-45-generic root=LABEL=cloudimg-rootfs ro console=tty1 console=ttyS0 nr_cpus=1 systemd.unit=kdump-tools.service irqpoll nousb ata_piix.prefer_ms_hyperv=0" /var/lib/kdump/vmlinuz

[Regression Potential]

The regression potential is extremely low, since the "nousb" parameter is not used since Bionic although is there. Any bugs we would have by changing this are still valid by not removing the option - the semantics with or without "nosub" is the same since from Bionic.

NOTICE we won't change Xenial, it can use kernel 4.4 which indeed disables USB by taking the "nousb" parameter.

See original description
Tags: seg
Guilherme G. Piccoli (gpiccoli)
affects: linux (Ubuntu) → makedumpfile (Ubuntu)
Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Cosmic):
status: New → Confirmed
Changed in makedumpfile (Ubuntu Bionic):
status: New → Confirmed
Changed in makedumpfile (Ubuntu Xenial):
status: New → Confirmed
Changed in makedumpfile (Ubuntu Cosmic):
importance: Undecided → High
Changed in makedumpfile (Ubuntu Bionic):
importance: Undecided → High
Changed in makedumpfile (Ubuntu Xenial):
importance: Undecided → High
assignee: nobody → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Bionic):
assignee: nobody → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Cosmic):
assignee: nobody → Guilherme G. Piccoli (gpiccoli)
Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Xenial):
assignee: Guilherme G. Piccoli (gpiccoli) → Heitor R. Alves de Siqueira (halves)
Changed in makedumpfile (Ubuntu Bionic):
assignee: Guilherme G. Piccoli (gpiccoli) → Heitor R. Alves de Siqueira (halves)
Changed in makedumpfile (Ubuntu Cosmic):
assignee: Guilherme G. Piccoli (gpiccoli) → Heitor R. Alves de Siqueira (halves)
Changed in makedumpfile (Ubuntu Disco):
assignee: Guilherme G. Piccoli (gpiccoli) → Heitor R. Alves de Siqueira (halves)
Heitor Alves de Siqueira (halves)
description: updated
description: updated
Heitor Alves de Siqueira (halves)
description: updated
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "debdiff for disco" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu Disco):
status: Confirmed → In Progress
Changed in makedumpfile (Ubuntu Cosmic):
status: Confirmed → In Progress
Changed in makedumpfile (Ubuntu Bionic):
status: Confirmed → In Progress
Changed in makedumpfile (Ubuntu Xenial):
status: Confirmed → In Progress
Revision history for this message
Thadeu Lima de Souza Cascardo (cascardo) wrote :

Have you seen this cause any issue when kdumping from bionic or any releases later than xenial?

Because we should not disable usb, and I would apply the opposite fix and just remove "nousb" from the command line.

The reason for this is supporting kdump on encrypted systems, where the user must type a password and use a USB keyboard.

Cascardo.

Changed in makedumpfile (Ubuntu Disco):
status: In Progress → Opinion
Revision history for this message
Heitor Alves de Siqueira (halves) wrote :

Thanks for bringing that up, Cascardo. That was one scenario we didn't consider, and you're right about it causing problems for users with encrypted systems.

It does seem more sensible to just remove the 'nousb' option in this case, since this parameter has no effect in recent kernels. I'll respin the patches and change this LP to better reflect this.
Thanks!

Heitor Alves de Siqueira (halves)
summary: - Replace "nousb" option in kdump command-line for the newer
- "usbcore.nousb"
+ Remove obsolete "nousb" option in kdump command-line for newer kernels
Heitor Alves de Siqueira (halves)
description: updated
Revision history for this message
Simon Quigley (tsimonq2) wrote :

Unsubscribing the Ubuntu Sponsors Team as there is nothing to sponsor.

Heitor Alves de Siqueira (halves)
tags: removed: patch sts
Guilherme G. Piccoli (gpiccoli)
tags: added: seg
Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Xenial):
status: In Progress → Won't Fix
Changed in makedumpfile (Ubuntu Disco):
status: Opinion → Confirmed
Changed in makedumpfile (Ubuntu Eoan):
status: Opinion → Confirmed
Changed in makedumpfile (Ubuntu Xenial):
assignee: Heitor Alves de Siqueira (halves) → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Bionic):
assignee: Heitor Alves de Siqueira (halves) → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Cosmic):
assignee: Heitor Alves de Siqueira (halves) → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Disco):
assignee: Heitor Alves de Siqueira (halves) → Guilherme G. Piccoli (gpiccoli)
Changed in makedumpfile (Ubuntu Eoan):
assignee: Heitor Alves de Siqueira (halves) → Guilherme G. Piccoli (gpiccoli)
Guilherme G. Piccoli (gpiccoli)
description: updated
Revision history for this message
Guilherme G. Piccoli (gpiccoli) wrote :

This is the debdiff containing the proposed solution for this LP.
I'd like to thanks specially Cascardo and Heitor for the fruitful discussions with regards the direction we should take in this LP.

Changed in makedumpfile (Ubuntu Cosmic):
status: In Progress → Confirmed
Changed in makedumpfile (Ubuntu Eoan):
status: Confirmed → In Progress
Changed in makedumpfile (Ubuntu Bionic):
status: In Progress → Confirmed
Mathew Hodson (mhodson)
Changed in makedumpfile (Ubuntu Cosmic):
status: Confirmed → Won't Fix
tags: added: patch
Revision history for this message
Steve Langasek (vorlon) wrote :

Ubuntu 19.04 reached end of life on January 23, 2020. I am therefore marking this bug 'wontfix' for that release.

Changed in makedumpfile (Ubuntu Disco):
status: Confirmed → Won't Fix
Guilherme G. Piccoli (gpiccoli)
no longer affects: makedumpfile (Ubuntu Cosmic)
no longer affects: makedumpfile (Ubuntu Disco)
Changed in makedumpfile (Ubuntu Focal):
status: New → Confirmed
Changed in makedumpfile (Ubuntu Eoan):
status: In Progress → Confirmed
Changed in makedumpfile (Ubuntu Groovy):
status: In Progress → Confirmed
Changed in makedumpfile (Ubuntu Focal):
importance: Undecided → Medium
Changed in makedumpfile (Ubuntu Groovy):
importance: High → Medium
Changed in makedumpfile (Ubuntu Eoan):
importance: High → Medium
Changed in makedumpfile (Ubuntu Bionic):
importance: High → Medium
Changed in makedumpfile (Ubuntu Xenial):
importance: High → Medium
Changed in makedumpfile (Ubuntu Focal):
assignee: nobody → Guilherme G. Piccoli (gpiccoli)
Revision history for this message
Brian Murray (brian-murray) wrote :

The Eoan Ermine has reached end of life, so this bug will not be fixed for that release

Changed in makedumpfile (Ubuntu Eoan):
status: Confirmed → Won't Fix
Revision history for this message
Brian Murray (brian-murray) wrote :

The Groovy Gorilla has reached end of life, so this bug will not be fixed for that release

Changed in makedumpfile (Ubuntu Groovy):
status: Confirmed → Won't Fix
dann frazier (dannf)
no longer affects: kdump-tools (Ubuntu)
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "lp1800562_eoan.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

Graham Inggs (ginggs)
tags: removed: patch
Revision history for this message
Graham Inggs (ginggs) wrote :

"patch" flag removed from the attachment
"patch" tag removed
~ubuntu-sponsors unsubscribed

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.