Re: [PATCH 08/28] nios2: MMU Fault handling

[Ley Foon Tan]

On Thu, Apr 24, 2014 at 11:22 PM, Ezequiel Garcia
<ezequiel.garcia@xxxxxxxxxxxxxxxxxx> wrote:
>> > I found that it's useful to add some printing here, just as ARM
>> > does. I carry this patch on my kernel:
>> >
>> > +               printk(KERN_INFO "%s: unhandled page fault (%d) at 0x%08lx, cause %ld\n",
>> > +                      current->comm, SIGSEGV, address, cause);
>> > +               show_regs(regs);
>> >
>> > Do you think we could do something like it? Maybe with a compile time option?
>> Yes, this is useful debug message. I can add this.
>> Prefer not to add new compile time option, I think this shouldn't
>> happen frequently.
>
> No, it shouldn't happen frequently. You can take a look at what ARM does.
> They use a DEBUG_USER compile-time option, much more complex than my silly
> proposal.
For now, we can print it always if this shouldn't happen frequently.
Compile time option can be added later if we have more other debug
stuff to print later.

Thanks.

Regard,
Ley Foon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/