Bug 24496 - kms + xv + xv_vsync + TV clone@1280x1024 = soft lockup.
Summary: kms + xv + xv_vsync + TV clone@1280x1024 = soft lockup.
Status: RESOLVED FIXED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/Radeon (show other bugs)
Version: XOrg git
Hardware: x86 (IA32) Linux (All)
: medium normal
Assignee: Default DRI bug account
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-13 04:41 UTC by Andy Furniss
Modified: 2009-10-14 12:18 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
limit default tv modes (1.08 KB, patch)
2009-10-13 09:55 UTC, Alex Deucher
no flags Details | Splinter Review
fix tv-out (2.42 KB, patch)
2009-10-14 08:13 UTC, Alex Deucher
no flags Details | Splinter Review
set active devices early (6.53 KB, patch)
2009-10-14 08:14 UTC, Alex Deucher
no flags Details | Splinter Review

Description Andy Furniss 2009-10-13 04:41:00 UTC
Rv670 Running latest drm-next + ddx - only just tested this case so don't know if it's a regression somewhere or not.

Filing as kms as ums doesn't offer a DIN mode > 1024x768, so doesn't have the problem.

If I start X with 1280x1024 res the TV s-video DIN also gets a 1280x1024 cloned mode (Although it only actually displays 1280x768).

mplayer -fs will softlock - SysRq works but I can find no messages.

Doesn't happen if -

DIN is turned off.

XV_VSYNC is 0

Monitor is 1280x1024 and DIN is 1024x768.
Comment 1 Alex Deucher 2009-10-13 07:32:04 UTC
Does it still happen if you force the DIN to a lower res like 1024x768 or 800x600 or something smaller?
Comment 2 Andy Furniss 2009-10-13 09:38:36 UTC
(In reply to comment #1)
> Does it still happen if you force the DIN to a lower res like 1024x768 or
> 800x600 or something smaller?
> 

No, it's OK if DIN is <= 1024x768.
Comment 3 Alex Deucher 2009-10-13 09:55:31 UTC
Created attachment 30349 [details] [review]
limit default tv modes

This patch should take care of it.
Comment 4 Andy Furniss 2009-10-13 12:21:23 UTC
(In reply to comment #3)
> Created an attachment (id=30349) [details]
> limit default tv modes
> 
> This patch should take care of it.
> 

Well it seems to, but because I've been testing this bug without turning the TV on I've only just noticed there has been a separate tv-out regression caused by

5a9bcacc0a56f0d9577494e834519480018a6cc3

Since this commit tv-out is displaying garbage (though it is related to what should be there)

This bug is not related to that regression - if I reset to before that commit tv-out works OK and this bug still exists.
Comment 5 Alex Deucher 2009-10-14 08:13:25 UTC
Created attachment 30401 [details] [review]
fix tv-out

this patch should fix tv-out
Comment 6 Alex Deucher 2009-10-14 08:14:36 UTC
Created attachment 30402 [details] [review]
set active devices early

You may need this patch as well.
Comment 7 Andy Furniss 2009-10-14 10:14:57 UTC
(In reply to comment #6)

Thanks, just been testing with all three patches and tv-out and this bug are fixed.

TV with fbcon and X limited to 1024x768 and mplayer -fs is working.

One minor thing I noticed, if DIN is turned off in X the fbcon TV is also off on VT switch/quit X. However it is possible to get it to output to TV by doing a VT switch, then VT back to X, after that fbcon outputs to TV again on further VT switches or quitting X.

I've also found an xserver issue sort of exposed by this change - it seems that starting X with clones of a different size causes it to segfault after one is disabled and something tries to change gamma. This happens in ums as well.
Comment 8 Alex Deucher 2009-10-14 10:30:08 UTC
(In reply to comment #7)
> 
> I've also found an xserver issue sort of exposed by this change - it seems that
> starting X with clones of a different size causes it to segfault after one is
> disabled and something tries to change gamma. This happens in ums as well.
> 

Can you open a new bug for this and attach the backtrace?
Comment 9 Andy Furniss 2009-10-14 12:18:08 UTC
(In reply to comment #8)

> Can you open a new bug for this and attach the backtrace?

Filed under radeon -

http://bugs.freedesktop.org/show_bug.cgi?id=24532


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.