* libc/stdio/fvwrite.c: Allow writing in larger chunks for fully

buffered and unbuffered files, to improve write performance.

newlib/ChangeLog

@@ -1,3 +1,8 @@
+2013-11-04  Terraneo Federico  <fede.tft@hotmail.it>
+
+	* libc/stdio/fvwrite.c: Allow writing in larger chunks for fully
+	buffered and unbuffered files, to improve write performance.
+
 2013-10-31  Terraneo Federico  <fede.tft@hotmail.it>
 
 	* libc/stdio/fvwrite.h: change type of uio_resid from int to size_t to

newlib/libc/stdio/fvwrite.c

@@ -21,6 +21,7 @@
 #include <string.h>
 #include <stdlib.h>
 #include <errno.h>
+#include <limits.h>
 #include "local.h"
 #include "fvwrite.h"
 
@@ -89,12 +90,14 @@ _DEFUN(__sfvwrite_r, (ptr, fp, uio),
   if (fp->_flags & __SNBF)
     {
       /*
-       * Unbuffered: write up to BUFSIZ bytes at a time.
+       * Unbuffered: Split buffer in the largest multiple of BUFSIZ < INT_MAX
+       * as some legacy code may expect int instead of size_t.
        */
       do
 	{
 	  GETIOV (;);
-	  w = fp->_write (ptr, fp->_cookie, p, MIN (len, BUFSIZ));
+	  w = fp->_write (ptr, fp->_cookie, p,
+			  MIN (len, INT_MAX - INT_MAX % BUFSIZ));
 	  if (w <= 0)
 	    goto err;
 	  p += w;
@@ -177,30 +180,24 @@ _DEFUN(__sfvwrite_r, (ptr, fp, uio),
 	      fp->_p += w;
 	      w = len;		/* but pretend copied all */
 	    }
-	  else if (fp->_p > fp->_bf._base && len > w)
+	  else if (fp->_p > fp->_bf._base || len < fp->_bf._size)
 	    {
-	      /* fill and flush */
+	      /* pass through the buffer */
+	      w = MIN (len, w);
 	      COPY (w);
-	      /* fp->_w -= w; *//* unneeded */
+	      fp->_w -= w;
 	      fp->_p += w;
-	      if (_fflush_r (ptr, fp))
+	      if (fp->_w == 0 && _fflush_r (ptr, fp))
 		goto err;
 	    }
-	  else if (len >= (w = fp->_bf._size))
+	  else
 	    {
 	      /* write directly */
+	      w = ((int)MIN (len, INT_MAX)) / fp->_bf._size * fp->_bf._size;
 	      w = fp->_write (ptr, fp->_cookie, p, w);
 	      if (w <= 0)
 		goto err;
 	    }
-	  else
-	    {
-	      /* fill and done */
-	      w = len;
-	      COPY (w);
-	      fp->_w -= w;
-	      fp->_p += w;
-	    }
 	  p += w;
 	  len -= w;
 	}