Re: [PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-05 Thread Josh Poimboeuf
On Fri, Aug 05, 2016 at 10:41:15AM -0500, Nilay Vaish wrote: > On 4 August 2016 at 17:22, Josh Poimboeuf wrote: > > There are two different pieces of code for starting a CPU: start_cpu0() > > and the end of secondary_startup_64(). They're identical except for the > > stack

Re: [PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-05 Thread Josh Poimboeuf
On Fri, Aug 05, 2016 at 10:41:15AM -0500, Nilay Vaish wrote: > On 4 August 2016 at 17:22, Josh Poimboeuf wrote: > > There are two different pieces of code for starting a CPU: start_cpu0() > > and the end of secondary_startup_64(). They're identical except for the > > stack setup. Combine the

Re: [PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-05 Thread Nilay Vaish
On 4 August 2016 at 17:22, Josh Poimboeuf wrote: > There are two different pieces of code for starting a CPU: start_cpu0() > and the end of secondary_startup_64(). They're identical except for the > stack setup. Combine the common parts into a shared start_cpu() > function.

Re: [PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-05 Thread Nilay Vaish
On 4 August 2016 at 17:22, Josh Poimboeuf wrote: > There are two different pieces of code for starting a CPU: start_cpu0() > and the end of secondary_startup_64(). They're identical except for the > stack setup. Combine the common parts into a shared start_cpu() > function. > > Signed-off-by:

[PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-04 Thread Josh Poimboeuf
There are two different pieces of code for starting a CPU: start_cpu0() and the end of secondary_startup_64(). They're identical except for the stack setup. Combine the common parts into a shared start_cpu() function. Signed-off-by: Josh Poimboeuf ---

[PATCH v2 04/44] x86/asm/head: use a common function for starting CPUs

2016-08-04 Thread Josh Poimboeuf
There are two different pieces of code for starting a CPU: start_cpu0() and the end of secondary_startup_64(). They're identical except for the stack setup. Combine the common parts into a shared start_cpu() function. Signed-off-by: Josh Poimboeuf --- arch/x86/kernel/head_64.S | 18