This is the mail archive of the
cygwin-developers
mailing list for the Cygwin project.
Re: Performance optimization in av::fixup - use buffered IO, not mapped file
On Dec 13 19:53, Corinna Vinschen wrote:
> On Dec 13 10:32, Eric Blake wrote:
> > On 12/13/2012 07:30 AM, Corinna Vinschen wrote:
> > >
> > > Below is my cut on the issue. It introduces a "sparse" mount option and
> > > sparsifies a file only if this mount option is set. I also slightly
> > > improved the code in fhandler_base::write and ftruncate so that it will
> > > not try to set the sparse flag on already sparse files.
> > >
> > > The code compiles, so it's basically correct. It's just not tested. ;)
> >
> > Also untested (so far) on my end, but I can agree to this - anyone that
> > _wants_ sparse files (such as for virtual disk image) will have to
> > enable that mount point option for the directory of those files in
> > question, but no one else is forced to use them. As sparse files are
> > _supposed_ to be an optimization, not being sparse is not a loss in
> > functionality, just in potential optimizations; but if Windows is not
> > taking advantage of optimizations even when a file IS sparse, then that
> > argues that we aren't losing much.
> >
> > It should not be too hard for me to rig the coreutils testsuite to
> > ensure that its tests of sparse operations are done on a mount
> > explicitly set up to allow sparse files.
>
> Yup, just add a temporary user mount point.
I applied my patch with a single fix. The logic for adding the "sparse"\
option string to mnt_opts was upside down.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat
- References:
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file
- Re: Performance optimization in av::fixup - use buffered IO, not mapped file