Re: [PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-31 Thread Christophe LEROY
Le 25/07/2017 à 13:19, Michael Ellerman a écrit : LEROY Christophe writes: Michael Ellerman a écrit : LEROY Christophe writes: Benjamin Herrenschmidt a écrit : When hitting below a VM_GROWSDOWN vma (typically growing the stack), we check whether it's a valid stack-growing instructio

Re: [PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-25 Thread Michael Ellerman
LEROY Christophe writes: > Michael Ellerman a écrit : > >> LEROY Christophe writes: >> >>> Benjamin Herrenschmidt a écrit : >>> When hitting below a VM_GROWSDOWN vma (typically growing the stack), we check whether it's a valid stack-growing instruction and we check the distance

Re: [PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-24 Thread LEROY Christophe
Michael Ellerman a écrit : LEROY Christophe writes: Benjamin Herrenschmidt a écrit : When hitting below a VM_GROWSDOWN vma (typically growing the stack), we check whether it's a valid stack-growing instruction and we check the distance to GPR1. This is largely open coded with lots of comm

Re: [PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-24 Thread Michael Ellerman
LEROY Christophe writes: > Benjamin Herrenschmidt a écrit : > >> When hitting below a VM_GROWSDOWN vma (typically growing the stack), >> we check whether it's a valid stack-growing instruction and we >> check the distance to GPR1. This is largely open coded with lots >> of comments, so move it o

Re: [PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-21 Thread LEROY Christophe
Benjamin Herrenschmidt a écrit : When hitting below a VM_GROWSDOWN vma (typically growing the stack), we check whether it's a valid stack-growing instruction and we check the distance to GPR1. This is largely open coded with lots of comments, so move it out to a helper. Did you have a look at

[PATCH 23/24] powerpc/mm: Cleanup check for stack expansion

2017-07-18 Thread Benjamin Herrenschmidt
When hitting below a VM_GROWSDOWN vma (typically growing the stack), we check whether it's a valid stack-growing instruction and we check the distance to GPR1. This is largely open coded with lots of comments, so move it out to a helper. While at it, make store_update_sp a boolean. Signed-off-by: