On Mon, Feb 20, 2012 at 07:38:32PM -0500, Christoph Hellwig wrote:
> Add an in-memory only flag to say we logged timestamps only, and use it to
> check if fdatasync can optimize away the log force.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
Looks good. Couple of minor things below.
Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
> +/*
> + * The timestamps in the core are dirty, but not nessecarily anything
necessarily
> + * else.
> + *
> + * This is an incore only value store in ilf_fields & co, which must
> + * never make it to disk, unlike the other fields above.
That comment is a bit incoherent. Perhaps:
* this is an incore-only flag stored in ili_fields which, unlike
* the above flags, must never make it to disk in the ilf_fields.
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
