[RFC] Fix compilation failure of remote-fileio.c

Daniel Jacobowitz drow@mvista.com
Mon Dec 29 16:01:00 GMT 2003


On Mon, Dec 29, 2003 at 08:58:43AM +0200, Eli Zaretskii wrote:
> > Date: Sun, 28 Dec 2003 18:38:00 -0500
> > From: Daniel Jacobowitz <drow@mvista.com>
> > 
> >        The value st_blocks gives the size of the file in 512-byte
> >        blocks.  (This may be smaller than st_size/512 e.g. when the
> >        file has holes.) The value st_blksize gives the "preferred"
> >        blocksize for efficient file system I/O.  (Writing to a file in
> >        smaller chunks may cause an inefficient read-modify-rewrite.)
> 
> Ah, right, I forgot about that.
> 
> (Is this optimal size for I/O still relevant for modern systems?)

Not as much as it used to be, but I believe so.

> > So you probably want st->st_size / 512 instead.
> 
> So, if I keep the conditional and use the divide-by-512 way for
> systems that don't have st_blocks, is the patch approved?  Or do I
> need to wait for someone else?
> 
> (Thanks for the other suggestions, I will use them.)

Hmm... since it doesn't compile as-is, I think this is OK.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer



More information about the Gdb-patches mailing list