Bug 9690 - [palmax] touchscreen logic bug prevents use in low loads
Summary: [palmax] touchscreen logic bug prevents use in low loads
Status: RESOLVED WONTFIX
Alias: None
Product: xorg
Classification: Unclassified
Component: Input/palmax (show other bugs)
Version: 7.0 (2005.12)
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL: http://bugs.debian.org/cgi-bin/bugrep...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-17 11:00 UTC by Brice Goglin
Modified: 2010-11-20 11:49 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Fix processing under low load (and fix a typo) (869 bytes, patch)
2007-01-17 11:01 UTC, Brice Goglin
no flags Details | Splinter Review

Description Brice Goglin 2007-01-17 11:00:19 UTC
Copy of the Debian bug explanation:
"The palmax touchscreen driver contains a bug related to
the parsing of data from the serial port. The problem 
occurs when less than five bytes have been read after the
current package to be processed, a very frequent case
except when the system is under high load. The result is
that touchscreen data is discarded instead of parsed,
rendering the touchscreen unresponsive. 

I have found the faulty logic and am attaching a patch 
which seems to solve the problem correctly for me, though
it could probably use a second set of eyes to check for 
other thinkos. 

I am using a Palmax PD-1100, and with this patch applied
my touchscreen seems to work fine."


This bug has been reported against to the Debian BTS 2 years ago against
xserver-xfree86, and again 3 months ago against xorg 7.0 (by the same person).
The same patch fixed the bug in both releases. Upstream did not change since 7.0
got released, so I guess the bug is still present.

By the way, the following patch also fixes a typo.
Comment 1 Brice Goglin 2007-01-17 11:01:18 UTC
Created attachment 8434 [details] [review]
Fix processing under low load (and fix a typo)
Comment 2 Daniel Stone 2007-02-27 01:35:49 UTC
Sorry about the phenomenal bug spam, guys.  Adding xorg-team@ to the QA contact so bugs don't get lost in future.
Comment 3 Jesse Adkins 2010-11-20 11:49:03 UTC
As of Feb 17 2009, this driver is set to fail in configure.ac because it does not have a maintainer. When a maintainer steps up for palmax, they can reopen this.

Until then, I'm closing it.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.