Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 7 Feb 2017 19:10:58 +0000
From:      Andrew Turner <andrew@fubar.geek.nz>
To:        John Baldwin <jhb@freebsd.org>
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>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 06 Feb 2017 10:43:09 -0800
John Baldwin <jhb@freebsd.org> 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



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