Date: Mon, 22 Jun 2009 15:19:41 -0500 From: Robert Noland <rnoland@FreeBSD.org> To: John Baldwin <jhb@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r194644 - head/sys/dev/pci Message-ID: <1245701981.1761.25.camel@balrog.2hip.net> In-Reply-To: <1245701631.1761.22.camel@balrog.2hip.net> References: <200906222008.n5MK87qh050928@svn.freebsd.org> <1245701631.1761.22.camel@balrog.2hip.net>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-fbMZGfMTmUqzWQSDLiaR Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Mon, 2009-06-22 at 15:13 -0500, Robert Noland wrote: > On Mon, 2009-06-22 at 20:08 +0000, John Baldwin wrote: > > Author: jhb > > Date: Mon Jun 22 20:08:06 2009 > > New Revision: 194644 > > URL: http://svn.freebsd.org/changeset/base/194644 > >=20 > > Log: > > Enable MSI in the MSI capability registers any time that the first me= ssage > > in an MSI group is enabled, not just if the address/data pair are not > > initialized. >=20 > This should fix the issues with broken interrupts with drm on Intel > graphics chips using msi. That is the slow windows after VT Switch > issue. It however is still currently broken on HEAD due to another > issue, which will hopefully be resolved soon. It may actually work now on non-SMP, though I haven't tested that case. robert. > =20 > > Reported by: rnoland > > MFC after: 1 week > >=20 > > Modified: > > head/sys/dev/pci/pci.c > >=20 > > Modified: head/sys/dev/pci/pci.c > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > --- head/sys/dev/pci/pci.c Mon Jun 22 19:35:39 2009 (r194643) > > +++ head/sys/dev/pci/pci.c Mon Jun 22 20:08:06 2009 (r194644) > > @@ -2883,8 +2883,10 @@ pci_setup_intr(device_t dev, device_t ch > > goto bad; > > dinfo->cfg.msi.msi_addr =3D addr; > > dinfo->cfg.msi.msi_data =3D data; > > - pci_enable_msi(child, addr, data); > > } > > + if (dinfo->cfg.msi.msi_handlers =3D=3D 0) > > + pci_enable_msi(child, dinfo->cfg.msi.msi_addr, > > + dinfo->cfg.msi.msi_data); > > dinfo->cfg.msi.msi_handlers++; > > } else { > > KASSERT(dinfo->cfg.msix.msix_alloc > 0, --=20 Robert Noland <rnoland@FreeBSD.org> FreeBSD --=-fbMZGfMTmUqzWQSDLiaR 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) iEYEABECAAYFAko/510ACgkQM4TrQ4qfROOR2gCfR+TJJ3yQr82lIN2au+tOHSv7 XL4AnjGk3Q2PleTrMK91e1hpLc3rovNY =H3oi -----END PGP SIGNATURE----- --=-fbMZGfMTmUqzWQSDLiaR--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1245701981.1761.25.camel>