Matthew Dillon wrote:
> :I've just upgraded to -CURRENT as of yesterday, and I'm noticing a
> :number of occasions where all activity ceases for a second or two at a
> :time; it seems to be related to IDE disk activity with the new ATA
> :driver, but I don't have much evidence.  I'm running a SiS 5591
> :chipset.  Has anybody else seen something like this?
> :
> :Greg
> 
>     It's possible that the blockages you are seeing are due to the ATA
>     driver, but it's also possible that they are due to a bug in the
>     buffer cache flushing code which the following patch fixes.  So try
>     the patch and see if that fixes your problem.  If it doesn't then
>     we can at least rule it out as being the cause of the problem you
>     are seeing.
[..]

Just one comment..  You've replaced vfs_bio's call to speedup_syncer() with
a bufdaemon speedup..    Granted I don't understand the details here, but
I'm curious why? (or why not use both?)  The reason I ask is that I wonder
whether removing of the speedup of the vnode cleanup rate is a good idea or
not..  or if the bufdaemon speedup does this as a side effect via the hooks
softupdates has got in the bio system.  This is not a criticism, just a
request for enlightenment. :-)

> @@ -1571,9 +1573,8 @@
>                       flags = VFS_BIO_NEED_ANY;
>               }
>  
> -             /* XXX */
> +             bd_speedup();   /* heeeelp */
>  
> -             (void) speedup_syncer();
>               needsbuffer |= flags;
>               while (needsbuffer & flags) {
>                       if (tsleep(&needsbuffer, (PRIBIO + 4) | slpflag,

Cheers,
-Peter




To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to