Commit 05d694ea authored by Michael Neuling's avatar Michael Neuling Committed by Benjamin Herrenschmidt

powerpc: Add length setting to set_dawr

Currently we set the length field in the DAWR to 0 which defaults it to one
double word (64bits) which is the same as the DABR.

Change this so that we can set it to longer values as supported by the DAWR.
Signed-off-by: default avatarMichael Neuling <mikey@neuling.org>
Signed-off-by: default avatarBenjamin Herrenschmidt <benh@kernel.crashing.org>
parent f53d168c
...@@ -409,7 +409,7 @@ static inline int set_dabr(struct arch_hw_breakpoint *brk) ...@@ -409,7 +409,7 @@ static inline int set_dabr(struct arch_hw_breakpoint *brk)
static inline int set_dawr(struct arch_hw_breakpoint *brk) static inline int set_dawr(struct arch_hw_breakpoint *brk)
{ {
unsigned long dawr, dawrx; unsigned long dawr, dawrx, mrd;
dawr = brk->address; dawr = brk->address;
...@@ -419,6 +419,14 @@ static inline int set_dawr(struct arch_hw_breakpoint *brk) ...@@ -419,6 +419,14 @@ static inline int set_dawr(struct arch_hw_breakpoint *brk)
<< (63 - 59); //* translate */ << (63 - 59); //* translate */
dawrx |= (brk->type & (HW_BRK_TYPE_PRIV_ALL)) \ dawrx |= (brk->type & (HW_BRK_TYPE_PRIV_ALL)) \
>> 3; //* PRIM bits */ >> 3; //* PRIM bits */
/* dawr length is stored in field MDR bits 48:53. Matches range in
doublewords (64 bits) baised by -1 eg. 0b000000=1DW and
0b111111=64DW.
brk->len is in bytes.
This aligns up to double word size, shifts and does the bias.
*/
mrd = ((brk->len + 7) >> 3) - 1;
dawrx |= (mrd & 0x3f) << (63 - 53);
if (ppc_md.set_dawr) if (ppc_md.set_dawr)
return ppc_md.set_dawr(dawr, dawrx); return ppc_md.set_dawr(dawr, dawrx);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment