From owner-svn-src-head@freebsd.org Tue Feb 7 19:11:00 2017 Return-Path: Delivered-To: svn-src-head@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 BE0CACD4969; Tue, 7 Feb 2017 19:11:00 +0000 (UTC) (envelope-from andrew@fubar.geek.nz) Received: from kif.fubar.geek.nz (kif.fubar.geek.nz [178.62.119.249]) by mx1.freebsd.org (Postfix) with ESMTP id 8D274A57; Tue, 7 Feb 2017 19:11:00 +0000 (UTC) (envelope-from andrew@fubar.geek.nz) Received: from zapp (global-5-144.nat-2.net.cam.ac.uk [131.111.5.144]) by kif.fubar.geek.nz (Postfix) with ESMTPSA id 51FB9D8561; Tue, 7 Feb 2017 19:05:27 +0000 (UTC) Date: Tue, 7 Feb 2017 19:10:58 +0000 From: Andrew Turner To: John Baldwin Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r313342 - head/sys/conf Message-ID: <20170207191058.65d72285@zapp> In-Reply-To: <2448004.hnTc9TClBV@ralph.baldwin.cx> References: <201702061441.v16EfYZx010320@repo.freebsd.org> <2448004.hnTc9TClBV@ralph.baldwin.cx> X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.29; amd64-portbld-freebsd12.0) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 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: Tue, 07 Feb 2017 19:11:00 -0000 On Mon, 06 Feb 2017 10:43:09 -0800 John Baldwin wrote: > On Monday, February 06, 2017 02:41:34 PM Andrew Turner wrote: > > Author: andrew > > Date: Mon Feb 6 14:41:34 2017 > > New Revision: 313342 > > URL: https://svnweb.freebsd.org/changeset/base/313342 > > > > Log: > > Only build the ACPI PCI drivers on x86, they are unlikely to be > > used on arm64 without dignificant changes. > > > > Obtained from: ABT Systems Ltd > > Sponsored by: The FreeBSD Foundation > > I still think this is not really the right approach. Nothing about > _BBN, _CRS, or _PRT, etc. is x86-specific. > the main issue is the code assumes a single PCIe root controller, e.g. the pci_cfg* KPI won't work on ThunderX with 6 independent PCIe root controllers. I expect much of the code could be re-enabled on arm64, however for now it's easier to disable it. Andrew