From: Ingo Molnar <mingo@kernel.org>
To: Adam Williamson <awilliam@redhat.com>,
Joerg Roedel <joro@8bytes.org>,
David Woodhouse <David.Woodhouse@intel.com>
Cc: Dave Jones <davej@redhat.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
Lan Tianyu <tianyu.lan@intel.com>,
Matthew Garrett <mjg@redhat.com>, Peter Anvin <hpa@zytor.com>,
the arch/x86 maintainers <x86@kernel.org>,
Robin Holt <holt@sgi.com>,
davej@fedoraproject.org, Len Brown <lenb@kernel.org>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
Linux ACPI <linux-acpi@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default
Date: Sat, 26 Oct 2013 11:15:01 +0200 [thread overview]
Message-ID: <20131026091501.GE14237@gmail.com> (raw)
In-Reply-To: <1382772154.3604.1.camel@adam.happyassassin.net>
* Adam Williamson <awilliam@redhat.com> wrote:
> On Fri, 2013-10-25 at 23:17 -0400, Dave Jones wrote:
> > On Fri, Oct 25, 2013 at 07:20:15PM -0700, Adam Williamson wrote:
> > > OK, so, findings: F16 and F17 live images reboot quickly. F18 is the
> > > first that reboots slowly.
> >
> > wait, stop. what does "reboots slowly" mean ?
> > In every one of the failure I've seen, without a quirk it means reboot
> > doesn't happen at all.
>
> If you read the original report - on Z1s, if you leave the system alone,
> it sits at the 'Restarting system.' message for about two minutes, then
> finally reboots. If you pass 'reboot=pci', reboot is as fast as you'd
> expect.
>
> https://bugzilla.kernel.org/show_bug.cgi?id=61721
So, the best range for the breakage we have is v3.3..v3.5.
As a blind shot into the dark, that turns out to be a range when a
number of irq-remapping, vt-d changes went upstream:
comet:~/tip> gll v3.3..v3.6 arch/x86 | grep -i remap
399988eea194 irq_remap: Fix compiler warning with CONFIG_IRQ_REMAP=y
79fec2c557cf Merge tag 'intr-remapping-ops-for-ingo' of git://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu into core/iommu
8a8f422d3b4f iommu: rename intr_remapping.[ch] to irq_remapping.[ch]
95a02e976c39 iommu: rename intr_remapping references to irq_remapping
263b5e8629c9 x86, iommu/vt-d: Clean up interfaces for interrupt remapping
5e2b930b0784 iommu/vt-d: Convert MSI remapping setup to remap_ops
9d619f657222 iommu/vt-d: Convert free_irte into a remap_ops callback
4c1bad6a0af1 iommu/vt-d: Convert IR set_affinity function to remap_ops
0c3f173a88c4 iommu/vt-d: Convert IR ioapic-setup to use remap_ops
4f3d8b67ad30 iommu/vt-d: Convert missing apic.c intr-remapping call to remap_ops
736baef4472d iommu/vt-d: Make intr-remapping initialization generic
f7219a5300ba x86: Introduce CONFIG_X86_DMA_REMAP
So if you are able to test current kernels, it might be an
additional data point to see whether the reboot delay (which appears
to be a reboot hang on other systems) is related to the following
kernel option:
CONFIG_IRQ_REMAP=y
(CONFIG_X86_DMA_REMAP is off on Fedora.)
IRQ_REMAP is somewhat of a dual personality feature, living half in
arch/x86/, half in drivers/iommu/. It should normally matter for
servers more than for laptops.
Btw., regarding your encrypted partitions boot problem, do you have
any non-encrypted filesystem? If yes then you could copy /[s]bin and
/lib to it and boot via init=/bin/bash, you ought to get a minimal
shell and be able to run /sbin/reboot.
Thanks,
Ingo
next prev parent reply other threads:[~2013-10-26 9:15 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-24 6:49 [PATCH] x86/ACPI: Make Sony Vaio Z1 series to use "reboot=pci" default tianyu.lan
2013-10-25 10:53 ` Ingo Molnar
2013-10-25 12:44 ` Dave Jones
2013-10-25 12:48 ` H. Peter Anvin
2013-10-25 12:54 ` Linus Torvalds
2013-10-25 16:47 ` Adam Williamson
2013-10-25 17:16 ` Linus Torvalds
2013-10-25 17:43 ` Adam Williamson
2013-10-25 19:44 ` Rafael J. Wysocki
2013-10-26 1:27 ` Adam Williamson
2013-10-26 2:20 ` Adam Williamson
2013-10-26 3:17 ` Dave Jones
2013-10-26 7:22 ` Adam Williamson
2013-10-26 9:15 ` Ingo Molnar [this message]
2013-10-26 17:08 ` Adam Williamson
2013-10-26 20:00 ` Linus Torvalds
2013-10-27 7:06 ` Adam Williamson
2013-11-01 14:02 ` Joerg Roedel
2013-11-18 23:43 ` Adam Williamson
2013-11-19 7:03 ` Ingo Molnar
2013-10-26 12:58 ` Lan Tianyu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131026091501.GE14237@gmail.com \
--to=mingo@kernel.org \
--cc=David.Woodhouse@intel.com \
--cc=awilliam@redhat.com \
--cc=davej@fedoraproject.org \
--cc=davej@redhat.com \
--cc=holt@sgi.com \
--cc=hpa@zytor.com \
--cc=joro@8bytes.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg@redhat.com \
--cc=rafael.j.wysocki@intel.com \
--cc=rjw@rjwysocki.net \
--cc=tglx@linutronix.de \
--cc=tianyu.lan@intel.com \
--cc=torvalds@linux-foundation.org \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.