Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-04-04 Thread Stephen Boyd
Quoting Will Deacon (2017-03-23 07:22:39) > On Fri, Feb 24, 2017 at 05:39:08PM -0800, Stephen Boyd wrote: > > Quoting James Morse (2017-02-20 03:10:10) > > > > > > You're right the user-address side of things will get caught in > > > do_page_fault(). > > > I was trying to badly-explain

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-04-04 Thread Stephen Boyd
Quoting Will Deacon (2017-03-23 07:22:39) > On Fri, Feb 24, 2017 at 05:39:08PM -0800, Stephen Boyd wrote: > > Quoting James Morse (2017-02-20 03:10:10) > > > > > > You're right the user-address side of things will get caught in > > > do_page_fault(). > > > I was trying to badly-explain

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-03-23 Thread Will Deacon
Hi Stephen, On Fri, Feb 24, 2017 at 05:39:08PM -0800, Stephen Boyd wrote: > Quoting James Morse (2017-02-20 03:10:10) > > On 17/02/17 15:53, Stephen Boyd wrote: > > > Quoting James Morse (2017-02-17 03:00:39) > > >> On 17/02/17 01:19, Stephen Boyd wrote: > > >>> diff --git a/arch/arm64/mm/fault.c

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-03-23 Thread Will Deacon
Hi Stephen, On Fri, Feb 24, 2017 at 05:39:08PM -0800, Stephen Boyd wrote: > Quoting James Morse (2017-02-20 03:10:10) > > On 17/02/17 15:53, Stephen Boyd wrote: > > > Quoting James Morse (2017-02-17 03:00:39) > > >> On 17/02/17 01:19, Stephen Boyd wrote: > > >>> diff --git a/arch/arm64/mm/fault.c

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-24 Thread Stephen Boyd
Quoting James Morse (2017-02-20 03:10:10) > Hi Stephen, > > On 17/02/17 15:53, Stephen Boyd wrote: > > Quoting James Morse (2017-02-17 03:00:39) > >> On 17/02/17 01:19, Stephen Boyd wrote: > >>> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c > >>> index 156169c6981b..8bd4e7f11c70

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-24 Thread Stephen Boyd
Quoting James Morse (2017-02-20 03:10:10) > Hi Stephen, > > On 17/02/17 15:53, Stephen Boyd wrote: > > Quoting James Morse (2017-02-17 03:00:39) > >> On 17/02/17 01:19, Stephen Boyd wrote: > >>> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c > >>> index 156169c6981b..8bd4e7f11c70

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-20 Thread James Morse
Hi Stephen, On 17/02/17 15:53, Stephen Boyd wrote: > Quoting James Morse (2017-02-17 03:00:39) >> On 17/02/17 01:19, Stephen Boyd wrote: >>> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c >>> index 156169c6981b..8bd4e7f11c70 100644 >>> --- a/arch/arm64/mm/fault.c >>> +++

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-20 Thread James Morse
Hi Stephen, On 17/02/17 15:53, Stephen Boyd wrote: > Quoting James Morse (2017-02-17 03:00:39) >> On 17/02/17 01:19, Stephen Boyd wrote: >>> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c >>> index 156169c6981b..8bd4e7f11c70 100644 >>> --- a/arch/arm64/mm/fault.c >>> +++

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-17 Thread James Morse
Hi Stephen, On 17/02/17 01:19, Stephen Boyd wrote: > If a page is marked read only we should print out that fact, > instead of printing out that there was a page fault. Right now we > get a cryptic error message that something went wrong with an > unhandled fault, but we don't evaluate the esr to

Re: [PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-17 Thread James Morse
Hi Stephen, On 17/02/17 01:19, Stephen Boyd wrote: > If a page is marked read only we should print out that fact, > instead of printing out that there was a page fault. Right now we > get a cryptic error message that something went wrong with an > unhandled fault, but we don't evaluate the esr to

[PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-16 Thread Stephen Boyd
If a page is marked read only we should print out that fact, instead of printing out that there was a page fault. Right now we get a cryptic error message that something went wrong with an unhandled fault, but we don't evaluate the esr to figure out that it was a read/write permission fault.

[PATCH v2] arm64: print a fault message when attempting to write RO memory

2017-02-16 Thread Stephen Boyd
If a page is marked read only we should print out that fact, instead of printing out that there was a page fault. Right now we get a cryptic error message that something went wrong with an unhandled fault, but we don't evaluate the esr to figure out that it was a read/write permission fault.