Re: [Cluster-devel] [GFS2 PATCH] Only do lo_incore_commit once

2008-01-29 Thread Steven Whitehouse
Hi, Now in the -nmw tree. Thanks, Steve. On Mon, 2008-01-28 at 11:20 -0600, Bob Peterson wrote: Hi, This patch is performance related. When we're doing a log flush, I noticed we were calling buf_lo_incore_commit twice: once for data bufs and once for metadata bufs. Since this is the

[Cluster-devel] [GFS2 PATCH] Only do lo_incore_commit once

2008-01-28 Thread Bob Peterson
Hi, This patch is performance related. When we're doing a log flush, I noticed we were calling buf_lo_incore_commit twice: once for data bufs and once for metadata bufs. Since this is the same function and does the same thing in both cases, there should be no reason to call it twice. Since we