From: Balbir S. <ba...@in...> - 2006-05-09 03:57:16
|
On Mon, May 08, 2006 at 02:19:52PM -0700, Andrew Morton wrote: > Balbir Singh <ba...@in...> wrote: > > > > @@ -550,6 +550,12 @@ struct task_delay_info { > > * Atomicity of updates to XXX_delay, XXX_count protected by > > * single lock above (split into XXX_lock if contention is an issue). > > */ > > + > > + struct timespec blkio_start, blkio_end; /* Shared by blkio, swapin */ > > + u64 blkio_delay; /* wait for sync block io completion */ > > + u64 swapin_delay; /* wait for swapin block io completion */ > > + u32 blkio_count; > > + u32 swapin_count; > > These fields are a bit mystifying. > > In what units are blkio_delay and swapin_delay? > > What is the meaning behind blkio_count and swapin_count? > > Better comments needed, please. Will add more detailed comments and send them as updates. Thanks, Balbir Singh, Linux Technology Center, IBM Software Labs |