Bug 180765 - /etc/pm/hooks/50modules incorrectly quotes $SUSPEND_MODULES
Summary: /etc/pm/hooks/50modules incorrectly quotes $SUSPEND_MODULES
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pm-utils
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Phil Knirsch
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-02-10 04:57 UTC by Bojan Smojver
Modified: 2015-03-05 01:15 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-02-24 09:53:48 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Bojan Smojver 2006-02-10 04:57:10 UTC
Description of problem:

In the for loops of this script (only the resume loop really matters, as rmmod
appears to be doing the right thing), the $SUSPEND_MODULES variable is in double
quotes, which makes multiple modules specified in /etc/sysconfig/pm file appear
as one word, therefore not modprobing modules properly on resume.


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


How reproducible:
Always.


Steps to Reproduce:
1. Put at least two modules in /etc/sysconfig/pm file (e.g.
SUSPEND_MODULES="button natsemi"
2. Suspend using pm-hibernate.
3. Resume - modules won't be there.

  
Actual results:
Modules don't load, because modprobe thinks everything after the first module
names are parameters used for that module. Each module should be modprobed
separately.


Expected results:
Should load fine, which it does without double quotes.


Additional info:
N/A

Comment 1 Phil Knirsch 2006-02-24 09:53:48 UTC
Just verified this and removed the wrong double quotes from the script for the
resume and the suspend case as the manpage for rmmod doesn't mention that
multiple modules can be removed at once.

Fix should be in next build.

Read ya, Phil

Comment 2 Bojan Smojver 2006-02-27 09:48:43 UTC
Strangely enough, I didn't see that in Rawhide yet...


Note You need to log in before you can comment on or make changes to this bug.