Re: [PATCHv6 16/16] x86/acpi: Add support for CPU offlining for ACPI MADT wakeup method

From: kirill.shutemov@xxxxxxxxxxxxxxx
Date: Sat Jan 27 2024 - 13:16:28 EST


On Fri, Jan 26, 2024 at 02:21:30PM +0000, Huang, Kai wrote:
> > +static int __init acpi_mp_setup_reset(u64 reset_vector)
> > +{
> > + pgd_t *pgd;
> > + struct x86_mapping_info info = {
> > + .alloc_pgt_page = alloc_pgt_page,
> > + .free_pgt_page = free_pgt_page,
> > + .page_flag = __PAGE_KERNEL_LARGE_EXEC,
> > + .kernpg_flag = _KERNPG_TABLE_NOENC,
> > + };
>
> Nit: Reverse Christmas-tree style
>
> [...]

Okay, will fix if new version is required.

> >
> > -#define ACPI_MP_WAKE_COMMAND_WAKEUP 1
> > +#define ACPI_MP_WAKE_COMMAND_WAKEUP 1
>
> Nit: Is this change intended?

Yes. Changed to indentation with tabs.

--
Kiryl Shutsemau / Kirill A. Shutemov