Need review for client-reopen changes

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

 



hi,
http://review.gluster.org/#change,4357    
http://review.gluster.org/#change,4358

are the changes I made to handle re-opens of files in the case where a disk is replaced while a brick is offline. The idea is to attempt re-opens after self-heal completes and the file could be opened. With these changes readv/fxattrop/writev/findelk for fds with remote-fd -1 are attempted using anon-fds and if the fop succeeds then the re-open is attempted for every 1024th success. 1024 is an arbitrary number I used. The re-open of files could fail because of posix lock re-acquisition failure, that is the reason re-opens are attempted periodically (for every 1024 successful fops on that fd).

I think the re-attempt logic could be better.
For instance, we can attempt re-open on the first success on anon-fd instead of waiting till 1024th success and if this re-open fails we could fall-back on 'periodic attempts' i.e. for every 1024 successes on the anon-fd.

Let me know your thoughts.

Pranith



[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