Re: CVS commit: src/bin/cp

[Juergen Hannken-Illjes <hannken%eis.cs.tu-bs.de@localhost>]

On Mon, Oct 25, 2010 at 06:30:54PM +0200, Joerg Sonnenberger wrote:
> On Mon, Oct 25, 2010 at 11:17:22AM +0200, Juergen Hannken-Illjes wrote:
> > On Sun, Oct 24, 2010 at 05:21:06AM +0000, David Holland wrote:
> > > On Fri, Oct 22, 2010 at 05:56:06PM +0000, Antti Kantee wrote:
> > >  > Disable mmap path.  With the current vnode locking scheme it has
> > >  > a very annoying property: if the source media is slow (like a slow
> > >  > network), the target file will be locked for the duration of the
> > >  > entire max 8MB write and cause processes attempting to e.g. stat()
> > >  > it to "tstile" (for several minutes in the worst case).  Revisit
> > >  > this if/when vnode locking gets a little smarter.
> > > 
> > > Wouldn't it be better to just ratchet back the block size to something
> > > like 64K that happens faster?
> > 
> > Or first fault the mapped region instead of madvise() like:
> 
> Do we implement MADV_WILLNEED?

According to the man page "This WILL NOT fault pages in from backing store".

-- 
Juergen Hannken-Illjes - hannken%eis.cs.tu-bs.de@localhost - TU Braunschweig 
(Germany)