Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Mar 2009 16:08:30 -0500
From:      Robert Noland <rnoland@FreeBSD.org>
To:        Brandon Gooch <jamesbrandongooch@gmail.com>
Cc:        Michiel Boland <michiel@boland.org>, FreeBSD Current <freebsd-current@freebsd.org>, Mattia Rossi <mrossi@swin.edu.au>
Subject:   Re: still problems with intel video
Message-ID:  <1238188110.8491.194.camel@balrog.2hip.net>
In-Reply-To: <179b97fb0903270014u12e5fd10w46a18c36efa9ddf8@mail.gmail.com>
References:  <49CB70BD.3040607@boland.org> <1238086577.1792.30.camel@balrog.2hip.net> <49CC43C4.7030905@swin.edu.au> <1238126285.8491.2.camel@balrog.2hip.net> <179b97fb0903262114i511ec294ief17475d673e70c9@mail.gmail.com> <1238136987.8491.173.camel@balrog.2hip.net> <179b97fb0903270014u12e5fd10w46a18c36efa9ddf8@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help

--=-bx4tddrYyiuGGcswbPut
Content-Type: multipart/mixed; boundary="=-y8fcTc4EOZiPJ55QduKo"


--=-y8fcTc4EOZiPJ55QduKo
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

On Fri, 2009-03-27 at 02:14 -0500, Brandon Gooch wrote:
> On Fri, Mar 27, 2009 at 1:56 AM, Robert Noland <rnoland@freebsd.org> wrot=
e:
> > On Thu, 2009-03-26 at 23:14 -0500, Brandon Gooch wrote:
> >> On Thu, Mar 26, 2009 at 10:58 PM, Robert Noland <rnoland@freebsd.org> =
wrote:
> >> > On Fri, 2009-03-27 at 14:11 +1100, Mattia Rossi wrote:
> >> >> Robert Noland wrote:
> >> >> > On Thu, 2009-03-26 at 13:10 +0100, Michiel Boland wrote:
> >> >> >
> >> >> >> Hi. I still have problems with a very slow display after logging=
 out of an X
> >> >> >> session and/or switching VTYs. The problem goes away if I add
> >> >> >> hw.pci.enable_msi=3D0 to /boot/loader.conf.
> >> >> >>
> >> >> >> Last csupped Mar 26 09:49 CET.
> >> >> >>
> >> >> >> System is Dell optiplex 745. Has built-in Intel Graphics Media A=
ccelerator 950
> >> >> >>
> >> >> >> Anything I can do to help solve this problem?
> >> >> >>
> >> >> >
> >> >> > I'm going to try and work on getting better debugging info from t=
he
> >> >> > intel driver.  I don't have access to any newer Intel hardware at=
 the
> >> >> > moment, so testing is tricky.
> >> >> >
> >> >> > There is a tuneable for just msi on drm hw.drm.msi.
> >> >> >
> >> >> > robert.
> >> >> >
> >> >> >
> >> >> Yep, correct - here it is again - just had to log out of KDE, and a=
fter
> >> >> logging in again, everything was slow as hell.
> >> >> I didn't fiddle with the msi settings, just rebooted the machine, a=
nd
> >> >> everything is fine again.
> >> >> So there must be something that works the first time X is started, =
but
> >> >> upon restart stuffs up. Like some lock or reference which is not fr=
eed.
> >> >
> >> > There is a problem with restarting X on at least some Intel chips...
> >> > This is a different issue, I was trying to look into that a little b=
it
> >> > yesterday, but it kinda works on this 915 that I have, so I haven't
> >> > isolated what is getting messed up.  Again, vt switch, suspend/resum=
e
> >> > are in the same ballpark, restart is not.
> >> >
> >> > robert.
> >> >
> >> >> Mat
> >> >> >> _______________________________________________
> >> >> >> freebsd-current@freebsd.org mailing list
> >> >> >> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> >> >> >> To unsubscribe, send any mail to "freebsd-current-unsubscribe@fr=
eebsd.org"
> >> >> >>
> >> >>
> >> > --
> >> > Robert Noland <rnoland@FreeBSD.org>
> >> > FreeBSD
> >> >
> >>
> >> I have been switching to the vty at which I started X in order to
> >> terminate X. If I try to terminate X while I'm in it, it just "hangs"
> >> -- I have to switch to the vty, ctrl-c it, wait, then blindly key in a
> >> reboot to get my system back up. IIRC, this started happening after I
> >> upgraded to Xorg 7.4.
> >>
> >> I thought that Mattia might be doing the same thing as a work-around
> >> for the freezing intel-driven display thing, thus the X-to-vty switch
> >> "fix" of disabling msi...
> >
> > Hrm, ok... So, vt switching on the I915 that I'm using to test(which
> > doesn't support msi) works pretty much perfectly.  Console is not
> > corrupted, (unless I've restarted X at least once).  Shutdown/reboot
> > from gnome also works as expected.
> >
> > I did most of the msi development on an i965gm, which did support msi
> > and vt switch also worked with msi at that time.  Restart was
> > problematic on the 965 then as well, which I believed to be an agp, or
> > agp/drm interaction issue.  So, unless some of the recent changes other
> > than msi, have broken 965, they should be mostly working ok as long as
> > you don't restart.
> >
> > The g31 and g45 are slightly different and all of my work for those has
> > been based on what Intel has released for linux and the small amount of
> > documentation that is available at this point.
> >
> > BTW, you debug output looks strange to me, did you also capture the drm
> > debug log from hw.dri.0.debug=3D1?
> >
> > robert.
> >
> >> -Brandon
> > --
> > Robert Noland <rnoland@FreeBSD.org>
> > FreeBSD
> >
>=20
> I've attached a copy of my /var/log/messages file after enabling
> hw.dri.0.debug, suspending, and then resuming. I disable it shortly
> after resume is complete.

Ok, this is a complete patch against HEAD.  It has even more debugging
around suspend/resume and also grabs the hardware lock when it starts to
suspend or resume.  I'm tempted to just grab the lock when we start
suspend and not release it until resume is complete, but it looks like
something is triggering a vt switch and we could deadlock on that if I
don't drop the lock.  I should be able to tell a little more from the
drm debug output of this patch.

robert.

>=20
> -Brandon
--=20
Robert Noland <rnoland@FreeBSD.org>
FreeBSD

--=-y8fcTc4EOZiPJ55QduKo
Content-Disposition: attachment; filename="drm-i915-interrupts.patch"
Content-Type: text/x-patch; name="drm-i915-interrupts.patch";
	charset="us-ascii"
Content-Transfer-Encoding: base64

SW5kZXg6IGRldi9kcm0vaTkxNV9kbWEuYw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KLS0tIGRldi9kcm0vaTkxNV9k
bWEuYwkocmV2aXNpb24gMTkwNDAyKQ0KKysrIGRldi9kcm0vaTkxNV9kbWEuYwkod29ya2luZyBj
b3B5KQ0KQEAgLTQ0NCw2ICs0NDQsOCBAQA0KIAlpZiAoZGV2X3ByaXYtPnNhcmVhX3ByaXYpDQog
CQlkZXZfcHJpdi0+c2FyZWFfcHJpdi0+bGFzdF9lbnF1ZXVlID0gZGV2X3ByaXYtPmNvdW50ZXI7
DQogDQorCURSTV9ERUJVRygiZW1pdHRpbmc6ICVkXG4iLCBkZXZfcHJpdi0+Y291bnRlcik7DQor
DQogCUJFR0lOX0xQX1JJTkcoNCk7DQogCU9VVF9SSU5HKE1JX1NUT1JFX0RXT1JEX0lOREVYKTsN
CiAJT1VUX1JJTkcoSTkxNV9CUkVBRENSVU1CX0lOREVYIDw8IE1JX1NUT1JFX0RXT1JEX0lOREVY
X1NISUZUKTsNCkBAIC00ODEsNiArNDgzLDcgQEANCiAJfQ0KIA0KIAlpOTE1X2VtaXRfYnJlYWRj
cnVtYihkZXYpOw0KKw0KIAlyZXR1cm4gMDsNCiB9DQogDQpJbmRleDogZGV2L2RybS9pOTE1X2Ry
di5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09DQotLS0gZGV2L2RybS9pOTE1X2Rydi5jCShyZXZpc2lvbiAxOTA0MDIp
DQorKysgZGV2L2RybS9pOTE1X2Rydi5jCSh3b3JraW5nIGNvcHkpDQpAQCAtNTIsNyArNTIsMTAg
QEANCiAJCXJldHVybiAtRU5PREVWOw0KIAl9DQogDQorCURSTV9MT0NLKCk7DQorCURSTV9ERUJV
Rygic3RhcnRpbmcgc3VzcGVuZFxuIik7DQogCWk5MTVfc2F2ZV9zdGF0ZShkZXYpOw0KKwlEUk1f
VU5MT0NLKCk7DQogDQogCXJldHVybiAoYnVzX2dlbmVyaWNfc3VzcGVuZChrZGV2KSk7DQogfQ0K
QEAgLTYxLDcgKzY0LDEwIEBADQogew0KIAlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2ID0gZGV2aWNl
X2dldF9zb2Z0YyhrZGV2KTsNCiANCisJRFJNX0xPQ0soKTsNCisJRFJNX0RFQlVHKCJzdGFydGlu
ZyByZXN1bWVcbiIpOw0KIAlpOTE1X3Jlc3RvcmVfc3RhdGUoZGV2KTsNCisJRFJNX1VOTE9DSygp
Ow0KIA0KIAlyZXR1cm4gKGJ1c19nZW5lcmljX3Jlc3VtZShrZGV2KSk7DQogfQ0KSW5kZXg6IGRl
di9kcm0vaTkxNV9pcnEuYw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KLS0tIGRldi9kcm0vaTkxNV9pcnEuYwkocmV2
aXNpb24gMTkwNDAyKQ0KKysrIGRldi9kcm0vaTkxNV9pcnEuYwkod29ya2luZyBjb3B5KQ0KQEAg
LTIwNSw3ICsyMDUsNyBAQA0KIAlpaXIgPSBJOTE1X1JFQUQoSUlSKTsNCiANCiAJaWYgKElTX0k5
NjVHKGRldikpIHsNCi0JCXZibGFua19zdGF0dXMgPSBJOTE1X1NUQVJUX1ZCTEFOS19JTlRFUlJV
UFRfU1RBVFVTOw0KKwkJdmJsYW5rX3N0YXR1cyA9IFBJUEVfU1RBUlRfVkJMQU5LX0lOVEVSUlVQ
VF9TVEFUVVM7DQogCQl2YmxhbmtfZW5hYmxlID0gUElQRV9TVEFSVF9WQkxBTktfSU5URVJSVVBU
X0VOQUJMRTsNCiAJfSBlbHNlIHsNCiAJCXZibGFua19zdGF0dXMgPSBJOTE1X1ZCTEFOS19JTlRF
UlJVUFRfU1RBVFVTOw0KQEAgLTM0Myw2ICszNDMsOSBAQA0KIA0KIAlEUk1fREVCVUcoImlycV9u
cj0lZCBicmVhZGNydW1iPSVkXG4iLCBpcnFfbnIsDQogCQkgIFJFQURfQlJFQURDUlVNQihkZXZf
cHJpdikpOw0KKwlEUk1fREVCVUcoImllcjogJTA4eCwgaW1yOiAlMDh4LCBwaXBlYXN0YXQ6ICUw
OHgsIHBpcGVic3RhdDogJTA4eFxuIiwNCisJICAgIEk5MTVfUkVBRChJRVIpLCBJOTE1X1JFQUQo
SUVSKSwNCisJICAgIEk5MTVfUkVBRChQSVBFQVNUQVQpLCBJOTE1X1JFQUQoUElQRUJTVEFUKSk7
DQogDQogCWk5MTVfdXNlcl9pcnFfZ2V0KGRldik7DQogCURSTV9XQUlUX09OKHJldCwgZGV2X3By
aXYtPmlycV9xdWV1ZSwgMyAqIERSTV9IWiwNCkBAIC00MDgsMTEgKzQxMSw4IEBADQogaW50IGk5
MTVfZW5hYmxlX3ZibGFuayhzdHJ1Y3QgZHJtX2RldmljZSAqZGV2LCBpbnQgcGlwZSkNCiB7DQog
CWRybV9pOTE1X3ByaXZhdGVfdCAqZGV2X3ByaXYgPSAoZHJtX2k5MTVfcHJpdmF0ZV90ICopIGRl
di0+ZGV2X3ByaXZhdGU7DQotCWludCBwaXBlY29uZl9yZWcgPSAocGlwZSA9PSAwKSA/IFBJUEVB
Q09ORiA6IFBJUEVCQ09ORjsNCi0JdTMyIHBpcGVjb25mOw0KIA0KLQlwaXBlY29uZiA9IEk5MTVf
UkVBRChwaXBlY29uZl9yZWcpOw0KLQlpZiAoIShwaXBlY29uZiAmIFBJUEVBQ09ORl9FTkFCTEUp
KQ0KKwlpZiAoIWk5MTVfcGlwZV9lbmFibGVkKGRldiwgcGlwZSkpDQogCQlyZXR1cm4gLUVJTlZB
TDsNCiANCiAJRFJNX1NQSU5MT0NLKCZkZXZfcHJpdi0+dXNlcl9pcnFfbG9jayk7DQpJbmRleDog
ZGV2L2RybS9kcm1faXJxLmMNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NCi0tLSBkZXYvZHJtL2RybV9pcnEuYwkocmV2
aXNpb24gMTkwNDMzKQ0KKysrIGRldi9kcm0vZHJtX2lycS5jCSh3b3JraW5nIGNvcHkpDQpAQCAt
MzQ1LDEzICszNDUsMTEgQEANCiAJc3RydWN0IGRybV9tb2Rlc2V0X2N0bCAqbW9kZXNldCA9IGRh
dGE7DQogCWludCBjcnRjLCByZXQgPSAwOw0KIA0KLQlEUk1fREVCVUcoIm51bV9jcnRjcz0lZFxu
IiwgZGV2LT5udW1fY3J0Y3MpOw0KIAkvKiBJZiBkcm1fdmJsYW5rX2luaXQoKSBoYXNuJ3QgYmVl
biBjYWxsZWQgeWV0LCBqdXN0IG5vLW9wICovDQogCWlmICghZGV2LT5udW1fY3J0Y3MpDQogCQln
b3RvIG91dDsNCiANCiAJY3J0YyA9IG1vZGVzZXQtPmNydGM7DQotCURSTV9ERUJVRygiY3J0Yz0l
ZFxuIiwgY3J0Yyk7DQogCWlmIChjcnRjID49IGRldi0+bnVtX2NydGNzKSB7DQogCQlyZXQgPSBF
SU5WQUw7DQogCQlnb3RvIG91dDsNCkBAIC0zNjYsNyArMzY0LDcgQEANCiAJICovDQogCXN3aXRj
aCAobW9kZXNldC0+Y21kKSB7DQogCWNhc2UgX0RSTV9QUkVfTU9ERVNFVDoNCi0JCURSTV9ERUJV
RygicHJlLW1vZGVzZXRcbiIpOw0KKwkJRFJNX0RFQlVHKCJwcmUtbW9kZXNldCwgY3J0YyAlZFxu
IiwgY3J0Yyk7DQogCQlpZiAoIWRldi0+dmJsYW5rW2NydGNdLmlubW9kZXNldCkgew0KIAkJCWRl
di0+dmJsYW5rW2NydGNdLmlubW9kZXNldCA9IDB4MTsNCiAJCQlpZiAoZHJtX3ZibGFua19nZXQo
ZGV2LCBjcnRjKSA9PSAwKQ0KQEAgLTM3NCw3ICszNzIsNyBAQA0KIAkJfQ0KIAkJYnJlYWs7DQog
CWNhc2UgX0RSTV9QT1NUX01PREVTRVQ6DQotCQlEUk1fREVCVUcoInBvc3QtbW9kZXNldFxuIik7
DQorCQlEUk1fREVCVUcoInBvc3QtbW9kZXNldCwgY3J0YyAlZFxuIiwgY3J0Yyk7DQogCQlpZiAo
ZGV2LT52YmxhbmtbY3J0Y10uaW5tb2Rlc2V0KSB7DQogCQkJRFJNX1NQSU5MT0NLKCZkZXYtPnZi
bF9sb2NrKTsNCiAJCQlkZXYtPnZibGFua19kaXNhYmxlX2FsbG93ZWQgPSAxOw0KQEAgLTQ3Nyw2
ICs0NzUsOSBAQA0KIAkJCW10eF91bmxvY2soJmRldi0+aXJxX2xvY2spOw0KIAkJfQ0KIA0KKwkJ
aWYgKHJldCA9PSBFV09VTERCTE9DSykNCisJCQlEUk1fRVJST1IoInRpbWVkIG91dCB3YWl0aW5n
IG9uIHZibGFua1xuIik7DQorDQogCQlpZiAocmV0ICE9IEVJTlRSICYmIHJldCAhPSBFUkVTVEFS
VCkgew0KIAkJCXN0cnVjdCB0aW1ldmFsIG5vdzsNCiANCkluZGV4OiBkZXYvZHJtL2RybV9zeXNj
dGwuYw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQ0KLS0tIGRldi9kcm0vZHJtX3N5c2N0bC5jCShyZXZpc2lvbiAxOTA0
MDIpDQorKysgZGV2L2RybS9kcm1fc3lzY3RsLmMJKHdvcmtpbmcgY29weSkNCkBAIC0zOCw2ICsz
OCw3IEBADQogc3RhdGljIGludAkgICBkcm1fdm1faW5mbyBEUk1fU1lTQ1RMX0hBTkRMRVJfQVJH
UzsNCiBzdGF0aWMgaW50CSAgIGRybV9jbGllbnRzX2luZm8gRFJNX1NZU0NUTF9IQU5ETEVSX0FS
R1M7DQogc3RhdGljIGludAkgICBkcm1fYnVmc19pbmZvIERSTV9TWVNDVExfSEFORExFUl9BUkdT
Ow0KK3N0YXRpYyBpbnQJICAgZHJtX3ZibGFua19pbmZvIERSTV9TWVNDVExfSEFORExFUl9BUkdT
Ow0KIA0KIHN0cnVjdCBkcm1fc3lzY3RsX2xpc3Qgew0KIAljb25zdCBjaGFyICpuYW1lOw0KQEAg
LTQ3LDYgKzQ4LDcgQEANCiAJeyJ2bSIsCSAgICBkcm1fdm1faW5mb30sDQogCXsiY2xpZW50cyIs
IGRybV9jbGllbnRzX2luZm99LA0KIAl7ImJ1ZnMiLCAgICBkcm1fYnVmc19pbmZvfSwNCisJeyJ2
YmxhbmsiLCAgICBkcm1fdmJsYW5rX2luZm99LA0KIH07DQogI2RlZmluZSBEUk1fU1lTQ1RMX0VO
VFJJRVMgKHNpemVvZihkcm1fc3lzY3RsX2xpc3QpL3NpemVvZihkcm1fc3lzY3RsX2xpc3RbMF0p
KQ0KIA0KQEAgLTMxMywzICszMTUsMjUgQEANCiAJZnJlZSh0ZW1wcHJpdnMsIERSTV9NRU1fRFJJ
VkVSKTsNCiAJcmV0dXJuIHJldGNvZGU7DQogfQ0KKw0KK3N0YXRpYyBpbnQgZHJtX3ZibGFua19p
bmZvIERSTV9TWVNDVExfSEFORExFUl9BUkdTDQorew0KKwlzdHJ1Y3QgZHJtX2RldmljZSAqZGV2
ID0gYXJnMTsNCisJY2hhciBidWZbMTI4XTsNCisJaW50IHJldGNvZGU7DQorCWludCBpOw0KKw0K
KwlEUk1fU1lTQ1RMX1BSSU5UKCJcbmNydGMgcmVmIGNvdW50ICAgIGxhc3QgICAgIGVuYWJsZWQg
aW5tb2Rlc2V0XG4iKTsNCisJZm9yKGkgPSAwIDsgaSA8IGRldi0+bnVtX2NydGNzIDsgaSsrKSB7
DQorCQlEUk1fU1lTQ1RMX1BSSU5UKCIgICUwMmQgICUwMmQgJTA4ZCAlMDhkICUwMmQgICAgICAl
MDJkXG4iLA0KKwkJICAgIGksIGF0b21pY19yZWFkKCZkZXYtPnZibGFua1tpXS5yZWZjb3VudCks
DQorCQkgICAgYXRvbWljX3JlYWQoJmRldi0+dmJsYW5rW2ldLmNvdW50KSwNCisJCSAgICBhdG9t
aWNfcmVhZCgmZGV2LT52YmxhbmtbaV0ubGFzdCksDQorCQkgICAgYXRvbWljX3JlYWQoJmRldi0+
dmJsYW5rW2ldLmVuYWJsZWQpLA0KKwkJICAgIGF0b21pY19yZWFkKCZkZXYtPnZibGFua1tpXS5p
bm1vZGVzZXQpKTsNCisJfQ0KKw0KKwlTWVNDVExfT1VUKHJlcSwgIiIsIC0xKTsNCitkb25lOg0K
KwlyZXR1cm4gcmV0Y29kZTsNCit9DQo=


--=-y8fcTc4EOZiPJ55QduKo--

--=-bx4tddrYyiuGGcswbPut
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (FreeBSD)

iEYEABECAAYFAknNQE4ACgkQM4TrQ4qfROPfnQCaAyc3MyGkmD79keZ+vnN/X2DH
OUAAmgOVV8PRKlbftTfInwM1SUvWsU4K
=i1Ol
-----END PGP SIGNATURE-----

--=-bx4tddrYyiuGGcswbPut--




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1238188110.8491.194.camel>