Please make powernow-k8 a module, to facilitate use of linux-phc

Bug #984288 reported by Ben A on 2012-04-17
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Medium
Tim Gardner
Precise
Medium
Tim Gardner
Quantal
Medium
Tim Gardner

Bug Description

Figured this is the best avenue as it would be a simple tweak for the maintainers.

The linux-phc project's phc-k8 module will work just fine on 3.2 kernels with some trivial tweaking. The catch is, the module needs to replace the kernel's own powernow-k8 driver to work correctly. In the generic Ubuntu kernel, powernow-k8 is builtin, necessitating a complete rebuild of the kernel. Were it a module instead, I could simply blacklist the module and be on my merry way.

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: linux-image-3.2.0-23-generic 3.2.0-23.36
ProcVersionSignature: Ubuntu 3.2.0-23.36-generic 3.2.14
Uname: Linux 3.2.0-23-generic x86_64
NonfreeKernelModules: nvidia wl
AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.24.
ApportVersion: 2.0.1-0ubuntu4
Architecture: amd64
AudioDevicesInUse:
 USER PID ACCESS COMMAND
 /dev/snd/controlC0: roothorick 2819 F.... pulseaudio
CRDA: Error: [Errno 2] No such file or directory
Card0.Amixer.info:
 Card hw:0 'NVidia'/'HDA NVidia at 0xc0000000 irq 17'
   Mixer name : 'Realtek ALC861-VD'
   Components : 'HDA:10ec0862,103c30e5,00100001 HDA:10573055,103c30e5,00100700'
   Controls : 26
   Simple ctrls : 14
Date: Tue Apr 17 13:56:22 2012
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Alpha amd64 (20111129.1)
MachineType: Hewlett-Packard HP Pavilion tx2000 Notebook PC
ProcEnviron:
 TERM=xterm
 PATH=(custom, no user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcFB: 0 VESA VGA
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.2.0-23-generic root=UUID=af115098-6412-4d55-82f1-6afe771c9fb1 ro quiet splash vt.handoff=7
RelatedPackageVersions:
 linux-restricted-modules-3.2.0-23-generic N/A
 linux-backports-modules-3.2.0-23-generic N/A
 linux-firmware 1.79
SourcePackage: linux
UpgradeStatus: No upgrade log present (probably fresh install)
dmi.bios.date: 04/18/2008
dmi.bios.vendor: Hewlett-Packard
dmi.bios.version: F.08
dmi.board.name: 30E5
dmi.board.vendor: Quanta
dmi.board.version: 88.0A
dmi.chassis.type: 10
dmi.chassis.vendor: Quanta
dmi.chassis.version: N/A
dmi.modalias: dmi:bvnHewlett-Packard:bvrF.08:bd04/18/2008:svnHewlett-Packard:pnHPPaviliontx2000NotebookPC:pvrRev1:rvnQuanta:rn30E5:rvr88.0A:cvnQuanta:ct10:cvrN/A:
dmi.product.name: HP Pavilion tx2000 Notebook PC
dmi.product.version: Rev 1
dmi.sys.vendor: Hewlett-Packard

Ben A (8-roothorick-gmail-com) wrote :
description: updated
Brad Figg (brad-figg) on 2012-04-17
Changed in linux (Ubuntu):
status: New → Confirmed
Joseph Salisbury (jsalisbury) wrote :

Can you provide some information on the status of this request with regards to getting it applied upstream? Has it been sent upstream, what sort of feedback has it received, is it getting applied to a subsystem maintainer's tree, etc?

Changed in linux (Ubuntu):
importance: Undecided → Medium
Ben A (8-roothorick-gmail-com) wrote :

...what? Maybe I wasn't all that clear. It's a config thing, not the kernel code itself; a simple matter of changing a "y" to an "m" in debian.master/config/config.common.ubuntu when building the kernel packages. (And making sure the module gets loaded, I suppose.)

Ben A (8-roothorick-gmail-com) wrote :

I think I see where you misread. I don't want linux-phc as part of Ubuntu itself (actually probably quite the opposite, it's advanced low-level stuff that by its very nature has a certain risk of system instability, and in the wrong hands, hardware damage), I just want a tiny tweak to the official kernels (specifically, making powernow-k8 a module instead of builtin) that would allow using the linux-phc modules via DKMS with the official kernel as-is.

tags: added: kernel-key
Tim Gardner (timg-tpi) on 2012-04-24
Changed in linux (Ubuntu Precise):
assignee: nobody → Tim Gardner (timg-tpi)
status: Confirmed → In Progress
Tim Gardner (timg-tpi) wrote :
Changed in linux (Ubuntu Precise):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Quantal):
status: New → Fix Committed
tags: added: patch
Luis Henriques (henrix) wrote :

This bug is awaiting verification that the kernel for Precise in -proposed solves the problem (3.2.0-24.38). Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-precise' to 'verification-done-precise'.

If verification is not done by one week from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-precise
Tim Gardner (timg-tpi) wrote :

Precise frequency governors are working as normal. Tagging as verification-done-precise.

tags: added: verification-done-precise
removed: verification-needed-precise
Ben A (8-roothorick-gmail-com) wrote :

Err... am I missing something? I have 3.2.0.24 #38 installed and powernow-k8 is still builtin. Is there some source build process I'm supposed to know about?

Tim Gardner (timg-tpi) wrote :

Ben - powernow-k8 is still built in, but you can cause it to not initialize by specifying the frequency governor of your choice as explained in the patch :

+ cpufreq_driver= [X86] Allow only the named cpu frequency scaling driver
+ to register. Example: cpufreq_driver=powernow-k8
+ Format: { none | STRING }
+

Specify 'cpufreq_driver=none' on the kernel command line to disallow _all_ frequency governors.

Ben A (8-roothorick-gmail-com) wrote :

I see. Not what I was expecting, but it works. I now have phc-k8 working with 3.2.0-24.38 official package and my VIDs are set the way I want them. Problem solved, thank you!

Changed in linux (Ubuntu Quantal):
assignee: nobody → Tim Gardner (timg-tpi)
importance: Undecided → Medium
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :
Download full text (10.6 KiB)

This bug was fixed in the package linux - 3.2.0-24.38

---------------
linux (3.2.0-24.38) precise-proposed; urgency=low

  [ Luis Henriques ]

  * Release Tracking Bug
    - LP: #991925

  [ Herton Ronaldo Krzesinski ]

  * linux: add Build-Depends for libnewt-dev, to enable perf TUI support
    - LP: #981717

  [ Tim Gardner ]

  * SAUCE: Allow filtering of cpufreq drivers
    - LP: #984288

  [ Upstream Kernel Changes ]

  * x86 bpf_jit: fix a bug in emitting the 16-bit immediate operand of AND
    - LP: #981162
  * tg3: Fix 5717 serdes powerdown problem
    - LP: #981162
  * sky2: dont overwrite settings for PHY Quick link
    - LP: #981162
  * rose_dev: fix memcpy-bug in rose_set_mac_address
    - LP: #981162
  * net: usb: cdc_eem: fix mtu
    - LP: #981162
  * Fix non TBI PHY access; a bad merge undid bug fix in a previous commit.
    - LP: #981162
  * ASoC: wm8994: Update WM8994 DCS calibration
    - LP: #981162
  * mtd: ixp4xx: oops in ixp4xx_flash_probe
    - LP: #981162
  * mtd: mips: lantiq: reintroduce support for cmdline partitions
    - LP: #981162
  * mtd: nand: gpmi: use correct member for checking NAND_BBT_USE_FLASH
    - LP: #981162
  * mtd: sst25l: initialize writebufsize
    - LP: #981162
  * mtd: block2mtd: initialize writebufsize
    - LP: #981162
  * mtd: lart: initialize writebufsize
    - LP: #981162
  * mtd: m25p80: set writebufsize
    - LP: #981162
  * ACPI: Do cpufreq clamping for throttling per package v2
    - LP: #981162
  * PNPACPI: Fix device ref leaking in acpi_pnp_match
    - LP: #981162
  * modpost: fix ALL_INIT_DATA_SECTIONS
    - LP: #981162
  * genirq: Adjust irq thread affinity on IRQ_SET_MASK_OK_NOCOPY return
    value
    - LP: #981162
  * tracing: Fix ftrace stack trace entries
    - LP: #981162
  * tracing: Fix ent_size in trace output
    - LP: #981162
  * m68k/mac: Add missing platform check before registering platform
    devices
    - LP: #981162
  * mac80211: fix possible tid_rx->reorder_timer use after free
    - LP: #981162
  * rtlwifi: rtl8192ce: rtl8192cu: rtl8192de: Fix low-gain setting when
    scanning
    - LP: #981162
  * drm: Validate requested virtual size against allocated fb size
    - LP: #981162
  * drm/radeon/kms: fix fans after resume
    - LP: #981162
  * drm/i915: no-lvds quirk on MSI DC500
    - LP: #981162
  * drm/i915: Add lock on drm_helper_resume_force_mode
    - LP: #981162
  * drm/i915: quirk away broken OpRegion VBT
    - LP: #981162
  * r8169: runtime resume before shutdown.
    - LP: #981162
  * target: Fix unsupported WRITE_SAME sense payload
    - LP: #981162
  * kgdb,debug_core: pass the breakpoint struct instead of address and
    memory
    - LP: #981162
  * kgdbts: Fix kernel oops with CONFIG_DEBUG_RODATA
    - LP: #981162
  * kgdbts: (1 of 2) fix single step awareness to work correctly with SMP
    - LP: #981162
  * kgdbts: (2 of 2) fix single step awareness to work correctly with SMP
    - LP: #981162
  * x86,kgdb: Fix DEBUG_RODATA limitation using text_poke()
    - LP: #981162
  * CIFS: Fix VFS lock usage for oplocked files
    - LP: #981162
  * ARM: tegra: remove Tegra30 errata from MACH_TEGRA_DT
    - LP: #981162
  * mmc: sdhci-dove: Fix compile error by...

Changed in linux (Ubuntu Precise):
status: Fix Committed → Fix Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

To post a comment you must log in.