Ticket #7407 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Rotating the screen causes X lockup (xrandr broken)

Reported by: erikg Owned by: ApprovalForUpdate
Priority: high Milestone: 8.2.0 (was Update.2)
Component: not assigned Version: Development build as of this date
Keywords: joyride-2269:+ 8.2-759:+ Cc: Rmyers, pepe, erikg
Action Needed: test in release Verified: no
Deployments affected: Blocked By:
Blocking:

Description

In joyride-2078 and joyride-2079, pressing the rotate button causes a slow white-out of the display.

The problem appears to be xrandr-related. Running xrandr -o 'left' in the terminal activity causes the same lockup.

Attachments

fix-rotate.patch (1.2 kB) - added by erikg 6 years ago.
patch from CosmicPenguin (Jordan Crouse), apply to xf86-video-geode git HEAD.

Change History

  Changed 6 years ago by erikg

  • owner set to erikg
  • status changed from new to assigned
  • version changed from Update.1 to Development build as of this date
  • milestone changed from Never Assigned to 9.1.0

  Changed 6 years ago by erikg

I have produced the identical issue by running 'xrandr -o left' in the Terminal.

  Changed 6 years ago by dsd

  • cc dsd added
  • keywords 8.2.0:? added

I think we should change milestone? This is an 8.2 regression which we should fix before release.

follow-up: ↓ 5   Changed 6 years ago by dsd

BTW, Brian made an interesting observation. This is not a complete freeze, because if you press the rotate button 4 times then you get your display back.

in reply to: ↑ 4   Changed 6 years ago by erikg

Replying to dsd:

BTW, Brian made an interesting observation. This is not a complete freeze, because if you press the rotate button 4 times then you get your display back.

Wow. Interesting.

For what it's worth I've established that the problem isn't in the xrandr executable. Digging into the Xrandr library.

  Changed 6 years ago by erikg

CosmicPenguin (per discussion in #xorg-devel) suggests that the problem is likely a loss of sync between the DCON and the GPU.

  Changed 6 years ago by Rmyers

  • cc Rmyers added

replying to dsd:

In more recent builds, four rotations _do_ _not_ get you back.

I reported this issue as 7651 (just closed as dup of this. As of 2137 -- earliest I tried, once the screen's goen it stays gone. Still there in 2230.

Replying to erikg:

But it used to work. Isn't it more likely a regression on our part?

  Changed 6 years ago by pepe

  • cc pepe added

  Changed 6 years ago by cjb

  • keywords blocks?:8.2.0 added

  Changed 6 years ago by gregorio

  • milestone changed from 9.1.0 to 8.2.0 (was Update.2)

  Changed 6 years ago by gregorio

  • keywords blocks:8.2.0 added; blocks?:8.2.0 removed

Changed 6 years ago by erikg

patch from CosmicPenguin (Jordan Crouse), apply to xf86-video-geode git HEAD.

  Changed 6 years ago by dsd

  • next_action changed from never set to package

This is a F9 package so I requested upstream inclusion of the patch: https://bugzilla.redhat.com/show_bug.cgi?id=458382

For the time being, I also added a public_rpm for it, which should be included in the next joyrides.

  Changed 6 years ago by erikg

I tested and verified the functionality the attached patch on an XO running joyride-2230. It resolves the issue. (dsd has thus moved forward to packaging).

  Changed 6 years ago by garycmartin

Tested with joyride-2269 on a B4 XO, rotate key is correctly rotating the screen.

  Changed 6 years ago by mstone

  • keywords joyride-2269:+ added

  Changed 6 years ago by dsd

  • cc erikg added; dsd removed
  • owner changed from erikg to dsd
  • status changed from assigned to new

  Changed 6 years ago by kimquirk

  • next_action changed from package to test in build

This can be tested in today's build. Still needs the correct packaging.

  Changed 6 years ago by kimquirk

  • keywords 8.2.0:? blocks:8.2.0 removed

In test, this no longer crashes. Will open a new bug taht some laptops don't rotate with the button. Should be closed as soon as the packaging is done.

  Changed 6 years ago by dsd

  • owner changed from dsd to ApprovalForUpdate
  • next_action changed from test in build to add to build

Please include xorg-x11-drv-geode-2.10.1-1.fc9 in the 8.2 stream. Right now this is coming from my public_rpms which is not so desirable.

  Changed 6 years ago by dsd

|TestCase|

Press rotate screen button and check that the XO doesn't immediately die a horrible death

  Changed 6 years ago by dsd

  • next_action changed from add to build to approve for release

  Changed 6 years ago by mstone

  • next_action changed from approve for release to add to release

  Changed 6 years ago by cscott

  • next_action changed from add to release to test in release

Committed changes to stable repo; should be in builds 758 and later. Please test.

  Changed 6 years ago by dsd

  • keywords 8.2-759:+ added
  • status changed from new to closed
  • resolution set to fixed

confirmed working in build 759

  Changed 6 years ago by frances

confirmed working from 2 different XOs running 8.2-759

Note: See TracTickets for help on using tickets.