Bug 213957 - Subsequent left/right mouse scroll singals are lost unless the mouse is moved
Summary: Subsequent left/right mouse scroll singals are lost unless the mouse is moved
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: usb (show other bugs)
Version: CURRENT
Hardware: Any Any
: --- Affects Only Me
Assignee: Hans Petter Selasky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-31 22:05 UTC by Yuri Victorovich
Modified: 2016-11-07 08:19 UTC (History)
1 user (show)

See Also:

Attachments
Generate extra button release event for T-axis. (605 bytes, patch)
2016-10-31 22:58 UTC, Hans Petter Selasky 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer freebsd_triage 2016-10-31 22:05:01 UTC 
Microsoft explorer mouse https://www.amazon.com/Microsoft-Explorer-Touch-Mouse-Black/dp/B00519UZM8 sends left/right scroll signals. These signals were mapped to button numbers 8/9 before r305828, and to 10/11 after that revision.

Subsequent left/right signals are lost unless the mouse is moved. I can see that interrupts arrive with hw.usb.ums.debug=16 debug log, but xev only shows a new ButtonPress event when the mouse is moved. This has been this way forever.

So I think the bug is in ums.c
Comment 1 Hans Petter Selasky freebsd_committer freebsd_triage 2016-10-31 22:50:32 UTC 
Hi,

Can you check what is configured in /etc/X11/Xorg.conf ?

Are you aware about the following article?

http://askubuntu.com/questions/331508/buttonmapping-positional-parameters



--HPS
Comment 2 Hans Petter Selasky freebsd_committer freebsd_triage 2016-10-31 22:58:38 UTC 
Created attachment 176355 [details]
Generate extra button release event for T-axis.

Does the attached patch make any difference?

--HPS
Comment 3 Yuri Victorovich freebsd_committer freebsd_triage 2016-10-31 23:05:33 UTC 
The problem is gone with this patch. Thanks!
Comment 4 commit-hook freebsd_committer freebsd_triage 2016-11-01 07:15:45 UTC 
A commit references this bug:

Author: hselasky
Date: Tue Nov  1 07:15:40 UTC 2016
New revision: 308165
URL: https://svnweb.freebsd.org/changeset/base/308165

Log:
  Make sure the virtual T-axis buttons generate button release event(s)
  for continuous tilting.

  PR:		213957
  MFC after:	3 days

Changes:
  head/sys/dev/usb/input/ums.c
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-11-07 08:15:24 UTC 
A commit references this bug:

Author: hselasky
Date: Mon Nov  7 08:14:29 UTC 2016
New revision: 308394
URL: https://svnweb.freebsd.org/changeset/base/308394

Log:
  MFC r308144 and r308165:
  Fixes for virtual T-axis buttons.

  Make sure the virtual T-axis buttons gets cleared for USB mice which has
  less than 6 buttons.

  Make sure the virtual T-axis buttons generate button release event(s)
  for continuous tilting.

  PR:		213919
  PR:		213957

Changes:
_U  stable/11/
  stable/11/sys/dev/usb/input/ums.c
Comment 6 commit-hook freebsd_committer freebsd_triage 2016-11-07 08:16:27 UTC 
A commit references this bug:

Author: hselasky
Date: Mon Nov  7 08:15:58 UTC 2016
New revision: 308395
URL: https://svnweb.freebsd.org/changeset/base/308395

Log:
  MFC r308144 and r308165:
  Fixes for virtual T-axis buttons.

  Make sure the virtual T-axis buttons gets cleared for USB mice which has
  less than 6 buttons.

  Make sure the virtual T-axis buttons generate button release event(s)
  for continuous tilting.

  PR:	213919
  PR:	213957

Changes:
_U  stable/10/
  stable/10/sys/dev/usb/input/ums.c
Comment 7 commit-hook freebsd_committer freebsd_triage 2016-11-07 08:17:30 UTC 
A commit references this bug:

Author: hselasky
Date: Mon Nov  7 08:17:24 UTC 2016
New revision: 308396
URL: https://svnweb.freebsd.org/changeset/base/308396

Log:
  MFC r308144 and r308165:
  Fixes for virtual T-axis buttons.

  Make sure the virtual T-axis buttons gets cleared for USB mice which has
  less than 6 buttons.

  Make sure the virtual T-axis buttons generate button release event(s)
  for continuous tilting.

  PR:	213919
  PR:	213957

Changes:
_U  stable/9/sys/
  stable/9/sys/dev/usb/input/ums.c
Comment 8 commit-hook freebsd_committer freebsd_triage 2016-11-07 08:19:33 UTC 
A commit references this bug:

Author: hselasky
Date: Mon Nov  7 08:18:51 UTC 2016
New revision: 308397
URL: https://svnweb.freebsd.org/changeset/base/308397

Log:
  MFC r308144 and r308165:
  Fixes for virtual T-axis buttons.

  Make sure the virtual T-axis buttons gets cleared for USB mice which has
  less than 6 buttons.

  Make sure the virtual T-axis buttons generate button release event(s)
  for continuous tilting.

  PR:	213919
  PR:	213957

Changes:
_U  stable/8/sys/
_U  stable/8/sys/dev/
_U  stable/8/sys/dev/usb/
  stable/8/sys/dev/usb/input/ums.c