Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754673AbcDMWiP (ORCPT ); Wed, 13 Apr 2016 18:38:15 -0400 Received: from mail.kernel.org ([198.145.29.136]:33679 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753500AbcDMWiO (ORCPT ); Wed, 13 Apr 2016 18:38:14 -0400 Date: Wed, 13 Apr 2016 15:38:09 -0700 (PDT) From: Stefano Stabellini X-X-Sender: sstabellini@sstabellini-ThinkPad-X260 To: Boris Ostrovsky cc: Stefano Stabellini , Vitaly Kuznetsov , x86@kernel.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Jiang Liu , "K. Y. Srinivasan" , linux-kernel@vger.kernel.org, konrad.wilk@oracle.com, david.vrabel@citrix.com, jgross@suse.com, "Luis R. Rodriguez" Subject: Re: Xen regression, Was: [PATCH] x86/irq: Probe for PIC presence before allocating descs for legacy IRQs In-Reply-To: <570E99AF.1020705@oracle.com> Message-ID: References: <1446470676-1877-1-git-send-email-vkuznets@redhat.com> <570CF69A.1020701@oracle.com> <570D4645.20004@oracle.com> <570D69FE.6090200@oracle.com> <570D77D1.7080701@oracle.com> <570DA089.6060507@oracle.com> <570E838A.7000104@oracle.com> <570E99AF.1020705@oracle.com> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2457 Lines: 55 On Wed, 13 Apr 2016, Boris Ostrovsky wrote: > On 04/13/2016 01:36 PM, Boris Ostrovsky wrote: > > On 04/12/2016 09:27 PM, Boris Ostrovsky wrote: > > > On 04/12/2016 07:15 PM, Stefano Stabellini wrote: > > > > On Tue, 12 Apr 2016, Boris Ostrovsky wrote: > > > > > On 04/12/2016 05:56 PM, Stefano Stabellini wrote: > > > > > > I am not sure, maybe you didn't have CONFIG_SPARSE_IRQ? > > > > > > But I am certain that 4.6-rc2, with the attached config, fails as > > > > > > Dom0 > > > > > > on QEMU with the following sequence of calls: > > > > > I did have CONFIG_SPARSE_IRQ and I just rebuilt 4.5.0 with your config > > > > > (4.6-rc3 doesn't build for me for some reason) and that booted dom0 as > > > > > well. > > > > > > > > > > BTW, what do you mean by "dom0 on QEMU"? > > > > I am running Xen and Linux inside a QEMU x86_64 emulated machine > > > > (nested > > > > virt). > > > > > > This I, of course, never tried. > > > > > > But given that things work in a single-level virt, doesn't this imply that > > > perhaps there is something in the emulation that's not quite right? > > > > OK, so this *is* broken on single level virt as well. It's just that we > > always end up using AHCI so lack of irq 14 (and 15) does not affect the > > system. And I guess in QEMU case it's IDE only, right? > > > > You patch does fix this but I wonder if we could change something in > > probe_8259A() so that we can continue using nr_legacy_irqs(). Using > > nr_legacy_irqs() and NR_IRQS_LEGACY at the same time is inconsistent and may > > cause us headaches in the future. > > > > I think we could use paravirt_has() feature that was added for similar reason > when we had a problem with RTC (commit > d8c98a1d1488747625ad6044d423406e17e99b7a). So we add paravirt_has(PIC) which > will only be set by dom0 and then probe_8259A() will not set legacy_pic to > null_legacy_pic when this flag is set. Maybe we could introduce a legacy_pic_xen in arch/x86/kernel/i8259.c or arch/x86/xen/enlighten.c? We could set legacy_pic = legacy_pic_xen from start_kernel, so that we can skip probe_8259A completely. > Note that paravirt_has() is being removed by > http://lists.xenproject.org/archives/html/xen-devel/2016-04/msg01415.html so > presumably we'd use new struct x86_legacy_features instead (copying Luis so > that if this is acceptable he could add it to his next spin). I would prefer to come up with a fix that is backportable to 4.3, 4.4 and 4.5.