Re: [UBUNTU:acpi/ec] Use semaphore instead of spinlock

From: Voluspa
Date: Thu Jun 15 2006 - 07:03:02 EST


On Wed, 14 Jun 2006 18:14:54 -0700 Randy Dunlap wrote:
> updated version:

As a user, it's great if this fixes people's keyboards and mice. But it's
not a panacea. Gkrellm reads CPU temperatures from
/proc/acpi/thermal_zone/*/temperature and that disturbs a time-critical
application like mplayer, both when reading normal video and hacked mms:
sound streams (ogg sound is OK, though):

rtc: lost some interrupts at 1024Hz.
rtc: lost some interrupts at 1024Hz.

The loss frequency is much lower with this patch, by something like an
estimated 5-10 times.

Also, here the notebook's touchpad gets clobbered by temperature reads
when dragging a small xterm window around:

psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: issuing reconnect request
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 4
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 - driver resynched.
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 4
psmouse.c: TouchPad at isa0060/serio4/input0 lost sync at byte 1
psmouse.c: TouchPad at isa0060/serio4/input0 - driver resynched.

So, here temperature reading stays off even if this patch hits mainline.
Battery reading is fine, strangely enough. Perhaps because I've set
a period of 30 seconds between updates.

Or it might be that a lot of work has gone in regarding acpi BAT...

Mvh
Mats Johannesson
--
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/