| Summary: | [piketon] DP hotplug will let system down | ||||||
|---|---|---|---|---|---|---|---|
| Product: | xorg | Reporter: | bo.b.wang | ||||
| Component: | Driver/intel | Assignee: | Chris Wilson <chris> | ||||
| Status: | VERIFIED FIXED | QA Contact: | Xorg Project Team <xorg-team> | ||||
| Severity: | critical | ||||||
| Priority: | high | ||||||
| Version: | unspecified | ||||||
| Hardware: | All | ||||||
| OS: | Linux (All) | ||||||
| Whiteboard: | |||||||
| i915 platform: | i915 features: | ||||||
| Attachments: |
|
||||||
|
Description
bo.b.wang
2011-04-21 02:42:56 UTC
Created attachment 45911 [details]
DP hotplug infomation
Look in dmesg for the complete OOPS. At a guess: commit fea4e9038079ae3300eddde50ab258e5b43c693f Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Sun Apr 17 06:38:35 2011 +0100 drm/i915/dp: Be paranoid in case we disable a DP before it is attached Given that the hardware may be left in a random condition by the BIOS, it is conceivable that we then attempt to clear the DP_PIPEB_SELECT bit without us ever enabling/attaching the DP encoder to a pipe. Thus causing a NULL deference when we attempt to wait for a vblank on that crtc. Reported-and-tested-by: Bryan Christ <bryan.christ@gmail.com> References: https://bugs.freedesktop.org/show_bug.cgi?id=36314 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: stable@kernel.org (In reply to comment #2) > Look in dmesg for the complete OOPS. This issue will let system down. I can't ssh to fetch dmesg (In reply to comment #3) > At a guess: > > commit fea4e9038079ae3300eddde50ab258e5b43c693f > Author: Chris Wilson <chris@chris-wilson.co.uk> > Date: Sun Apr 17 06:38:35 2011 +0100 > > drm/i915/dp: Be paranoid in case we disable a DP before it is attached > > Given that the hardware may be left in a random condition by the BIOS, > it is conceivable that we then attempt to clear the DP_PIPEB_SELECT bit > without us ever enabling/attaching the DP encoder to a pipe. Thus > causing a NULL deference when we attempt to wait for a vblank on that > crtc. > > Reported-and-tested-by: Bryan Christ <bryan.christ@gmail.com> > References: https://bugs.freedesktop.org/show_bug.cgi?id=36314 > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: stable@kernel.org There is another testdisplay issue that seems to https://bugs.freedesktop.org/show_bug.cgi?id=36314. But , I can't get that commit in stable or unstable kernel. So, Would you like to tell me how can I get that commit? Your reply will be appreciated Now ickle/drm-intel-staging commit 8ff887c847a8c88cee2cddffb8b94f41ebd5e4db Author: Chris Wilson <chris@chris-wilson.co.uk> Date: Sun Apr 17 06:38:35 2011 +0100 drm/i915/dp: Be paranoid in case we disable a DP before it is attached (In reply to comment #6) > Now ickle/drm-intel-staging commit 8ff887c847a8c88cee2cddffb8b94f41ebd5e4db > Author: Chris Wilson <chris@chris-wilson.co.uk> > Date: Sun Apr 17 06:38:35 2011 +0100 > > drm/i915/dp: Be paranoid in case we disable a DP before it is attached Hi Chris, I have tested this commit.And DP hotpluging is no problem. (In reply to comment #7) > (In reply to comment #6) > > Now ickle/drm-intel-staging commit 8ff887c847a8c88cee2cddffb8b94f41ebd5e4db > > Author: Chris Wilson <chris@chris-wilson.co.uk> > > Date: Sun Apr 17 06:38:35 2011 +0100 > > > > drm/i915/dp: Be paranoid in case we disable a DP before it is attached > > Hi Chris, I have tested this commit.And DP hotpluging is no problem. Tested-by: Wang Bo <bo.b.wang@intel.com> *** Bug 36561 has been marked as a duplicate of this bug. *** The commit should have been pushed. |
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.