| Summary: | [BSW Bisected] RC6 timeout mode patch causes random GPU freq drops and up to -40% performance regressions | ||||||
|---|---|---|---|---|---|---|---|
| Product: | DRI | Reporter: | wendy.wang | ||||
| Component: | DRM/Intel | Assignee: | Intel GFX Bugs mailing list <intel-gfx-bugs> | ||||
| Status: | CLOSED FIXED | QA Contact: | Intel GFX Bugs mailing list <intel-gfx-bugs> | ||||
| Severity: | major | ||||||
| Priority: | highest | CC: | eero.t.tamminen, intel-gfx-bugs, james.ausmus | ||||
| Version: | unspecified | ||||||
| Hardware: | Other | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| i915 platform: | i915 features: | ||||||
| Attachments: |
|
||||||
|
Description
wendy.wang
2015-01-06 11:26:12 UTC
What other tests this particular commit affected? (In reply to Eero Tamminen from comment #1) > What other tests this particular commit affected? We are investigating, will update on 8th Jan. Test with below cases: lightsmark openarena warsow01 GpuTest_v05_triangle_Windowed cs portal hoverjet TRex_FixedTimeStepOffscreen FillTestC24Z16_Offscreen manhattan 1.We found the following case happen regression because of bad commit 2.listed regression percentage Test with below cases: lightsmark -13% openarena -33% warsow01 -9% cs -28% portal -28% hoverjet -16% TRex_FixedTimeStepOffscreen -22% FillTestC24Z16_Offscreen -12% http://lists.freedesktop.org/archives/intel-gfx/2015-January/058300.html Could you please test the revert? Also, I'd like to see the difference of power savings measure with and without this revert. Tested with revert commit, all the impacted cases can recovered to normal performance FPS values lightsmark -13% openarena -33% warsow01 -9% cs -28% portal -28% hoverjet -16% TRex_FixedTimeStepOffscreen -22% FillTestC24Z16_Offscreen -12% unigine-valley_1_0 -17% EgyptHD_FixedTimeOffscreen -14% manhattan -22% OglDrvCtx_6 60.54 -24% But the bad commit is also very important which was to fix RC6 residency issue. So we need developer comments to see if these case perf regression required revert the bad commit. This seems to cause GPU frequency to fluctuate randomly. Some test-runs go at full speed, some fluctuate, for test cases that are fully GPU bound. I've also seen the effect in several other tests (e.g. Unigine Valley and SynMark Deferred). This kind of behavior breaks performance testing so *it needs to be reverted* until GPU freq issue if fixed. Wendy, could you please try Ville's patch and check if it fix performance and still allows RC6 good residency? Otherwise I'll continue the patch I had started here to change mode and treshold according to idle and boost requests... Verified Ville's patch on BSW B1+ FAB2 RVP 1. RC6 residency are good: RC6 residency when idle: 100% RC6 residency when media workload: 79.7% 2. Previous impacted performance FPS also improved except OglDrvCtx case , Below percentage is for Ville's fix patch vs. first bad commit lightsmark 28.00% openarena 2.60% warsow01 6.55% unigine-vally_1_0 26.44% cs 44.76% portal 58.72% 3DMMES2_taiji -2.76% 3DMMES2_hoverjet -1.87% glb270_TRex_FixedTimeStepOffscreen 2.78% glb270_EgyptHD_FixedTimeOffscreen -1.54% glb270_FillTestC24Z16_Offscreen 4.17% glb3016_manhattan 33.33% OglDrvCtx_6 -9.96% OglVSInstancing_6 -0.21% commit eca5aff51ea6c4d9b4936d94c61707cee4b12902 Author: Ville Syrjälä <ville.syrjala@linux.intel.com> Date: Mon Jan 19 13:50:49 2015 +0200 drm/i915: Configure GEN6_RP_DOWN_TIMEOUT on CHV Bug has been verified as PASS using latest -nightly branch kernel. Closing old verified. |
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.