From: Thomas Richter <richter@rus.uni-stuttgart.de>
To: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: Bug reports on 830MG patches (thanks, but more trouble)
Date: Fri, 06 Jun 2014 19:24:02 +0200 [thread overview]
Message-ID: <5391F932.8010404@rus.uni-stuttgart.de> (raw)
In-Reply-To: <20140606084621.GF27580@intel.com>
Hi Ville, hi others,
>> As Ville already said, resume from "suspend-to-ram" is broken. No
>> surprise, old broken bios. However, there is a big difference between
>> the kernel with the pipe-A quirk disabled, and the one with pipe-a and
>> pipe-b quirks enabled: If resumed without the quirk, the display is
>> dead, but you can at least re-initiate it with vbetool post, then
>> restart the X server. So at least, the kernel is working.
>>
>> With the pipe A quirk enabled, something dies immediately when
>> attempting to resume from suspend. The machine is completely
>> unresponsive, also not reachable over the network. The kernel dies away.
>
> For me resume gets stuck even w/o i915. So something is different
> clearly between our machines. Might be some driver, or might the BIOS.
> Do you have the latest BIOS on that thing?
Yup, I flashed the bios last week with version 1.07, the latest I could
find on the Fujitsu pages. It was 1.06 before, though I did not observe
any difference between 1.06 and 1.07 with respect to the broken resume
operation.
There is a difference between "freeze" and "freeze". Yes, w/o the pipe-A
quirk, the screen goes black, but you can reach it over the network, and
you can even type into the console and submit the video restart from
there. So, it's not frozen, only the display is "off", though it looks
the same for the untrained eye.
Maybe the bios configuration between yours and mine is different?
>> This is *probably* related to the dying kernel when booting with vga=792
>> (or vga=791 or vga=790, all cause the same problem, no working machine.)
>
> I must admit I didn't try the vga= options. I'll give them a go and see
> what happens.
Please do. I have narrowed it down to the case where text mode vs.
graphics mode makes a difference. That is, a frame buffer console alone
is sufficient for the machine to get stuck when loading i915.
>> ----
>>
>> Thanks for the patch and for all the work, unfortunately it does not
>> quite work as presented. I applied the patch correctly to
>> drm-intel-nightly, and it compiles fine.
>
> You applied all 15 patches, right?
Exactly, all 15 of them.
>>
>> However, the following issues exist:
>>
>> *) If you boot with the kernel parameter vga=792 (i.e. the same console
>> resolutions as initiated by KMS) the system just hangs with a black
>> screen. Booting aborts, nothing happens.
>>
>> *) Watermark settings are *still* not correct. *SIGH*, *sigh*
>>
>> intel_reg_read 0x20d8
>> 0x20D8 : 0x11F0104
>
> I did warn you. If you'd just taken my branch you would have gotten the
> hack to avoid that.
Ok, I'll try tonite to clone from your repository instead of
intel-drm-nightly and report back. But if you have a patch concerning
the watermark that works and is *accepted* into intel-drm-next, I would
be thankful indeed. This is really annoying...
(It's not that it is hard to work around the watermark issue....)
So long,
Thomas
next prev parent reply other threads:[~2014-06-06 17:24 UTC|newest]
Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-05 16:15 [PATCH 00/15] drm/i915: Fix 830M/ns2501 for real, well almost ville.syrjala
2014-06-05 16:15 ` [PATCH 01/15] drm/i915: Use named initializers for gmch wm params ville.syrjala
2014-06-05 20:43 ` Chris Wilson
2014-06-05 21:02 ` Thomas Richter
2014-06-05 21:33 ` Bug reports on 830MG patches (thanks, but more trouble) Thomas Richter
2014-06-06 8:46 ` Ville Syrjälä
2014-06-06 17:24 ` Thomas Richter [this message]
2014-06-06 20:08 ` Ville Syrjälä
2014-06-06 21:09 ` Thomas Richter
2014-06-06 21:41 ` Ville Syrjälä
2014-06-08 21:29 ` [PATCH] Check for a min level when computing the watermark Thomas Richter
2014-06-06 16:38 ` [PATCH 01/15] drm/i915: Use named initializers for gmch wm params Daniel Vetter
2014-06-05 16:15 ` [PATCH 02/15] drm/i915: Fix gen2 planes B and C max watermark value ville.syrjala
2014-06-05 16:15 ` [PATCH 03/15] drm/i915: Don't get hw state from DVO chip unless DVO is enabled ville.syrjala
2014-06-06 16:39 ` Daniel Vetter
2014-06-05 16:15 ` [PATCH 04/15] drm/i915: ns2501 is on DVOB ville.syrjala
2014-06-06 16:57 ` Daniel Vetter
2014-06-06 21:46 ` Ville Syrjälä
2014-06-05 16:15 ` [PATCH 05/15] drm/i915: Enable DVO between mode_set and dpms hooks ville.syrjala
2014-06-05 16:15 ` [PATCH 06/15] drm/i915: Don't call DVO mode_set hook on DPMS changes ville.syrjala
2014-06-05 16:15 ` [PATCH 07/15] drm/i915: Kill useless ns2501_dump_regs ville.syrjala
2014-06-05 16:15 ` [PATCH 08/15] drm/i915: Rewrite ns2501 driver a bit ville.syrjala
2014-06-05 16:15 ` [PATCH 09/15] drm/i915: Ignore VBT int_crt_support on 830M ville.syrjala
2014-06-06 17:00 ` Daniel Vetter
2014-06-06 19:44 ` [PATCH v2 " ville.syrjala
2014-06-06 20:13 ` Daniel Vetter
2014-06-07 20:37 ` [Patch] Add minimum watermark level for I830 Thomas Richter
2014-06-06 21:15 ` [PATCH v2 09/15] drm/i915: Ignore VBT int_crt_support on 830M Bob Paauwe
2014-06-06 22:23 ` Daniel Vetter
2014-06-06 22:51 ` Jesse Barnes
[not found] ` <2094_1402093395_53923F53_2094_10301_1_CAKMK7uGAnNP4VR9+zXd0KD5v0Vo=XuDS=NhRNFRqHKcae7T4XQ@mail.gmail.com>
2014-06-07 17:32 ` Thomas Richter
2014-10-24 13:23 ` Jani Nikula
2014-10-24 14:11 ` Ville Syrjälä
2014-06-05 16:15 ` [PATCH 10/15] drm/i915: Fix DVO 2x clock enable " ville.syrjala
2014-06-05 16:16 ` [PATCH 11/15] Revert "drm/i915: Nuke pipe A quirk on i830M" ville.syrjala
2014-06-05 16:16 ` [PATCH 12/15] drm/i915: Add pipe B force quirk for 830M ville.syrjala
2014-06-05 16:16 ` [PATCH 13/15] drm/i915: Eliminate rmw from .update_primary_plane() ville.syrjala
2014-06-06 0:02 ` Matt Roper
2014-06-06 19:45 ` [PATCH v2 " ville.syrjala
2014-06-05 16:16 ` [PATCH 14/15] drm/i915: Call .update_primary_plane in intel_{enable, disable}_primary_hw_plane() ville.syrjala
2014-06-06 0:02 ` Matt Roper
2014-06-06 8:40 ` Ville Syrjälä
2014-06-06 19:46 ` [PATCH v2 " ville.syrjala
2014-06-05 16:16 ` [PATCH 15/15] drm/i915: Check pixel clock in ns2501 mode_valid hook ville.syrjala
2014-06-06 19:47 ` [PATCH 16/15] drm/i915: Pass intel_crtc to intel_disable_pipe() and intel_wait_for_pipe_off() ville.syrjala
2014-06-06 19:47 ` [PATCH 17/15] drm/i915: Disable double wide even when leaving the pipe on ville.syrjala
2014-06-06 22:09 ` [PATCH v2 " ville.syrjala
2014-06-08 23:14 ` Deadlock in intel_enable_pipe_a() Thomas Richter
2014-06-09 6:47 ` [PATCH] drm/i915: Avoid double mutex lock applying pipe A quirk during sanitize_crtc() Chris Wilson
2014-06-09 8:30 ` Ville Syrjälä
2014-06-09 8:50 ` Chris Wilson
[not found] ` <28223_1402303866_5395757A_28223_3428_1_20140609085045.GE16767@nuc-i3427.alporthouse.com>
2014-06-09 10:57 ` Partial success - Fixing resume from s2ram on S6010 Thomas Richter
2014-06-09 11:08 ` Ville Syrjälä
[not found] ` <28223_1402312148_539595D3_28223_4884_1_20140609110857.GM27580@intel.com>
2014-06-09 11:19 ` Thomas Richter
2014-06-09 11:31 ` Ville Syrjälä
[not found] ` <2086_1402313568_53959B5F_2086_895_1_20140609113155.GN27580@intel.com>
2014-06-09 12:33 ` Thomas Richter
2014-06-09 12:57 ` Thomas Richter
2014-06-09 18:41 ` Thomas Richter
2014-06-09 19:46 ` [PATCH] drm/i915: Init important ns2501 registers ville.syrjala
[not found] ` <28223_1402343538_53961072_28223_7661_1_1402343204-28608-1-git-send-email-ville.syrjala@linux.intel.com>
2014-06-09 20:58 ` Thomas Richter
2014-06-09 22:29 ` Thomas Richter
2014-06-10 14:04 ` Ville Syrjälä
[not found] ` <29040_1402409145_539710B9_29040_2220_1_20140610140430.GD27580@intel.com>
2014-06-10 16:38 ` Thomas Richter
2014-06-18 16:03 ` i830GM on IBM R31 works with alm_fixes5 repository Thomas Richter
2014-06-10 7:02 ` [PATCH] drm/i915: Avoid double mutex lock applying pipe A quirk during sanitize_crtc() Daniel Vetter
2014-06-10 8:53 ` Ville Syrjälä
2014-06-10 9:22 ` Daniel Vetter
2014-06-10 6:59 ` Daniel Vetter
2014-06-10 7:13 ` Chris Wilson
2014-06-06 19:47 ` [PATCH 18/15] drm/i915: Preserve VGACNTR bits from the BIOS ville.syrjala
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5391F932.8010404@rus.uni-stuttgart.de \
--to=richter@rus.uni-stuttgart.de \
--cc=intel-gfx@lists.freedesktop.org \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox