From owner-freebsd-current@freebsd.org Wed Aug 29 13:33:06 2018 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 9F072108A811 for ; Wed, 29 Aug 2018 13:33:06 +0000 (UTC) (envelope-from rhurlin@gwdg.de) Received: from emailer.gwdg.de (emailer.gwdg.de [134.76.10.24]) (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 35FA08D405; Wed, 29 Aug 2018 13:33:06 +0000 (UTC) (envelope-from rhurlin@gwdg.de) Received: from excmbx-24.um.gwdg.de ([134.76.9.234] helo=email.gwdg.de) by mailer.gwdg.de with esmtp (Exim 4.90_1) (envelope-from ) id 1fv0an-00042n-32; Wed, 29 Aug 2018 15:33:05 +0200 Received: from krabat.raven.hur (91.8.156.212) by EXCMBX-24.um.gwdg.de (134.76.9.234) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1531.3; Wed, 29 Aug 2018 15:33:04 +0200 Subject: Re: r336921 broke booting on MBP 2017, EFIRT related To: Kyle Evans CC: , FreeBSD Current References: <499f05f4-4fab-9b31-5d37-83ecb554013c@yuripv.net> <23535142-fcd6-a37f-c1d6-70ae68b3872f@gwdg.de> From: Rainer Hurling Message-ID: <44b19fe0-8607-8002-674c-47548de32d5b@gwdg.de> Date: Wed, 29 Aug 2018 15:33:04 +0200 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:60.0) Gecko/20100101 Thunderbird/60.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: de-DE Content-Transfer-Encoding: 7bit X-Originating-IP: [91.8.156.212] X-ClientProxiedBy: EXCMBX-22.um.gwdg.de (134.76.9.232) To EXCMBX-24.um.gwdg.de (134.76.9.234) X-Virus-Scanned: (clean) by clamav X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.27 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: Wed, 29 Aug 2018 13:33:06 -0000 Am 29.08.18 um 15:25 schrieb Kyle Evans: > On Wed, Aug 29, 2018 at 8:20 AM Rainer Hurling wrote: >> >> Am 29.08.18 um 11:37 schrieb Yuri Pankov: >>> Hi, >>> >>> I've noticed that all recent snapshots (ALPHA3, ALPHA2, ALPHA1, >>> 20180802) fail to boot on MBP 2017: >>> >>> kbd0 at kbdmux0 >>> netmap: loaded module >>> nexus0 >>> >>> Fatal trap 12: page fault while in kernel mode >>> cpuid = 2: apic id = 02 >>> fault virtual address = 0x74c64a50 >>> fault code = supervisor read data, page not present >>> instruction pointer = 0x20: 0x7abece31 >>> stack pointer = 0x28: 0xffffffff82b2f7c0 >>> frame pointer = 0x28: 0xffffffff82b2f810 >>> code segment = base 0x0, limit 0xfffff, type 0x1b >>> = DPL 0, pres 1, long 1, def32 0, gran 1 >>> processor eflags = interrupt enabled, resume, IOPL = 0 >>> current process = 0 (swapper) >>> [ thread pid 0 tid 100000 ] >>> Stopped at 0x7abece31: calll *0x18(%rax) >>> db> >>> >>> Sadly, there's no support for internal keyboard yet (it's connected via >>> SPI), and external USB one stops working. >>> >>> A (not so quick) bisect is pointing at r336921, which enabled EFIRT. >>> >>> Some questions here: >>> - is this something that can/should be fixed? >>> - can we print some "enabling EFIRT" message to the console to make >>> guesses about the problem source a bit easier? >> >> >> I have almost exactly the same trap on a DELL Latitude E6520 with ALPHA3. > > Hmm... that's a good data point. I might have a nearby Dell on-hand > with same firmware to reproduce with, then. > >> Only _with_ 'OPTIONS EFIRT' enabled in the kernel _and_ deactivating it >> via efi.rt.disabled=1 in /boot/loader.conf, it works for me. >> >> An oddity is, that the spelling of the loader tuneable has to be >> efi.rt.disabled, not efi.rt_disabled (note the dot instead of an >> underscore!). The one with the underscore, as mentioned in UPDATING, >> does not work for me. Isn't this a typo somewhere in the code? >> > > The UPDATING entry was later amended to reflect the new spelling > ("efi.rt.disabled") Oops, I must have missed it. Thanks for the update. > > Thanks, > > Kyle Evans >