Received-SPF: pass (google.com: domain of linux-kernel+bounces-178180-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <97bd95480a8b9951edc9ee2d2648d1b9c574e3b0.camel@redhat.com>
Subject: Re: Early boot regression from f0551af0213 ("x86/topology: Ignore
 non-present APIC IDs in a present package")
From: Lyude Paul <lyude@redhat.com>
To: Thomas Gleixner <tglx@linutronix.de>, "Linux regression tracking
 (Thorsten Leemhuis)" <regressions@leemhuis.info>
Cc: x86@kernel.org, linux-kernel@vger.kernel.org, Mario Limonciello
	 <mario.limonciello@amd.com>, Borislav Petkov <bp@alien8.de>, Linux kernel
 regressions list <regressions@lists.linux.dev>
Date: Mon, 13 May 2024 19:32:56 -0400
In-Reply-To: <d3fe5278e7cd5af6c62b470b281b547b67e3959a.camel@redhat.com>
References: <3d77cb89857ee43a9c31249f4eab7196013bc4b4.camel@redhat.com>
	 <20240418082703.GCZiDZVyra7qOQbyqn@fat_crate.local>
	 <fd040809d95b3e12b2fdc78a2409e187716bc66f.camel@redhat.com>
	 <87plumxz4x.ffs@tglx>
	 <abbb7d7ca781f6c664e4c5b1dffc19394ac79691.camel@redhat.com>
	 <87le59vw1y.ffs@tglx>
	 <3a0afe545747e5314a9cb6bbaa9ce90b259ddfac.camel@redhat.com>
	 <87edautcmz.ffs@tglx>
	 <3b1d16e357c1f9badeef405366492f05af26c085.camel@redhat.com>
	 <878r11t8zu.ffs@tglx> <016902d9-3858-4c65-b3ec-f7a5103af63c@amd.com>
	 <51d0dff8-2888-463c-95ab-71b491f12a8f@leemhuis.info> <877cg4ppd5.ffs@tglx>
	 <ea927dad269cc21de1d0baf3d6c9f66ee025b862.camel@redhat.com>
	 <d2c6f335a6eb5892b0d894d5df4a6e713fa013b5.camel@redhat.com>
	 <87jzjxn6s5.ffs@tglx>
	 <d3fe5278e7cd5af6c62b470b281b547b67e3959a.camel@redhat.com>
Organization: Red Hat Inc.
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.52.1 (3.52.1-1.fc40) 
Precedence: bulk
MIME-Version: 1.0

Oh! I am not sure what changed, but I realized that you might also want to =
see
the serial output from the debugging patch that you had sent me a while ago=
.
So I just built that against the 6.9 kernel and it actually seems to boot
without needing me to pass intremap=3Doff. So it seems like that might actu=
ally
be a fix!

On Mon, 2024-05-13 at 19:18 -0400, Lyude Paul wrote:
> On Mon, 2024-05-13 at 16:08 +0200, Thomas Gleixner wrote:
> > On Wed, May 08 2024 at 19:21, Lyude Paul wrote:
> > > Regarding the test results - I tried possible_cpus all the way up to =
17
> > > and nothing got the machine to boot. However, possible_cpus=3D8
> > > intremap=3Doff did get the machine to boot successfully
> >=20
> > Oh. That's interesting.
> >=20
> > Does v6.9 (released yesterday) boot with just 'intremap=3Doff' too?
>=20
> Yes - it still boots. As well I finally got the serial console adapter in=
, but
> I don't see any additional output:
>=20
> [    0.146896] printk: legacy console [tty0] enabled
> [    0.147433] printk: legacy console [ttyS0] enabled
> [    1.951453] ACPI: Core revision 20230628
> [    1.955466] clocksource: hpet: mask: 0xffffffff max_cycles: 0xffffffff=
, max_idle_ns: 133484873504 ns
> [    1.964698] APIC: Switch to symmetric I/O mode setup
> [    1.970973] AMD-Vi: Using global IVHD EFR:0xf77ef22294ada, EFR2:0x0
> [    1.978491] ..TIMER: vector=3D0x30 apic1=3D0 pin1=3D2 apic2=3D-1 pin2=
=3D-1
> [    1.996945] Kernel panic - not syncing: timer doesn't work through Int=
errupt-remapped IO-APIC
> [    2.005376] CPU: 0 PID: 0 Comm: swapper/0 Not tainted 6.9.0Lyude-Test+=
 #7
> [    2.012248] Hardware name: MSI MS-7A39/A320M GAMING PRO (MS-7A39), BIO=
S 1.I0 01/22/2019
> [    2.020160] Call Trace:
> [    2.022584]  <TASK>
> [    2.024666]  dump_stack_lvl+0x5d/0x80
> [    2.028499]  panic+0x118/0x2c8
> [    2.031522]  ? timer_irq_works+0x54/0xf0
> [    2.035408]  panic_if_irq_remap.cold+0x5/0x9
> [    2.039631]  setup_IO_APIC+0x398/0x860
> [    2.043475]  ? _raw_spin_unlock_irqrestore+0x1d/0x40
> [    2.048393]  ? clear_IO_APIC_pin+0x125/0x1e0
> [    2.052617]  apic_intr_mode_init+0xcc/0xe0
> [    2.056672]  x86_late_time_init+0x24/0x40
> [    2.060820]  start_kernel+0x8be/0x960
> [    2.064446]  x86_64_start_reservations+0x24/0x30
> [    2.069015]  x86_64_start_kernel+0x95/0xa0
> [    2.073069]  common_startup_64+0x13e/0x141
> [    2.077280]  </TASK>
> [    2.079447] ---[ end Kernel panic - not syncing: timer doesn't work th=
rough Interrupt-remapped IO-APIC ]---
>=20
>=20
> >=20
> > Thanks,
> >=20
> >         tglx
> >=20
>=20

--=20
Cheers,
 Lyude Paul (she/her)
 Software Engineer at Red Hat