[PATCH] C/R: simplify checkpoint debugging

Dan Smith danms at us.ibm.com
Mon Jun 22 14:07:31 PDT 2009


SH> Quoting Nathan Lynch (ntl at pobox.com):
>> To enable printing of checkpoint/restart messages, one must enable
>> CHECKPOINT_DEBUG and boot with ckpt_debug= ... something I haven't
>> been able to figure out.  Further complicating matters is that if
>> DYNAMIC_DEBUG is enabled, you still won't get any debugging output
>> unless you manually enable debug output for all the functions you're
>> interested in via <debugfs>/dynamic_debug/control.  At this stage of
>> development most users will want all C/R messages either on or off and
>> don't need a fancy filtering mechanism.
>> 
>> Change ckpt_debug to print messages if CHECKPOINT_DEBUG=y.  Otherwise
>> it will emit nothing.  Remove ckpt_debug= setup code.  Reword
>> CHECKPOINT_DEBUG config help text appropriately.
>> 
>> Signed-off-by: Nathan Lynch <ntl at pobox.com>

SH> Yes please!

Enthusiastically seconded.  Or thirded, or whatever it would be.

-- 
Dan Smith
IBM Linux Technology Center
email: danms at us.ibm.com


More information about the Containers mailing list