The patch looks ok to me, although I think that we could make it much simpler by ripping out the acpi-support fallback entirely (which is totally unsupported and broken now), and just call pm-{suspend,hibernate}. pm-utils makes sure to read the necessary quirks itself, so we don't even need the d-bus magic.
Not calling hal will also be more robust for the future when hal will be deprecated in favor of devicekit-power.
The patch looks ok to me, although I think that we could make it much simpler by ripping out the acpi-support fallback entirely (which is totally unsupported and broken now), and just call pm-{suspend, hibernate} . pm-utils makes sure to read the necessary quirks itself, so we don't even need the d-bus magic.
Not calling hal will also be more robust for the future when hal will be deprecated in favor of devicekit-power.