Re: [PATCH 2/2] x86/mm: replace a goto by merging two if clause

2019-10-01 Thread Wei Yang
On Mon, Sep 23, 2019 at 11:22:31AM +0200, Peter Zijlstra wrote: >On Thu, Sep 19, 2019 at 10:08:44AM +0800, Wei Yang wrote: >> There is only one place to use good_area jump, which could be reduced by >> merging the following two if clause. >> >> Signed-off-by: Wei Yang >> --- >> arch/x86/mm/fault

Re: [PATCH 2/2] x86/mm: replace a goto by merging two if clause

2019-09-23 Thread Wei Yang
On Mon, Sep 23, 2019 at 11:22:31AM +0200, Peter Zijlstra wrote: >On Thu, Sep 19, 2019 at 10:08:44AM +0800, Wei Yang wrote: >> There is only one place to use good_area jump, which could be reduced by >> merging the following two if clause. >> >> Signed-off-by: Wei Yang >> --- >> arch/x86/mm/fault

Re: [PATCH 2/2] x86/mm: replace a goto by merging two if clause

2019-09-23 Thread Peter Zijlstra
On Thu, Sep 19, 2019 at 10:08:44AM +0800, Wei Yang wrote: > There is only one place to use good_area jump, which could be reduced by > merging the following two if clause. > > Signed-off-by: Wei Yang > --- > arch/x86/mm/fault.c | 11 +-- > 1 file changed, 5 insertions(+), 6 deletions(-)

[PATCH 2/2] x86/mm: replace a goto by merging two if clause

2019-09-18 Thread Wei Yang
There is only one place to use good_area jump, which could be reduced by merging the following two if clause. Signed-off-by: Wei Yang --- arch/x86/mm/fault.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c index 9d18b73b5f