Franck Bui-Huu wrote:
>
> Signed-off-by: Franck Bui-Huu <fbuihuu@xxxxxxxxx>
> ---
> arch/mips/mm/tlbex.c | 83 +++++++++++++++----------------------------------
> 1 files changed, 26 insertions(+), 57 deletions(-)
>
> diff --git a/arch/mips/mm/tlbex.c b/arch/mips/mm/tlbex.c
> index 923515e..4775e4c 100644
> --- a/arch/mips/mm/tlbex.c
> +++ b/arch/mips/mm/tlbex.c
> @@ -705,6 +705,22 @@ il_bgez(u32 **p, struct reloc **r, unsigned int reg, enum label_id l)
> i_bgez(p, reg, 0);
> }
>
> +/*
> + * For debug purposes.
> + */
> +static inline void dump_handler(const u32 *handler, int count)
> +{
> + int i;
> +
> + pr_debug("\t.set push\n");
> + pr_debug("\t.set noreorder\n");
> +
> + for (i = 0; i < count; i++)
> + pr_debug("\t%p\t.word 0x%08x\n", &handler[i], handler[i]);
> +
> + pr_debug("\t.set pop\n");
> +}
I don't like this patch. I wrote the code to
a) print the handler before the (potentially fatal) memcpy. Touching
EBASE for the first time is a place where things like to go wrong.
b) avoid printing leading nops which are never executed. The trailing
nops have less potential for confusion.
Thiemo
