From owner-svn-src-head@FreeBSD.ORG Fri Sep 26 01:01:52 2014 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 06C78B93 for ; Fri, 26 Sep 2014 01:01:52 +0000 (UTC) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id DA9BDC22 for ; Fri, 26 Sep 2014 01:01:51 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.9/8.14.9) with ESMTP id s8Q11pRd081931 for ; Fri, 26 Sep 2014 01:01:51 GMT (envelope-from bdrewery@freefall.freebsd.org) Received: (from bdrewery@localhost) by freefall.freebsd.org (8.14.9/8.14.9/Submit) id s8Q11pTs081929 for svn-src-head@freebsd.org; Fri, 26 Sep 2014 01:01:51 GMT (envelope-from bdrewery) Received: (qmail 18114 invoked from network); 25 Sep 2014 20:01:46 -0500 Received: from unknown (HELO ?10.10.0.24?) (freebsd@shatow.net@10.10.0.24) by sweb.xzibition.com with ESMTPA; 25 Sep 2014 20:01:46 -0500 Message-ID: <5424BAF0.2000902@FreeBSD.org> Date: Thu, 25 Sep 2014 20:01:36 -0500 From: Bryan Drewery Organization: FreeBSD User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.1.1 MIME-Version: 1.0 To: Warner Losh Subject: Re: svn commit: r271201 - head/sys/dev/ahci References: <201409061820.s86IKpnw042259@svn.freebsd.org> <5424B0C0.30202@FreeBSD.org> <4A1B4605-B3AE-4B8D-B039-AD1A86B561B8@bsdimp.com> In-Reply-To: <4A1B4605-B3AE-4B8D-B039-AD1A86B561B8@bsdimp.com> OpenPGP: id=6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="sexQDWoQKTtDpARPcdoD4bs9tWlXrtmR0" Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 26 Sep 2014 01:01:52 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --sexQDWoQKTtDpARPcdoD4bs9tWlXrtmR0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 9/25/2014 7:59 PM, Warner Losh wrote: > Hey Bryan, >=20 > Sorry for the hassle. This didn=92t show up in the hardware I had, but = did show up in Mav=92s hardware. It was like this less than a day, I thou= ght=85 >=20 > Warner >=20 Yes only a day or two. I was just unlucky about it. Now I am able to get updated to latest head. >=20 > On Sep 25, 2014, at 5:18 PM, Bryan Drewery wrote= : >=20 >> This drove me crazy for the past few days! >> >> https://lists.freebsd.org/pipermail/freebsd-current/2014-September/052= 215.html >> >> Not sure about the first trace there, but the others I encountered wer= e >> due to this not being fixed yet. smh@ tracked the ordering down and th= en >> we found this. I couldn't try a newer build easily since my system wou= ld >> panic on just buildworld :) >> >> On 9/6/2014 1:20 PM, Warner Losh wrote: >>> Author: imp >>> Date: Sat Sep 6 18:20:50 2014 >>> New Revision: 271201 >>> URL: http://svnweb.freebsd.org/changeset/base/271201 >>> >>> Log: >>> Restore order of interrupt setup. Minor problems can result by >>> setting up the interrupts too early: >>> >>> Reviewed by: mav@ >>> Sponsored by: Netflix >>> >>> Modified: >>> head/sys/dev/ahci/ahci.c >>> head/sys/dev/ahci/ahci_pci.c >>> >>> Modified: head/sys/dev/ahci/ahci.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/ahci/ahci.c Sat Sep 6 18:08:21 2014 (r271200) >>> +++ head/sys/dev/ahci/ahci.c Sat Sep 6 18:20:50 2014 (r271201) >>> @@ -229,6 +229,15 @@ ahci_attach(device_t dev) >>> >>> ahci_ctlr_setup(dev); >>> >>> + /* Setup interrupts. */ >>> + if (ahci_setup_interrupt(dev)) { >>> + bus_dma_tag_destroy(ctlr->dma_tag); >>> + bus_release_resource(dev, SYS_RES_MEMORY, ctlr->r_rid, >>> + ctlr->r_mem); >>> + rman_fini(&ctlr->sc_iomem); >>> + return ENXIO; >>> + } >>> + >>> i =3D 0; >>> for (u =3D ctlr->ichannels; u !=3D 0; u >>=3D 1) >>> i +=3D (u & 1); >>> >>> Modified: head/sys/dev/ahci/ahci_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/ahci/ahci_pci.c Sat Sep 6 18:08:21 2014 (r271200) >>> +++ head/sys/dev/ahci/ahci_pci.c Sat Sep 6 18:20:50 2014 (r271201) >>> @@ -417,13 +417,6 @@ ahci_pci_attach(device_t dev) >>> ctlr->numirqs =3D 1; >>> } >>> >>> - if (ahci_setup_interrupt(dev)) { >>> - if (ctlr->msi) >>> - pci_release_msi(dev); >>> - bus_release_resource(dev, SYS_RES_MEMORY, ctlr->r_rid, ctlr->r_mem= ); >>> - return ENXIO; >>> - } >>> - >>> error =3D ahci_attach(dev); >>> if (error !=3D 0) >>> if (ctlr->msi) >>> >> >> >> --=20 >> Regards, >> Bryan Drewery >> >=20 --=20 Regards, Bryan Drewery --sexQDWoQKTtDpARPcdoD4bs9tWlXrtmR0 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (MingW32) iQEcBAEBAgAGBQJUJLrxAAoJEDXXcbtuRpfP6QkH/0Kg2rJdU4eKDIMVUa45fdRr tx2bHmsB6g3ka+E5L98uMoWxFOiRPSgqifEQYSyxkKn04n2hOarvyRLtmkNInAO6 bepmJAuWSva/K5W0F0wu9RgFqe+KI3284kopKmuG6cVVFS62n8MLy6rShmESe5Gv daVWbyPELbBZ8vfMBXWhOnfXv0ywzDKPZMrHTs+AOKd70KgCVUUVsG487ZGDOYT7 s9lLlcx/lTvg/fcr+KakaWtugvnKmqQV1Mbmmkt+gikQaw6IPXr/cUSQxKXlxCDc aBRocIugYryaDihnAX0YiDwZqljQRopP5mDnpciqtjHCPpZ82pd83O2IA0ERbCs= =32pZ -----END PGP SIGNATURE----- --sexQDWoQKTtDpARPcdoD4bs9tWlXrtmR0--