xfs handling on battery not working correctly

Bug #645974 reported by Andre D
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pm-utils
Won't Fix
Medium
pm-utils (Ubuntu)
Fix Released
Low
Bryce Harrington

Bug Description

Binary package hint: pm-utils

The file /usr/lib/pm-utils/power.d/xfs_buffer has a typo, preventing /proc/sys/fs/xfs/xfssyncd_centisecs and xfsbufd_centisecs from being updated correctly when on battery power.

Currently, it has a function like this:

xfs_battery() {
    state_exists xfs_buffer_default || \
        read_values |savestate xfs_buffer_default
    write_values "$XFS_AGE_BUFFER" "$XFS_BUFD" "$XFS_SYNCD"
}

Whereas it should be:

xfs_battery() {
    state_exists xfs_buffer_default || \
        read_values |savestate xfs_buffer_default
    write_values "$XFS_AGE_BUFFER" "$XFS_SYNCD" "$XFS_BUFD"
}

Patch is attached.

This is on maverick beta with latest updates (pm-utils 1.4.1-2)

Related branches

Revision history for this message
Andre D (ad-andred) wrote :
Bryce Harrington (bryce)
Changed in pm-utils (Ubuntu):
status: New → Triaged
importance: Undecided → Low
assignee: nobody → Bryce Harrington (bryce)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pm-utils - 1.4.1-8ubuntu2

---------------
pm-utils (1.4.1-8ubuntu2) precise; urgency=low

  * Add 91-xfs-buffer.patch: Use correct argument order when setting
    XFS bufd and syncd while on battery. (Thanks to Andre D)
    (LP: #645974)
 -- Bryce Harrington <email address hidden> Mon, 07 Nov 2011 18:34:40 -0800

Changed in pm-utils (Ubuntu):
status: Triaged → Fix Released
Revision history for this message
In , Martin Pitt (pitti) wrote :

pm/power.d/xfs_buffer:

xfsfiles="age_buffer_centisecs xfssyncd_centisecs xfsbufd_centisecs"
[...]
    write_values "$XFS_AGE_BUFFER" "$XFS_BUFD" "$XFS_SYNCD"

That call writes the values the wrong way around.

Revision history for this message
In , Martin Pitt (pitti) wrote :

Created attachment 55377
pm/power.d/xfs_buffer: Fix wrong argument ordering

Patch originally created by Andre Draszik in https://bugs.launchpad.net/pm-utils/+bug/645974.

Changed in pm-utils:
importance: Unknown → Medium
status: Unknown → Confirmed
Revision history for this message
In , Ajax-a (ajax-a) wrote :

pm-utils hasn't been touched in eight years, none of this is likely to get addressed. Closing bugs and disabling the bz product.

Changed in pm-utils:
status: Confirmed → Won't Fix
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.