boost 的bug?

smzgl

boost是1.4.3的


boost::asio::detail::task_io_service::run()中调用了typename call_stack<task_io_service>::context ctx(this);

而context 的构造函数实际是把this放到向一个堆栈顶部, 析构函数是将堆栈首个元素移除.


这两个过程都是没有加锁的.


如果这样来调用岂不是有bug了?

1. for (int i = 0; i < 2 * CPUNUM; ++i)
   
2.     boost::thread *t = new boost::thread(boost::bind(&boost::asio::io_service::run, &io_service));

复制代码

之前看错了, 在boost的例子中, 只是单个线程而已.

没本

/trunk/boost/asio/detail/task_io_service.hpp 最近更新是3月31号，从trunk弄份最新的看看吧。

smzgl

回复 2# 没本
对比过了, 没有区别呀.

没本

那就向作者报bug吧，他改了就永远不会有这bug了。

tyc611

只有当数据（变量）需要在多个线程中进行读写时才需要加锁，而从你给的信息中不能得到什么结论

smzgl

回复 4# 没本


     就是会出出现多个线程同时读写的情况,我才有这样子的疑问

beyond_touch

只有当数据（变量）需要在多个线程中进行读写时才需要加锁，而从你给的信息中不能得到什么结论
tyc611 发表于 2010-06-04 22:45

对boost，我也不是很懂。所以如果你有确凿证据，请忽略我说的东西。

我同意他的观点，楼主所讲的多线程（例子中），只是两个线程（一个在等待，一个在执行run）。不存在并发保护问题。

另外，Ｃ＋＋库的设计就是这个样子，只提供最小限度的并发保护。如果你想并发运行，需要自己提供并发保障。

偶只是菜鸟

smzgl

回复 8# beyond_touch


    typename call_stack<task_io_service>::context ctx(this); 这句代码恰好放在上锁之前....

beyond_touch

回复  beyond_touch


    typename call_stack::context ctx(this); 这句代码恰好放在上锁之前....
smzgl 发表于 2010-06-07 12:19

1.   // Mutex to protect access to internal data.
   
2.   boost::asio::detail::mutex mutex_;
   

复制代码

偶只是菜鸟
前面是我错了。那个锁根本不是提供并发保护的。
"task_io_service"是在asynchronous io分类下边。并没有什么提供并发保护的理由。
设计只是基于异步，根本没有把并发考虑在内。

楼主不同意我的想法我也不打算再回复了