[Pm-utils] pm-utils 1.3.0-rc3 available for testing
victor.lowther at gmail.com
Tue Feb 16 12:23:30 PST 2010
On Wed, Feb 3, 2010 at 3:30 PM, Michael Biebl <mbiebl at gmail.com> wrote:
> Hi Victor!
> 2010/1/15 Michael Biebl <mbiebl at gmail.com>:
> > 2009/12/14 Victor Lowther <victor.lowther at gmail.com>:
> >> On Thu, 2009-12-10 at 23:16 -0600, Victor Lowther wrote:
> >>> On Sat, 2009-12-05 at 22:24 -0600, Victor Lowther wrote:
> >>> > PM-utils 1.3.0-rc1 is not available for testing. Notable changes
> >>> > the pm-utils 1.2 series:
> >>> Fresh after -rc1, -rc2 is out. This includes several bugfixes forward
> >>> ported from the soon-to-be-released pm-utils-1.2.7, along with a new
> >>> --quirk-test parameter that bypasses the sanity checking imposed by the
> >>> quirk database handler -- now you too can see what happens when you
> >>> your video card in userspace while running a KMS driver!
> >> Now at 1.3.0-rc3! What should have been a tiny bugfix in is_set ended
> >> up breaking everything.
> > Here is some feedback/suggestions
> > 1.) suspend/hibernate broke for me. I got this in pm-suspend.log
> > /usr/lib/pm-utils/sleep.d/98-video-quirk-db-handler suspend
> > suspend:/usr/lib/pm-utils/sleep.d/98-video-quirk-db-handler: line 225:
> > ((: 0x10de == : syntax error: operand expected (error token is "==
> > ")
> > success.
> > The result was, that no quirks were applied. I'm having bash 4.1
> Thanks to some tireless debugging of Martin Pitt, we were able to
> track this down and push some important fixes to make it actually work
> for systems using quirks
> One issue we noticed is, that the pm-utils quirksdb uses ";" as
> separator in its match targets.
> We were wondering, if we shouldn't just use "|" directly.
> import-fdi-quirkdb makes one suggest that is the intention, but
> apparently what ends up in the *.quirkdb files is ";"
> Victor, what's your take on this? See also
Yeah, I intended | to be used where an or seperator was required (match this
or that). The only time i did not to that was where the data to be matched
was purely numeric, but we should use | there as well.
> > 2.) Could we please rename 98-video-quirk-db-handler to
> > for consistencies sake
> I just went ahead and did that.
> > 3.) It would be great, if 98-video-quirk-db-handler was a /bin/sh script.
> With the advanced features used in there, that is probably not easily
> possible, so let's skip that for now.
Oh, it could be done, but right now all the regex matching relies heavily on
using bash native regex handling -- in theory egrep would also work without
making any modifications to the regexes in the .quirkdb files, but that
would involve forking lots of egreps when scanning through the files.
> > 4.) I think we should do a one-time conversion of hal-infos' suspend
> > quirks and ship that within pm-utils.
> Aside from that, having an updated pm-quirks tarball based on the
> latest hal-info git master would be nice.
I was planning on making it a separate tarball.
> Victor, do you have time to roll a new, up-to-date of pm-quirks (maybe
> including that in pm-utils directly)
> and also a 1.3.0-rc4 with the fixes that have been committed.
Sure. I apologize for not having done so sooner, but 10 hour days and
sleep-averse toddlers do not make for the best conditions to code in.
> Why is it that all of the instruments seeking intelligent life in the
> universe are pointed away from Earth?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hal