From owner-freebsd-current@freebsd.org Fri Jul 31 18:04:51 2015 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id C72F99B0907 for ; Fri, 31 Jul 2015 18:04:51 +0000 (UTC) (envelope-from ler@lerctr.org) Received: from thebighonker.lerctr.org (thebighonker.lerctr.org [IPv6:2001:470:1f0f:3ad:223:7dff:fe9e:6e8a]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "thebighonker.lerctr.org", Issuer "COMODO RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 9A6881B94; Fri, 31 Jul 2015 18:04:51 +0000 (UTC) (envelope-from ler@lerctr.org) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lerctr.org; s=lerami; h=Content-Transfer-Encoding:Content-Type:MIME-Version:Subject:References:In-Reply-To:Message-ID:Date:CC:To:From; bh=An9QwjfNh13W9Yqx3lzqYIK6pci/9Z4Ovxt+HYcZjqE=; b=C9DIq80I/wZs+yizbP1B7IfHYxNc4az3fQ1o8SESfnqYuYBsMq0mjvEIRKE50kNYBg9zGc4ehvqzZE0Oqzc53mcn9OCmvjyxBPbub4DGQmkfdgSXbVe5bmP7h2M0NKdusgvDrMvVteP2EalWlCBaG6bkdZDDGPvDibI0Nfs9Azw=; Received: from 66-87-121-151.pools.spcsdns.net ([66.87.121.151]:26778 helo=[51.70.229.151]) by thebighonker.lerctr.org with esmtpsa (TLSv1.2:DHE-RSA-AES256-SHA:256) (Exim 4.85 (FreeBSD)) (envelope-from ) id 1ZLEfq-000DBH-KM; Fri, 31 Jul 2015 13:04:51 -0500 From: Larry Rosenman To: Warner Losh CC: Glen Barber , Benno Rice , Date: Fri, 31 Jul 2015 13:04:49 -0500 Message-ID: <14ee5498308.2846.a6b027662ce9a8103f88b1e74d3c5524@lerctr.org> In-Reply-To: References: <20150730201351.GH90754@FreeBSD.org> <20150730201517.GI90754@FreeBSD.org> <20150730221728.GK90754@FreeBSD.org> <452d406c68674df3834c75bd3736e6dc@thebighonker.lerctr.org> <20150731041815.GO90754@FreeBSD.org> <20150731042114.GP90754@FreeBSD.org> <20150731065016.GT90754@FreeBSD.org> <20150731102721.GA824@oldtbh.lerctr.org> <20150731133214.GV90754@FreeBSD.org> <14ee48f81d8.2846.a6b027662ce9a8103f88b1e74d3c5524@lerctr.org> <9B59F252-D41D-4714-A983-356E525359FD@bsdimp.com> <14ee53e57a8.2846.a6b027662ce9a8103f88b1e74d3c5524@lerctr.org> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 AquaMail/1.5.7.29 (build: 21070094) Subject: Re: pmspcv panic on boot on this box MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Score: -1.0 (-) X-LERCTR-Spam-Score: -1.0 (-) X-Spam-Report: SpamScore (-1.0/5.0) ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 X-LERCTR-Spam-Report: SpamScore (-1.0/5.0) ALL_TRUSTED=-1, SHORTCIRCUIT=-0.0001 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 31 Jul 2015 18:04:51 -0000 I'll try it when I get back to Austin tomorrow. On July 31, 2015 1:03:43 PM Warner Losh wrote: > It replaces it. > > Or you could just add the #if 0 bits to remove the atomic_cmpset_32 from > the picture entirely. > > All this state saving should be done in attach anyway, so it is mis-located > here… But the current > patch will tell me if my theory of the crime is correct and offer a path > forward... > > Warner > > > > On Jul 31, 2015, at 11:52 AM, Larry Rosenman wrote: > > > > Is this with or without Benno's patch? > > > > > > On July 31, 2015 12:24:11 PM Warner Losh wrote: > > > >> Try the following patch. There’s a fundamental misunderstanding of > newbus that’s screwing things up… > >> > >> Also available at http://people.freebsd.org/~imp/patch-queue/pms > >> > >> The problem is that the first time through for ahd0 we’re setting > cardMap[0] to 5. The second time through it is already 5, so we say ‘oh, > this has been probed before’ and return 2. This causes antiapi_probe() to > return 0, because the card has already been probed before. This is wrong on > so many levels, but I’ll suppress channelling my inner bde and stop here. > >> > >> Warner > >> > >> diff -r 1805eb187340 sys/dev/pms/freebsd/driver/common/lxutil.c > >> --- a/sys/dev/pms/freebsd/driver/common/lxutil.c > >> +++ b/sys/dev/pms/freebsd/driver/common/lxutil.c > >> @@ -757,18 +757,25 @@ STATIC int agtiapi_ProbeCard( device_t d > >> { > >> int idx; > >> static U32 cardMap[4] = { 0, 0, 0, 0 }; > >> + u_int16_t agtiapi_vendor; // PCI vendor ID > >> u_int16_t agtiapi_dev; // PCI device ID > >> AGTIAPI_PRINTK("agtiapi_ProbeCard: start\n"); > >> > >> +#if 0 > >> if ( ! atomic_cmpset_32( &cardMap[thisCard], 0, 5 ) ) { // card > already ran > >> AGTIAPI_PRINTK( "We'll only ID this card once -- %d\n", thisCard ); > >> return 2; // error return value; card already ran this function > >> } > >> else { > >> +#else > >> + { > >> +#endif > >> + agtiapi_vendor = pci_get_vendor( dev ); // get PCI vendor ID > >> agtiapi_dev = pci_get_device( dev ); // get PCI device ID > >> for( idx = 0; idx < COUNT(ag_card_type); idx++ ) > >> { > >> - if( ag_card_type[idx].deviceId == agtiapi_dev ) > >> + if( ag_card_type[idx].deviceId == agtiapi_dev && > >> + ag_card_type[idx].vendorId == agtiapi_vendor) > >> { // device ID match > >> memset( (void *)&agCardInfoList[ thisCard ], 0, > >> sizeof(ag_card_info_t) ); > >> > >> > >> > >> > On Jul 31, 2015, at 8:41 AM, Larry Rosenman wrote: > >> > > >> > Please do pull it from GENERIC until this is fixed in HEAD and RELENG/10. > >> > > >> > > >> > On July 31, 2015 8:32:17 AM Glen Barber wrote: > >> > > >> >> On Fri, Jul 31, 2015 at 05:27:22AM -0500, Larry Rosenman wrote: > >> >> > Ok, I made a GENERIC-NOPMS, without the device pmspcv, and adjusted > my custom > >> >> > to include GENERIC-NOPMS. And we boot (I'm typing this from a ssh > session > >> >> > to the box). > >> >> > > >> >> > >> >> Larry, thank you very much for testing this. > >> >> > >> >> Benno, for 10.2-RELEASE, I think we're going to pull pmspcv from GENERIC > >> >> and issue an EN for the driver update when this is fixed. > >> >> > >> >> I think this should be pulled from GENERIC in head and stable/10 in the > >> >> meantime, as well. > >> >> > >> >> Glen > >> >> > >> > > >> > > >> > _______________________________________________ > >> > freebsd-current@freebsd.org mailing list > >> > http://lists.freebsd.org/mailman/listinfo/freebsd-current > >> > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org" > >> > > > > >