Re: [Patch 2/8] Sync block I/O and swapin delay collection

[Andrew Morton]

Balbir Singh <balbir@xxxxxxxxxx> 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.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/