fastdfs的api是多线程和多进程安全吗？

happy_fastdfs

目前使用c的api，在处理用户并发访问时，想跟作者确认一下api是否多线程和多进程安全，多谢！

g_tracker_servers是个全局变量，客户端和Storage都用到，客户端应该非线程安全吧？像函数：
tracker_get_connection
tracker_get_connection_ex
所有访问g_tracker_servers的地方都应该加锁。

[ 本帖最后由 happy_fastdfs 于 2009-10-10 16:05 编辑 ]

happy_fish100

可以做到多线程安全。
加锁是一种做法，但不建议这么用，因为会导致堵塞和等待，效率很差。
建议每个线程使用自己的tracker server，不要使用全局的。
比如：
TrackerServerInfo tracker_server;
if (tracker_get_connection_ex(&tracker_server) != 0)
{
     //连接tracker server失败！
}

happy_fastdfs

tracker_get_connection_ex函数
        pCurrentServer = g_tracker_servers + g_tracker_server_index;//线程A 根据g_tracker_server_index memcpy出最后这个tracker结构体时,线程B刚好将g_tracker_server_index++到越界位置，memcpy时会越界访问。
        memcpy(pTrackerServer, pCurrentServer, sizeof(TrackerServerInfo));
        pTrackerServer->sock = -1;
        if (tracker_connect_server(pTrackerServer) == 0)
        {
                g_tracker_server_index++;//线程B刚好将g_tracker_server_index++到越界位置
                if (g_tracker_server_index >= g_tracker_server_count)
                {
                        g_tracker_server_index = 0;
                }
                return 0;
        }
所以后面版本看看能不能实现几个不使用全局变量g_tracker_servers和 g_tracker_server_index的tracker连接管理函数。可以把配置文件中的tracker列表读到全局变量中，因为运行时只读，然后每个线程get connection时从全局配置中读出各个tracker的ip和port，构建线程自己的tracker_servers。

搜索代码看到应该是Storage也用到全局变量g_tracker_servers，幸好Storage需要重启才会改变g_tracker_server_count，看来server在运行时没发现线程安全问题。

我的理解对不？

happy_fish100

的确存储LZ所说的问题，V1.22版中已经修复此问题。
LZ对FastDFS研究得比较深入啊，再次赞一个！

happy_fish100

修正后的代码片段如下：

        int server_index;

        server_index = pTrackerGroup->server_index;
        if (server_index >= pTrackerGroup->server_count)
        {
                server_index = 0;
        }
        pCurrentServer = pTrackerGroup->servers + server_index;

以前tracker server守护进程的计数器也不是线程安全的，采用了类似的做法解决了多线程访问导致读取的计数器越界的问题。

happy_fastdfs

用个局部变量 int server_index;避免了加锁，你的技巧也令人佩服啊！
再用个参数pTrackerServer memcpy出来，而不是直接使用pTrackerGroup内的指针，避免了不安全的隐患，学习了！

发现这个问题，归功于同事chenwq看api后的质疑，因为他是多线程环境使用api。
我今天对比测试一下，如还有问题或还有其他地方线程安全问题，今天再回复这个贴。

happy_fastdfs

从svn取代码测试OK，非常感谢，期待新版本的正式发布。

jackywdx

实在佩服happyfish的编程水平和有耐心地回答用户提出的问题。这两天才刚开始看fastdfs，要好好地学习一下。
需要向happy_fastdfs 学习

sopato

期待fastdfs逐渐完善和强大起来，成为一款高质量的应用系统。很感谢作者和发现问题的各位。