Комментарии 49
буст вообще гиганское непонятно что
но такое удобное
но такое удобное
+7
Суровый язык этот с++ Можно не то что в ногу себе выстрелить, а запросто повеситься в абсолютно пустой комнате
+47
Вы сравниваете внешнюю библиотеку (буст) со встроенными средствами языка в конкретном компиляторе, это несколько некорректно. Бусту нужно быть совместимым с кучей компиляторов и платформ, наверняка у него есть причины на такую вот чушь. Вы еще сравните реализацию лямбд в бусте и С++11. Конечно, конкретная реализация std::shared_ptr в новейшем стандарте будет работать быстрее общей невесть-когда написанной реализацией в библиотеке.
+17
Хотя общий вывод, конечно, верный: "Используйте std::shared_ptr, выделяете память через make/allocate shared и будьте счастливы"
+5
Вы в целом правы, но не в данном случае. В бусте есть все необходимое для правильной реализации. В частности, там есть move constructor для shared_ptr, но он есть местами, например его нету для aliasing, а должен быть, обычный ведь есть. Кроме того, в бусте есть библиотека MOVE, которая позволяет реализовать rvalue refernce для старых компиляторов. Да, она только недавно появилась, но сам механизм ее работы я видел в бусте еще несколько лет назад. В общем, переносимость не оправдание. Именно в этом случае переносимости действительно можно добиться.
Да чер с ними, атомарными операциями. Почему boost::make_shared 2 раза выделяет память? Для нормальной реализации даже частичной специализации не надо.
Да чер с ними, атомарными операциями. Почему boost::make_shared 2 раза выделяет память? Для нормальной реализации даже частичной специализации не надо.
0
Хотелось бы узнать версию boost и увидеть ссылку на страницу, указывающую, что эта реализация полностью поддерживает последний стандарт языка. :)
0
Проверял на последнем релизе 1.48. Посмотрите в исходники и увидите rvalue references в shared_ptr (я смотрел логи изменений в svn, они там еще в 2009 г. появились).
Немножко спасает #define BOOST_HAS_RVALUE_REFS, но все равно не до конца (атомарные операции остаются).
Немножко спасает #define BOOST_HAS_RVALUE_REFS, но все равно не до конца (атомарные операции остаются).
0
По идее BOOST_HAS_RVALUE_REFS должен решить все проблемы. Можно дизассемблированный фрагмент с BOOST_HAS_RVALUE_REFS?
BOOST_HAS_RVALUE_REFS должна определятся автоматически для VC++ компилятора версии 1600 (VC++ 2010) и больше (смотри boost\config\compiler\visualc.hpp, дефайн BOOST_NO_RVALUE_REFERENCES). Поэтому строка #define BOOST_HAS_RVALUE_REFS по идее не должна влиять вообще.
BOOST_HAS_RVALUE_REFS должна определятся автоматически для VC++ компилятора версии 1600 (VC++ 2010) и больше (смотри boost\config\compiler\visualc.hpp, дефайн BOOST_NO_RVALUE_REFERENCES). Поэтому строка #define BOOST_HAS_RVALUE_REFS по идее не должна влиять вообще.
0
Явно его включил, те же грабли.
Вот код:
pastebin.com/H0uuHV1p
дизассемблированная f() (выполняет одну атомарная операцию)
pastebin.com/2vKsJ0Km
дизассемблированная boost::make_shared (выполняет две атомарных инструкции)
pastebin.com/k6GPa1VP
дизассемблированная boost::static_pointer_cast (выполняет одну атомарную операцию)
pastebin.com/14MZsWtt
на самом деле все очевидно просто:
f():
static_pointer_cast не умеет принимать rvalue, поэтому происходит копирование временного объекта, созданного make_shared. Когда он удаляется выполняется атомарный cmp exchange
make_shared:
2 атомарных вызова в
return boost::shared_ptr( pt, pt2 ); // Это алиас конструктор копирования
Один — создание нового указателя из pt, второй — удаление pt.
(shared_ptr в бусте не имеет alias move конструктора)
static_pointer_cast:
просто не умеет принимать rvalue, поэтому создает новый откастованый указатель методом копирования. во время создания вызывается атомарный инкремент
Вот код:
pastebin.com/H0uuHV1p
дизассемблированная f() (выполняет одну атомарная операцию)
pastebin.com/2vKsJ0Km
дизассемблированная boost::make_shared (выполняет две атомарных инструкции)
pastebin.com/k6GPa1VP
дизассемблированная boost::static_pointer_cast (выполняет одну атомарную операцию)
pastebin.com/14MZsWtt
на самом деле все очевидно просто:
f():
static_pointer_cast не умеет принимать rvalue, поэтому происходит копирование временного объекта, созданного make_shared. Когда он удаляется выполняется атомарный cmp exchange
make_shared:
2 атомарных вызова в
return boost::shared_ptr( pt, pt2 ); // Это алиас конструктор копирования
Один — создание нового указателя из pt, второй — удаление pt.
(shared_ptr в бусте не имеет alias move конструктора)
static_pointer_cast:
просто не умеет принимать rvalue, поэтому создает новый откастованый указатель методом копирования. во время создания вызывается атомарный инкремент
+1
Спасибо, понятно. По идее у вас тогда BOOST_HAS_RVALUE_REFS вообще не должен влиять ни на что, потому что он уже задефайнин самим бустом.
Погляжу потом static_pointer_cast и shared_ptr у буста и Microsoft, может патч отправлю бусту. Главное ничего не сломать с какими-то другими компиляторами :-)
Погляжу потом static_pointer_cast и shared_ptr у буста и Microsoft, может патч отправлю бусту. Главное ничего не сломать с какими-то другими компиляторами :-)
0
По идее конечно им нужно баг запостить www.boost.org/support/bugs.html
0
А можно исходники теста? Хотел бы сам посмотреть под дизассембером.
0
intrusive_ptr не пользовали?
0
мне было проще std::shared_ptr, впрочем, думается мне там та же проблема (intrusive_ptr_add_ref/intrusive_ptr_release) вызывается когда надо и когда не надо.
0
ну в том-то и дело, что вы можете эти функции сами написать как надо. при этом в заведомо однопоточной среде не надо будет заморачиваться с синхронизацией. Но в целом согласен, да, незачем при move дергать счетчик. Ну что ж, стандарт новый, реализация еще не обкатанная)
0
В однопоточной среде — да. А вот отследить, что intrusive_ptr_add_ref/intrusive_ptr_release вызвалось так «по приколу» потому что буст не имеет все необходимые мув конструкторы нельзя. Впрочем, возможно, что intrusive_ptr этим не страдает, но он неудобный в любом случае. Сейчас std::shared_ptr работает так же быстро при условии, что у вас нету фреймворка, который считает ссылки за вас (как например для COM объектов, но там выигрыш будет слишком незначительным по сравнению с временем выполнения COM)
0
Очень интересное выступление, где автор реализации стандартной библиотеки в Microsoft, Stephan T. Lavavej, рассказывает, как ему удалось оптимизировать std::shared_ptr.
+9
Вы уверены, что в конфиге для вашего компилятора все нужные define'ы определены? Может, он думает, что какие фичи C++11 им не поддерживаются?
PS Можно на код взглянуть? Так, ради интереса… Лучше после препроцессора :)
PS Можно на код взглянуть? Так, ради интереса… Лучше после препроцессора :)
+2
на код чего? если измерения, то нет я его удалил утром случайно :(
ну там все тривиально — 10М раз выполняем
boost::static_pointer_cast( boost::allocate_shared( TAllocator() ) );
и иже с ним
аллокатор — Loki Small Object (без него тоже пробовал)
таймер — QueryPerformanceCounter + affinity
#define BOOST_HAS_RVALUE_REFS — определял, не помагает
ну там все тривиально — 10М раз выполняем
boost::static_pointer_cast( boost::allocate_shared( TAllocator() ) );
и иже с ним
аллокатор — Loki Small Object (без него тоже пробовал)
таймер — QueryPerformanceCounter + affinity
#define BOOST_HAS_RVALUE_REFS — определял, не помагает
0
return std::move( pRes )
Возвращается ж всегда и так rvalue — зачем здесь std::move?
+5
Я бы посоветовал поговорить (написать письмо) STL из Microsoft (http://channel9.msdn.com/Tags/stephan-t-lavavej), он эксперт по Boost и его интерпретации в VC++. Он если надо и баг в VC++ откроет.
+2
А не проще для сильно нагруженных участков вообще отказаться от умных указателей? Я обычно так и стараюсь делать, а в умные указатели оборачивать только долгоживущие и гулящие где попало обьекты.
0
Если у вас общий объект на несколько потоков — то нет. В общем случае вы не знаете, какой поток закончит работать последним. А именно он должен будет освободить память. Придется либо городить какую-то синхронизацию (и получить тот же shared_ptr, вид сбоку), либо так переворачивать архитектуру приложения, что за создание/удаление объектов отвечает один конкретный поток — но далеко не всегда это можно сделать без существенной потери производительности.
+2
Не проще, потому что проще один раз оптимизировать (найти правильный) указатель за пол дня, чем потом каждый раз думать про освобождение памяти. Эм, поясню. Сильно нагруженый «участок» размазан почти по всему коду, т.к. это механизм передачи данных через мультиметоды на самом деле. А указатель — это указатель на то нечто, которое эти самые данные хранит.
auto pData = CreateData( Component_ID( 4 ), Component_Value( 3. ) );
FireEvent( GetEvent(), std::move( pData ) );
как-то так
auto pData = CreateData( Component_ID( 4 ), Component_Value( 3. ) );
FireEvent( GetEvent(), std::move( pData ) );
как-то так
+1
Библиотеки с открытым кодом, такие как boost или QT, которые считаются неотъемлемой частью разработки и которые используют огромное количество людей, не лишены ошибок. Но все, кто используют библиотеки, являются одновременно огромной армией тестеров, находящих коллизии и баги. Поэтому используя открытые и широко распространенные библиотеки, мы получаем все таки достаточно надежные инструменты для программирования.
+3
Спасибо за инфу, будем знать. Но ты еще напиши в boost developer mailing list, глядишь допилят.
0
Он замедляет команду приблизительно на 2 порядка, т.к. требует сброса кеш линии, а значит использовать его нужно как можно реже.
Можно с этого места поподробнее? Когда-то я делал тесты, и 2 порядка (напомню, что 2 порядка — это в 100 раз) я там не обнаружил. Т.е. вопрос: откуда там 2 порядка и почему происходит сброс кеш линии?
Можно с этого места поподробнее? Когда-то я делал тесты, и 2 порядка (напомню, что 2 порядка — это в 100 раз) я там не обнаружил. Т.е. вопрос: откуда там 2 порядка и почему происходит сброс кеш линии?
+2
окей, соврал, не на 2 порядка, на 1. На 2 — если учесть что там около 10 лишних локов.
Еще точнее
__asm lock xadd dword ptr[n], eax;
медленнее
__asm xadd dword ptr[n], eax;
в 6-15 раз на моем процессоре в зависимости от того, сколько потоков этим занимаются.
(когда несколько потоков на разных процессорах занимаются атомарными операциями над одним и тем же аргументом им приходится синхронизировать между собой кеш, тогда замедление в 15 раз, когда поток один — тогда в 6 раз)
Еще точнее
__asm lock xadd dword ptr[n], eax;
медленнее
__asm xadd dword ptr[n], eax;
в 6-15 раз на моем процессоре в зависимости от того, сколько потоков этим занимаются.
(когда несколько потоков на разных процессорах занимаются атомарными операциями над одним и тем же аргументом им приходится синхронизировать между собой кеш, тогда замедление в 15 раз, когда поток один — тогда в 6 раз)
0
Согласно пункту 8.1.4 «Effects of a LOCK Operation on Internal Processor Caches» интеловской книжки «Intel® 64 and IA-32 Architectures Developer's Manual: Vol. 3A»
сброса кэша не происходит. Наоборот, начиная с P6, сигнал LOCK# может быть не активирован, а вместо него может быть использован механизм аппаратной поддержки когерентности кэша. Сама операция с префиксом LOCK при этом будет выполнена в кэше.
сброса кэша не происходит. Наоборот, начиная с P6, сигнал LOCK# может быть не активирован, а вместо него может быть использован механизм аппаратной поддержки когерентности кэша. Сама операция с префиксом LOCK при этом будет выполнена в кэше.
+1
В общем:
N = 100 M (операций)
K = 10 (потоков)
На одном ядре ( SetProcessAffinityMask( GetCurrentProcess(), 1 ); раскомментировано )
const of one operation: 1.85526 ns.
const of one operation: 6.47227 ns. (3.5x)
На всех ядрах ( SetProcessAffinityMask( GetCurrentProcess(), 1 ); закомментировано )
const of one operation: 0.721436 ns.
const of one operation: 10.7002 ns. (14.9х)
pastebin.com/Jkpfz4t2
Вы можете сказать, что это не много, но я вам скажу, что 6 ns при общем времени обработки события в 70 ns это дофига как много.
N = 100 M (операций)
K = 10 (потоков)
На одном ядре ( SetProcessAffinityMask( GetCurrentProcess(), 1 ); раскомментировано )
const of one operation: 1.85526 ns.
const of one operation: 6.47227 ns. (3.5x)
На всех ядрах ( SetProcessAffinityMask( GetCurrentProcess(), 1 ); закомментировано )
const of one operation: 0.721436 ns.
const of one operation: 10.7002 ns. (14.9х)
pastebin.com/Jkpfz4t2
Вы можете сказать, что это не много, но я вам скажу, что 6 ns при общем времени обработки события в 70 ns это дофига как много.
0
А без static_pointer_cast — какие цифры?
0
Вот тесты именно с поинтерами:
x86:
x64:
Код:
pastebin.com/jAKQSZKz
x86:
std, make_shared: const of one iteration: 79.6456 ns.
boost, make_shared: const of one iteration: 154.441 ns.
std, make_shared + cast: const of one iteration: 96.3465 ns.
std, make_shared + fast cast: const of one iteration: 88.3443 ns.
boost, make_shared + cast: const of one iteration: 170.912 ns.
std, allocate_shared: const of one iteration: 41.9773 ns.
boost, allocate_shared: const of one iteration: 106.236 ns.
std, allocate_shared + cast: const of one iteration: 51.8958 ns.
std, allocate_shared + fast cast: const of one iteration: 47.2173 ns.
boost, allocate_shared + cast: const of one iteration: 122.333 ns.
std, release: const of one iteration: 139.189 ns.
boost, release: const of one iteration: 136.266 ns.
std, release + cast: const of one iteration: 152.798 ns.
std, release + fast cast: const of one iteration: 147.057 ns.
boost, release + cast: const of one iteration: 154.359 ns.
x64:
std, make_shared: const of one iteration: 59.2896 ns.
boost, make_shared: const of one iteration: 111.826 ns.
std, make_shared + cast: const of one iteration: 75.6804 ns.
std, make_shared + fast cast: const of one iteration: 63.9302 ns.
boost, make_shared + cast: const of one iteration: 127.658 ns.
std, allocate_shared: const of one iteration: 52.4064 ns.
boost, allocate_shared: const of one iteration: 99.92 ns.
std, allocate_shared + cast: const of one iteration: 66.9683 ns.
std, allocate_shared + fast cast: const of one iteration: 62.9726 ns.
boost, allocate_shared + cast: const of one iteration: 114.167 ns.
std, release: const of one iteration: 101.677 ns.
boost, release: const of one iteration: 100.807 ns.
std, release + cast: const of one iteration: 117.108 ns.
std, release + fast cast: const of one iteration: 106.495 ns.
boost, release + cast: const of one iteration: 114.833 ns.
Код:
pastebin.com/jAKQSZKz
0
Я хочу сказать что этого достаточно чтобы уже пофиксить статью.
0
Сдаётся мне, что проблема тут не в финальном std::move (его наличии или отсутствии), а в том, что boost::static_pointer_cast (впрочем, как и std::static_pointer_cast) получает свой аргумент по обычной (а не r-value) ссылке. Отсюда и щёлканье счётчиком. Т. е. move-семантика тут (по логике вещей) и не должна работать. Есть предположение (которая подтвердила проверка на компиляторе), что код без static_pointer_cast'а бдут работать так, как ожидает автор — никаких lock xadd не выполняется. Проверялось на таком коде:
class BaseClass
{
public:
virtual ~BaseClass() {;}
void Foo()
{
std::cout << «Foo called» << std::endl;
}
};
class DerivedClass: public BaseClass
{
public:
};
boost::shared_ptr MakeSharedPtr()
{
boost::shared_ptr ptr = boost::make_shared();
return ptr;
}
int main()
{
auto ptr = MakeSharedPtr();
ptr->Foo();
}
При добавлении в этот код boost::static_pointer_cast'а всё начинает работать именно так, как описано топикстартером.
class BaseClass
{
public:
virtual ~BaseClass() {;}
void Foo()
{
std::cout << «Foo called» << std::endl;
}
};
class DerivedClass: public BaseClass
{
public:
};
boost::shared_ptr MakeSharedPtr()
{
boost::shared_ptr ptr = boost::make_shared();
return ptr;
}
int main()
{
auto ptr = MakeSharedPtr();
ptr->Foo();
}
При добавлении в этот код boost::static_pointer_cast'а всё начинает работать именно так, как описано топикстартером.
+2
Угловые скобки были съедены. MakeSharedPtr должен выглядеть так:
boost::shared_ptr<BaseClass> MakeSharedPtr()
{
boost::shared_ptr<BaseClass> ptr = boost::make_shared<DerivedClass>();
return ptr;
}
0
внутри boost::make_shared уже есть 2е атомарных инструкции,
см. habrahabr.ru/blogs/cpp/138658/#comment_4631952
static_pointer_cast тоже мог бы принять rvalue, т.к. временный неименованый объект это всегда rvalue, но у static_pointer_cast нету соответствующей реализации (в std ее удалось добавить «малой кровью»)
см. habrahabr.ru/blogs/cpp/138658/#comment_4631952
static_pointer_cast тоже мог бы принять rvalue, т.к. временный неименованый объект это всегда rvalue, но у static_pointer_cast нету соответствующей реализации (в std ее удалось добавить «малой кровью»)
0
std::static_pointer_cast с r-value в качестве параметра — не по стандарту. По крайней мере, документ за номером N3242 такой специализации не специфицирует. Может быть просто забыли, может быть были какие-то глубокие причины.
+1
Вот это самый логичный аргумент по поводу того, почему этой реализации там нет…
0
есть политики, это хорошо: есть, например shared_ptr без поддержки многопоточности
все остальное еще хуже буста:
— нету make/allocate shared, значит на каждый указатель минимум 2 выделения памяти (если не использовать internal ref counting)
— нету кастов (static_pointer_cast и иже с ним)
— полностью отсутствует поддержка rvalue references, в бусте ее немножко есть
все остальное еще хуже буста:
— нету make/allocate shared, значит на каждый указатель минимум 2 выделения памяти (если не использовать internal ref counting)
— нету кастов (static_pointer_cast и иже с ним)
— полностью отсутствует поддержка rvalue references, в бусте ее немножко есть
0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Производительность shared_ptr и C++11: почему я не верю библиотекам