header_postinst_hook should use /etc/kernel/header_postinst.d instead

Bug #120049 reported by Matt Domsch
4
Affects Status Importance Assigned to Milestone
kernel-package (Ubuntu)
Confirmed
Medium
Unassigned
linux-source-2.6.22 (Ubuntu)
Fix Released
Medium
Ben Collins

Bug Description

The linux-image package has preinst and postinst scripts that let other packages hook in, by placing a script in /etc/kernel/preinst.d/ and /etc/kernel/postinst.d/.
The linux-headers package also has a way for other packages to hook in, but it requires editing /etc/kernel-img.conf and adding a header_postinst_hook=... line, which only allows for a single application to hook there.

Dell DKMS 2.0.17 will make use of the /etc/kernel/preinst.d/ and postinst.d/ directories to be aware of linux-image package updates, but it also needs to hook linux-headers postinst. It would be nice for linux-headers to have a similar directory mechanism, ala /etc/kernel/header_preinst.d/ and header_postinst.d/, rather than needing to edit /etc/kernel-img.conf.

Revision history for this message
Matt Domsch (matt-domsch) wrote :
Revision history for this message
Ben Collins (ben-collins) wrote :

This actually needs to be handled in kernel-package for feisty.

In gutsy, it's controlled by linux-source itself.

Changed in linux-source-2.6.20:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Changed in linux-source-2.6.22:
assignee: nobody → ubuntu-kernel-team
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Amit Bhutani (amit.bhutani) wrote :

Has this been resolved? Status says "Triaged". What exactly does this mean?

Revision history for this message
Tim Gardner (timg-tpi) wrote :
Changed in kernel-package:
status: Triaged → Invalid
Changed in linux-source-2.6.22:
status: Triaged → Invalid
Changed in redfish:
status: New → Invalid
Revision history for this message
Tim Gardner (timg-tpi) wrote :

Need to rethink this.

Changed in linux-source-2.6.22:
assignee: ubuntu-kernel-team → timg-tpi
status: Invalid → Confirmed
Revision history for this message
Matt Domsch (matt-domsch) wrote :

Patch to kernel-package to make linux-headers search in /etc/kernel/header_postinst.d/ and /etc/kernel/src_postinst.d/ and related .d/$version/ directories for files to execute rather than relying solely upon the header_postinst_hook variable.

Changed in dell:
importance: Undecided → Medium
status: Invalid → Triaged
Changed in kernel-package:
status: Invalid → Confirmed
Revision history for this message
Matt Domsch (matt-domsch) wrote :

same patch applies to gutsy and feisty both.

Revision history for this message
Ben Collins (ben-collins) wrote :

Patch applied to headers-postinst hook in ubuntu-gutsy.git

Changed in linux-source-2.6.22:
assignee: timg-tpi → ben-collins
status: Confirmed → Fix Committed
Revision history for this message
Ben Collins (ben-collins) wrote :

linux-source-2.6.22 (2.6.22-11.34) gutsy; urgency=low

  [Alan Stern]

  * USB: disable autosuspend by default for non-hubs
    - LP: #85488

  [Ben Collins]

  * ubuntu: Enable LEDS_TRIGGERS and related options
    - Needed for iwlwifi
  * ubuntu: Add real ABI files for virtual flavour
  * ubuntu: Re-enable missing CONFIG_SERPENT for hppa64
    - Noticed by Lamont
  * ubuntu: Add linux-headers postinst to handle hooks
    - LP: #125816
  * ubuntu: Add support for /etc/kernel/headers_postinst.d/ to
    headers-postinst
    - LP: #120049
  * cell: Add binary-custom flavour "cell" to support ps3

  [Mattia Dongili]

  * sony-laptop: restore the last user requested brightness level on
    resume.
    - LP: #117331

  [Tejun Heo]

  * ata_piix: fix suspend/resume for some TOSHIBA laptops
    - LP: #139045
  * PCI: export __pci_reenable_device()
    - needed for ata_piix change

  [Tim Gardner]

  * Enable Sierra Wireless MC8775 0x6813
    - LP: #131167

  [Zhang Rui]

  * ACPI: work around duplicate name "VID" problem on T61
    - Noted by mjg59

 -- Ben Collins <email address hidden> Sun, 16 Sep 2007 22:31:47 -0400

Changed in linux-source-2.6.22:
status: Fix Committed → Fix Released
Revision history for this message
Jose De la Rosa (jose-de-la-rosa) wrote :

Closing, per Ben Collins' comments on 9/19/07

Changed in dell:
status: Triaged → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote : Kernel team bugs

Per a decision made by the Ubuntu Kernel Team, bugs will longer be assigned to the ubuntu-kernel-team in Launchpad as part of the bug triage process. The ubuntu-kernel-team is being unassigned from this bug report. Refer to https://wiki.ubuntu.com/KernelTeamBugPolicies for more information. Thanks.

Changed in somerville:
importance: Undecided → Medium
status: New → Fix Released
no longer affects: dell
Revision history for this message
Timothy R. Chavez (timrchavez) wrote :

The bug task for the somerville project has been removed by an automated script. This bug has been cloned on that project and is available here: https://bugs.launchpad.net/bugs/1306152

no longer affects: somerville
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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