From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44050) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W6jtr-0001oI-9i for qemu-devel@nongnu.org; Fri, 24 Jan 2014 11:46:40 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W6jtk-00042O-TD for qemu-devel@nongnu.org; Fri, 24 Jan 2014 11:46:35 -0500 Received: from mail-qc0-x232.google.com ([2607:f8b0:400d:c01::232]:43653) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W6jtk-00041c-Nq for qemu-devel@nongnu.org; Fri, 24 Jan 2014 11:46:28 -0500 Received: by mail-qc0-f178.google.com with SMTP id m20so4641863qcx.23 for ; Fri, 24 Jan 2014 08:46:26 -0800 (PST) Date: Fri, 24 Jan 2014 11:46:09 -0500 From: "Gabriel L. Somlo" Message-ID: <20140124164608.GB1293@ERROL.INI.CMU.EDU> References: <20140120212517.GD18508@ERROL.INI.CMU.EDU> <52DE4CDC.4070501@redhat.com> <20140121110202.GA22693@redhat.com> <52DE5471.5090901@redhat.com> <20140121114454.GC22693@redhat.com> <20140121181101.GB1323@ERROL.INI.CMU.EDU> <20140121183851.GA26382@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140121183851.GA26382@redhat.com> Subject: Re: [Qemu-devel] [PATCH] ACPI: Add IRQ resource to HPET._CRS on Mac OS X List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: imammedo@redhat.com, agraf@suse.de, lersek@redhat.com, qemu-devel@nongnu.org, pbonzini@redhat.com On Tue, Jan 21, 2014 at 08:38:51PM +0200, Michael S. Tsirkin wrote: > On Tue, Jan 21, 2014 at 01:11:01PM -0500, Gabriel L. Somlo wrote: > > Apple hardware invariably adds "IRQNoFlags() {0, 8}" to HPET._CRS, > > and, at least on piix+smp, an OS X guest will panic unless IRQNoFlags > > is present. On the other hand, Windows XP bluescreens whenever > > IRQNoFlags is present. This patch conditionally includes IRQNoFlags > > only when detecting the presence of an AppleSMC (as a heuristic > > predictor for running a Mac OS X guest). Querying _OS and/or _OSI > > directly was considered and rejected because such queries MAY NOT > > be stateless from the OSPM standpoint, leading to potentially > > unpredictable behavior. > > > > Signed-off-by: Gabriel Somlo > > Fine with me. > Acked-by: Michael S. Tsirkin > > I'll pick this up if no one complains. In the mean time I updated the bootloader I was using (chameleon) to the latest svn (2345), and was able to bring up Lion in addition to SnowLeopard. On Lion, without this patch (i.e. without IRQNoFlags on the HPET), all four combinations (piix vs q35) X (up vs. smp) work fine. I'd like to hold off on having this applied for now, until I get a chance to figure out whether it matters moving forward (MountainLion, Mavericks, etc). I'm not sure it's worth polluting the HPET DSDT node with the "(LEqual(\_SB.PCI0.ISA.SMC._STA, 0x0B))" hack, only to support (piix + smp) on an old version of OS X. I'll follow up if anything new and interesting happens, but please ignore it it for now... Thanks, --Gabriel