This is the mail archive of the cygwin-developers mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]