Bug 54181

Summary: apm bios is never turned on if the kernel switch apm=on is specified.
Product: [Retired] Red Hat Raw Hide Reporter: Shinya Narahara <naraha_s>
Component: kernelAssignee: Dave Jones <davej>
Status: CLOSED CURRENTRELEASE QA Contact: Brock Organ <borgan>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.0CC: pfrields
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-10-30 03:42:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Shinya Narahara 2001-10-01 00:24:03 UTC
From Bugzilla Helper:
User-Agent: Mozilla/4.75 [ja] (WinNT; U)

Description of problem:
If the kernel switch "apm=on" is specified, the apm bios isnot turned on
because of kernel bug, probably from kernel-2.4.9-ac#.


Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1.specify kernel switch "apm=on" on lilo/grub configuration file.
2.reboot.
3.


Actual Results:  The file /proc/apm isn't appeared and apmd never work.


Expected Results:  The file /proc/apm is appeared and apmd works normally.

Additional info:

This is just a kernel bug. If you don't specify the switch, the apm bios is turned on normally.
In kernel 2.4.9-ac# and 2.4.10, if the switch is specified, the value apm_disabled is set as 0 in
function apm_setup(). But in function apm_init(), the value is checked whether it is -1 or not.
Quick hack for kernel-2.4.10 is below:

--- linux/arch/i386/kernel/apm.c	Tue Sep 18 14:52:35 2001
+++ linux/arch/i386/kernel/apm.c.s_nara	Sun Sep 30 20:42:18 2001
@@ -1589,7 +1589,7 @@
 		if (strncmp(str, "off", 3) == 0)
 			apm_disabled = 1;
 		if (strncmp(str, "on", 2) == 0)
-			apm_disabled = 0;
+			apm_disabled = -1;
 		if ((strncmp(str, "allow-ints", 10) == 0) ||
 		    (strncmp(str, "allow_ints", 10) == 0))
  			apm_info.allow_ints = 1;

Comment 1 Arjan van de Ven 2001-10-01 10:32:10 UTC
Thanks for the bugreport; looks indeed a bug.