From owner-freebsd-stable@freebsd.org Tue Jul 19 01:18:26 2016 Return-Path: Delivered-To: freebsd-stable@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 3CE1FB9C8F9 for ; Tue, 19 Jul 2016 01:18:26 +0000 (UTC) (envelope-from markmi@dsl-only.net) Received: from asp.reflexion.net (outbound-mail-211-153.reflexion.net [208.70.211.153]) (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 D445417E5 for ; Tue, 19 Jul 2016 01:18:25 +0000 (UTC) (envelope-from markmi@dsl-only.net) Received: (qmail 17004 invoked from network); 19 Jul 2016 01:18:12 -0000 Received: from unknown (HELO mail-cs-02.app.dca.reflexion.local) (10.81.19.2) by 0 (rfx-qmail) with SMTP; 19 Jul 2016 01:18:12 -0000 Received: by mail-cs-02.app.dca.reflexion.local (Reflexion email security v7.90.4) with SMTP; Mon, 18 Jul 2016 21:18:13 -0400 (EDT) Received: (qmail 19829 invoked from network); 19 Jul 2016 01:18:12 -0000 Received: from unknown (HELO iron2.pdx.net) (69.64.224.71) by 0 (rfx-qmail) with (AES256-SHA encrypted) SMTP; 19 Jul 2016 01:18:12 -0000 X-No-Relay: not in my network X-No-Relay: not in my network X-No-Relay: not in my network X-No-Relay: not in my network Received: from [192.168.0.105] (ip70-189-131-151.lv.lv.cox.net [70.189.131.151]) by iron2.pdx.net (Postfix) with ESMTPSA id E399F1C407A; Mon, 18 Jul 2016 18:18:11 -0700 (PDT) From: Mark Millard Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable Subject: FYI: svn commit: r301453 - in head/sys: arm/arm arm64/arm64 dev/fdt dev/gpio dev/iicbus dev/ofw dev/pci dev/vnic kern mips/mips sys Message-Id: <73BDB217-7D9D-4D54-8DE6-A1DE3BA9FFD2@dsl-only.net> Date: Mon, 18 Jul 2016 18:18:16 -0700 To: freebsd-arm , FreeBSD PowerPC ML , FreeBSD Current , freebsd-stable@freebsd.org Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\)) X-Mailer: Apple Mail (2.3124) X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jul 2016 01:18:26 -0000 This is just an FYI copied from svn-src-head: = https://lists.freebsd.org/pipermail/svn-src-head/2016-July/089328.html = says the below. I'm just giving it somewhat wider exposure. Likely any = replies should be to = https://lists.freebsd.org/pipermail/svn-src-head/2016-July/089328.html = instead of to this message. Nathan Whitehorn nwhitehorn at freebsd.org on Tue Jul 19 00:11:53 UTC = 2016 wrote: > Could you please describe what this change is in more detail? >=20 > It breaks a lot of encapsulations we have worked very hard to = maintain,=20 > moves ARM code into MI parts of the kernel, and the OFW parts violate=20= > IEEE 1275 (the Open Firmware standard). In particular, there is no=20 > guarantee that the interrupts for a newbus (or OF) device are encoded = in=20 > a property called "interrupts" (or, indeed, in any property at all) on=20= > that node and there are many, many device trees where that is not the=20= > case (e.g. ones with interrupt maps, as well as Apple hardware). By=20 > putting that knowledge into the OF root bus device, which we have = tried=20 > to keep it out of, this enforces a standard that doesn't actually = exist. >=20 > I'm hesitant to ask for reversion on something that landed 6 weeks ago=20= > without me noticing, but this needs a lot more architectural work = before=20 > any parts of the kernel should use it. > -Nathan >=20 > On 06/05/16 09:20, Svatopluk Kraus wrote: > > Author: skra > > Date: Sun Jun 5 16:20:12 2016 > > New Revision: 301453 > > URL: https://svnweb.freebsd.org/changeset/base/301453 > > > > Log: > > INTRNG - change the way how an interrupt mapping data are = provided > > to the framework in OFW (FDT) case. > > > > This is a follow-up to r301451. > > > > Differential Revision: https://reviews.freebsd.org/D6634 > > > > Modified: > > head/sys/arm/arm/nexus.c > > head/sys/arm64/arm64/gic_v3.c > > head/sys/arm64/arm64/nexus.c > > head/sys/dev/fdt/simplebus.c > > head/sys/dev/gpio/ofw_gpiobus.c > > head/sys/dev/iicbus/ofw_iicbus.c > > head/sys/dev/ofw/ofw_bus_subr.c > > head/sys/dev/ofw/ofw_bus_subr.h > > head/sys/dev/ofw/ofwbus.c > > head/sys/dev/pci/pci_host_generic.c > > head/sys/dev/vnic/mrml_bridge.c > > head/sys/dev/vnic/thunder_mdio_fdt.c > > head/sys/kern/subr_intr.c > > head/sys/mips/mips/nexus.c > > head/sys/sys/intr.h . . . (I omit the actual modifications) . . . =3D=3D=3D Mark Millard markmi at dsl-only.net