Re: [PATCH] Add support for profiling Loongson 2E

[Fuxin Zhang <fxzhang@ict.ac.cn>]

Dajie Tan 写道:
>> > Yeah,this change is to enhance the robust of oprofile. When using
>> > performace counter manually(writting control register in a module, no
>> > need to use the oprofile),I usually make kernel panic if I do not
>> > initialize the oprofile and enable the overflow interrupt carelessly.
>> > So, this change can avoid this panic. :D
>>
>> This panic is good and should stay. It shows that you've made a mistake.
>>
>> john
>>
>
> This panic is caused by accessing a null pointer.Do you think that
> accessing a null
> pointer is allowed in a robust system ?
I think you don't really understand Ralf and John's responses. The 
problem should not be workarounded in that position.
Your code should ensure that overflow interrupt will not be enabled when 
oprofile is not enabled, or something like this. If everyone choose to 
'fix' problems via patching generic code, it would be a maintainence 
headache. Why other platform s won't have your problem?

It is not to say that the panic is ok, but it is your problem if others 
are not suffering the same panic, and should be fixed in your code.
>
>
>
>

-- 
------------------------------------------------
Fuxin Zhang

Vice Researcher
Microprocessor Research Center, Institute of Computing Technolgy
Chiese Academy of China
Beijing, 100080
Email: fxzhang@ict.ac.cn
http://www.ict.ac.cn

------------------------------------------------