[Cbe-oss-dev] [PATCH 7:9] spufs: Provide ctxt debug file

Luke Browning lukebr at linux.vnet.ibm.com
Wed Jun 4 23:43:06 EST 2008


On Wed, 2008-06-04 at 15:18 +0800, Jeremy Kerr wrote:
> Hi Luke,
> 
> If this is for debugging, should we make it conditional on DEBUG? That 
> should prevent casual usage.
> 
> > @@ -2613,6 +2666,7 @@ struct tree_descr spufs_dir_contents[] =
> >  	{ "tid", &spufs_tid_fops, 0444, },
> >  	{ "stat", &spufs_stat_fops, 0444, },
> >  	{ "switch_log", &spufs_switch_log_fops, 0444 },
> > +	{ "x_ctxt", &spufs_x_ctxt_fops, 0444, },
> 
> .. to stick with convention (hidden file, and the fact that we use 'ctx' 
> everywhere), let's call it ".ctx"

This was Christoph's suggestion, x_*, as it was used somewhere else for
the same purpose.

If we put it under DEBUG, we will lose a lot.  Currently, we don't have
a way to analyze kernel state when applications hang in the field.  The
mechanism needs to be available when the problem occurs.  Many problems
are not easily recreated and many customers are not willing to reboot or
they are not willing to reboot with a different (DEBUG) kernel.

Luke





More information about the cbe-oss-dev mailing list