[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20180208123418.GG30270@localhost.localdomain>
Date: Thu, 8 Feb 2018 20:34:18 +0800
From: Baoquan He <bhe@...hat.com>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
Cc: linux-kernel@...r.kernel.org, tglx@...utronix.de, mingo@...hat.com,
hpa@...or.com, x86@...nel.org, douly.fnst@...fujitsu.com,
joro@...tes.org, uobergfe@...hat.com, prarit@...hat.com
Subject: Re: [PATCH v2 1/2] x86/apic/kexec: Enable legacy irq mode before
jump to kexec/kdump kernel
On 02/07/18 at 01:48pm, Eric W. Biederman wrote:
> ebiederm@...ssion.com (Eric W. Biederman) writes:
> > Now that I see that I agree in essence with this patch series.
> > I don't agree with the implemenation details.
> >
> > Can you please split disable_IO_APIC and switch_to_legacy_irq_mode
> > in a single patch.
>
> Now that I think about it can you call the function not
> switch_to_legacy_irq_mode but restore_boot_irq_mode. And the
> corresponding x86_io_apic_ops not .disable but .restore.
>
> That should make the purpose of the code clearer, and help avoid
> mistakes like the one that led to this regression.
Thanks for your comments and great suggestion. Have made change as you
suggested, will post after test passed.
Thanks
Baoquan
Powered by blists - more mailing lists