Re: Multiword instruction support and memory poll PC reporting.

Hi Oliver,

On Fri, Apr 25, 2008 at 10:43:18AM +0100, Oliver Ford wrote:
> Kevin O'Connor wrote:
>> I have a couple of questions on this:
>>
>> * why report the address on 'IRQS' polling? Wouldn't external
>> interrupts have essentially random pc values?
>>
> I see your point, this was just a quick addition that wasn't going to be
> permanent but turned out to be quite useful. I guess I could pass 0 in as
> faultaddr for the checkPolls(.., &data->irqpoll ..) line and then change
> the reporting to not report 0s.
>> * why report VA instead of MVA? Every other part of haret uses MVAs.
>>
>>
> I was using it to find out which modules where changing the registers I was
> watching. The LSMOD commands reports VAs and without knowing the PID of
> each process, the MVAs weren't much use to be.

Okay - I guess LSMOD does report VAs. However, ADDR2MOD takes MVAs,
and one can work out the pid/module from lsmod by using the same
formula that addr2mod uses.

>> I think its a great idea, but I see it only being useful for
>> sophisticated users. I want to be careful not to confuse newer users.
>>
>>
> Fair enough. This will probably go for the L1 table merging code as well.
> Is there some way things like this can be added as options so if anyone
> like me tries doing this, they have them available?

I think we can just turn it on by default. I'd just suppress
reporting for IRQS variable, report the MVA, and make sure
haretconsole is updated with the new format. (The IRQS variable is
heavily used - even by inexperienced users - while the TRACES variable
is already pretty esoteric.)

I'll see if I can get haretconsole updated, and then I'll check in the
patch.

Thanks,
-Kevin
Received on Thu May 01 2008 - 10:55:16 EDT