TODO file

[Philippe E. <ph...@cl...>]

hi,

TODO file:

>> o prepare for rtc:
>>  - move apic init to module init

what you means? Apic init are already marked as __init function.

I've not looked implementation of init/exit section but
lvpt_apic_restore() is marked __exit and is called from
oprof_init() which is marked as __init. Is it safe ?

>>  - remove user-space proc type code

- so op_start can check parameters only after probing the module
 because op_events need to know the proc type. It is not probably
a problem but

- op_help is usable as a standalone programm to see all the events so we
need to keep the user space proc type code

>> o determine where we need ctx_sw_on/off to prevent pre-emption + test

Hummm, I'm probably unable to make that even if I understand what it
means.

>> o move .oprofile to name=value pairs to avoid conflicts on change.

There is a "thing" for that in util/persistent.h and an example test in
oprofile-tests/understanting/persistent-test.cpp, but I don't like the
output format I use. I hate persistency of C++ object...


phil