Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-16 Thread Juergen Gross
On 16.09.21 12:29, Borislav Petkov wrote: On Thu, Sep 16, 2021 at 12:09:27PM +0300, Mike Rapoport wrote: I think the first sentence about reserving memory before memblock allocations are possible is important and I think we should keep it. I expanded that comment this way: /*

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-16 Thread Borislav Petkov
On Thu, Sep 16, 2021 at 12:09:27PM +0300, Mike Rapoport wrote: > I think the first sentence about reserving memory before memblock > allocations are possible is important and I think we should keep it. I expanded that comment this way: /* * Do some memory reservations *before*

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-16 Thread Mike Rapoport
On Wed, Sep 15, 2021 at 01:00:20PM +0200, Borislav Petkov wrote: > You forgot to Cc Mike, lemme add him. > > And drop stable@ too. > > On Tue, Sep 14, 2021 at 01:06:22PM +0200, Juergen Gross wrote: > > On 14.09.21 12:03, Jan Beulich wrote: > > > On 14.09.2021 11:41, Juergen Gross wrote: > > > >

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-15 Thread Borislav Petkov
You forgot to Cc Mike, lemme add him. And drop stable@ too. On Tue, Sep 14, 2021 at 01:06:22PM +0200, Juergen Gross wrote: > On 14.09.21 12:03, Jan Beulich wrote: > > On 14.09.2021 11:41, Juergen Gross wrote: > > > Commit a799c2bd29d19c565 ("x86/setup: Consolidate early memory > > >

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-14 Thread Juergen Gross
On 14.09.21 12:03, Jan Beulich wrote: On 14.09.2021 11:41, Juergen Gross wrote: Commit a799c2bd29d19c565 ("x86/setup: Consolidate early memory reservations") introduced early_reserve_memory() to do all needed initial memblock_reserve() calls in one function. Unfortunately the call of

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-14 Thread Marek Marczykowski-Górecki
On Tue, Sep 14, 2021 at 11:41:08AM +0200, Juergen Gross wrote: > Commit a799c2bd29d19c565 ("x86/setup: Consolidate early memory > reservations") introduced early_reserve_memory() to do all needed > initial memblock_reserve() calls in one function. Unfortunately the > call of early_reserve_memory()

Re: [PATCH] x86/setup: call early_reserve_memory() earlier

2021-09-14 Thread Jan Beulich
On 14.09.2021 11:41, Juergen Gross wrote: > Commit a799c2bd29d19c565 ("x86/setup: Consolidate early memory > reservations") introduced early_reserve_memory() to do all needed > initial memblock_reserve() calls in one function. Unfortunately the > call of early_reserve_memory() is done too late for