Re: [PATCH 3/3 v3] vfs_glusterfs: Samba VFS module for glusterfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2013-05-20 at 03:11 -0400, Anand Avati wrote:
> Implement a Samba VFS plugin for glusterfs based on gluster's gfapi.
> This is a "bottom" vfs plugin (not something to be stacked on top of
> another module), and translates (most) calls into closest actions
> on gfapi.

Thanks for getting back to us.

The comment I have is however still around the realloc stuff.  Is
talloc_realloc() so slow that there is such value in having the code
realloc 1024 or 16 more entries than are required?

It would seem clearer if we just allocated exactly the array size
needed.   That would also allow bounds checkers like valgrind to
correctly know the 'real' allocated size. 

It just smells to me like premature optimisation, and means you have to
do the partial array memset() which always makes me nervous. 

That said, this isn't an objection, just an observation.   

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org





[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux