Re: [PATCH 1/1] fat: improve sync performance by grouping writes revised

[Holden Karau]

Keep in mind these results are with a slow external drive, but yah
fat32 sync performance is still really bad.

This patch is just meant to make fat32 sync performance better, not
necessarily make it usable for everyone [one step at a time and all
that]. FAT32 sync is quite slow (see
http://readlist.com/lists/vger.kernel.org/linux-kernel/22/111761.html
), so if you want fast performance [or have drives with limited write
cycles] async is almost definitely the way to go.

I have a patch [not this one] which adds a module param ("fast") which
syncs the FAT tables after 30 seconds of no write activity, but I
don't think the majority of people are interested in it.

On 10/31/06, OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> wrote:
> "Holden Karau" <holden@xxxxxxxxxxxxx> writes:
>
> > The performance increase is pretty small. Using an old external dirve
> > I had lying around I got:
> > diff -y stock/10k modified/10k
> > 10240+0 records in                                            | 1024+0
> > records in
> > 10240+0 records out                                           | 1024+0
> > records out
> > 5242880 bytes transferred in 18.280922 seconds (286795 bytes/ | 524288
> > bytes transferred in 1.824985 seconds (287283 bytes/se
>
> 1024 records out 1.824985 seconds. Is there decrease case?  I assume
> the result is same. So, we would need different approach.
>
> > diff -y stock/1k modified/1k
> > 1024+0 records in                                               1024+0
> > records in
> > 1024+0 records out                                              1024+0
> > records out
> > 524288 bytes transferred in 1.777250 seconds (295000 bytes/se | 524288
> > bytes transferred in 1.764748 seconds (297089 bytes/se
> --
> OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>


--
Cell: 613-276-1645
-
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/