[11305] more safe code in vmaps and Field class - Ru-MaNGOS

**newsbot** · 02.04.2011, 17:40

автор: Ambal

m src/game/vmap/BIH.cpp

m src/game/vmap/BIH.h

m src/game/vmap/WorldModel.cpp

m src/shared/Database/Field.h

m src/shared/revision_nr.h

[11305] more safe code in vmaps and Field class



Signed-off-by: Ambal

Дальше...

**Ambal** · 02.04.2011, 18:00

Кто-то может собрать мангос с '-fast-math' и глянуть не стали ли vmaps лучше себя вести?

rsa · 03.04.2011, 09:54

а конкретнее, какая именно опция нужна? в gcc -ffast-math не работает если используется оптимизация (любой уровень).
-ffast-math
Sets -fno-math-errno, -funsafe-math-optimizations, -ffinite-math-only,
-fno-rounding-math, -fno-signaling-nans and -fcx-limited-range.

**Ambal** · 04.04.2011, 21:29

Именно этот флаг и нужен: -ffast-math. Он по-умолчанию не активируется если задействованы флаги оптимизации - включать нужно принудительно.

**virusav** · 04.04.2011, 22:43

Вопрос по VMAP: создал перса, примерно на уровне 10 портанулся в Штормград, пошел к тренеру оружейных навыков и по другим столицам и тренерам - все нормально, перс входит.
На 20-22 уровне уже не могу войти в те здания, куда входил на 10-м.

Это интеллектуальный учет роста перса со временем?

**Ambal** · 04.04.2011, 23:05

virusav, встречный вопрос на засыпку: собирали с -ffast-math? Когда персу подняли лвл до 20-22 логаут делали?

**virusav** · 05.04.2011, 10:02

Компилю в VS2008, никаких опцию не задаю, еще не смотрел, где это можно задать.
С 10 по 20-22 уровни была куча логаутов, несколько ребутов сервера.

**Ambal** · 05.04.2011, 13:11

Цитата:

Сообщение от virusav

Компилю в VS2008, никаких опцию не задаю.

Неведомая хня детектед (с) Если вы не трогаете опции компилятора, то сей феномен я лично объяснить на данном этапе немогу...

Vinolentus · 05.04.2011, 15:59

Цитата:

Вопрос по VMAP: создал перса, примерно на уровне 10 портанулся в Штормград, пошел к тренеру оружейных навыков и по другим столицам и тренерам - все нормально, перс входит.
На 20-22 уровне уже не могу войти в те здания, куда входил на 10-м.

Это интеллектуальный учет роста перса со временем?

http://getmangos.com/community/topic/15883/

**virusav** · 05.04.2011, 16:29

Про это думал, но факт остается фактом.

rsa · 05.04.2011, 20:24

Цитата:

Сообщение от Ambal

Именно этот флаг и нужен: -ffast-math. Он по-умолчанию не активируется если задействованы флаги оптимизации - включать нужно принудительно.

возможно я совсем плохо перевожу, но по моему ман говорит что этот флаг не даст никакого эффекта после выбора любого уровня оптимизации...

включил, скомпилил (с оптимизацией O2, без нее тормозит), поставил. результата никакого не вижу ни я ни игроки. что хоть должно измениться-то?

**Ambal** · 05.04.2011, 22:24

Цитата:

Сообщение от rsa

результата никакого не вижу ни я ни игроки. что хоть должно измениться-то?

Должна измениться загрузка ЦП в меньшую сторону и вполне вероятно появиться траблы с вмапс, collision detection и всяческими спеллами, меняющими характеристики игрока на опр %

Если загрузка по ЦП станет меньше, меня интересует разница. В случае если овчинка будет стоить выделки можно взяться за патч, приводящий в порядок наши вычисления и сравнения чисел с плавающей запятой.

P.S. В мануале по GCC сей функционал по-умолчанию не активируется при выборе любого вида оптимизации аля -Ox по причине несовместимости вычислений над float/double с IEEE754.

rsa · 06.04.2011, 07:32

А. Тогда я не помощник. Изменение нагрузки замерить не смогу по причине кучи задач и ее сильной неравномерности. К тому же я сомневаюсь чтобы вносимая -ffast-math ошибка могла вообще как-то повлиять что на скорость что на точность (там минимум 6й знак плавает если вообще плавает). На глаз пока баранка. Посмотрим на стабильность, пока претензий нет...

KiriX · 06.04.2011, 09:26

Цитата:

Сообщение от virusav

Компилю в VS2008, никаких опцию не задаю, еще не смотрел, где это можно задать.

Хороший вопрос... Может заодно в теме подскажете?

Вообще не помню на форуме такого вопроса и ответа на него.

**Ambal** · 06.04.2011, 10:12

Цитата:

Сообщение от KiriX

Хороший вопрос... Может заодно в теме подскажете?

В Visual Studio эта опция активируется в закладке C/C++||Code Generation установкой значения Floating Point Model равного "Fast" или напрямую опцией компилятора "/fp:fast". Также настоятельно рекомендую вместе с этим флагом включить оптимизации для SSE/SSE2.

rsa: информация по стабильности и наличии/отсутствии багов меня тоже устроят

**Vladimir** · 06.04.2011, 11:15

Цитата:

Сообщение от rsa

А. Тогда я не помощник. Изменение нагрузки замерить не смогу по причине кучи задач и ее сильной неравномерности. К тому же я сомневаюсь чтобы вносимая -ffast-math ошибка могла вообще как-то повлиять что на скорость что на точность (там минимум 6й знак плавает если вообще плавает). На глаз пока баранка. Посмотрим на стабильность, пока претензий нет...

fast math пропускает проверки на NaN и INF соответствеено могут не работают некоторые проверки и изминится реультаты сравненеи

rsa · 06.04.2011, 12:32

Цитата:

Сообщение от Ambal

rsa: информация по стабильности и наличии/отсутствии багов меня тоже устроят

аптайм сутки, проблем нет пока. через недельку передоложу.
PS для информации, обсужденный краш с NAN значениями в GetHeight() получен _без_ включения fast_math.

**Ambal** · 06.04.2011, 12:47

Цитата:

Сообщение от rsa

обсужденный краш с NAN значениями в GetHeight() получен _без_ включения fast_math.

Я добавил проверку в [11320], которая поможет выследить тех мобов, у которых проблемы с координатами из-за багов в коде/базе.

02.04.2011, 17:40	#1
newsbot Модератор Регистрация: 06.03.2010 Сообщений: 988 Сказал(а) спасибо: 0 Поблагодарили 23 раз(а) в 14 сообщениях	[11305] more safe code in vmaps and Field class автор: Ambal m src/game/vmap/BIH.cpp m src/game/vmap/BIH.h m src/game/vmap/WorldModel.cpp m src/shared/Database/Field.h m src/shared/revision_nr.h [11305] more safe code in vmaps and Field class Signed-off-by: Ambal Дальше...

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
[11010] Remove duplicated code to start transaction in Mail class.	newsbot	CMaNGOS Commits	0	15.01.2011 14:50
[10982] Optimize memory usage of Field class used in DB code. Should also slightly speedup server st	newsbot	CMaNGOS Commits	0	07.01.2011 19:38
[10940] Apply mangos code style to in class {} indent.	newsbot	CMaNGOS Commits	0	30.12.2010 03:11
[10865] Make DB code thread-safe. Original patch by Machiavelli and Kero99.	newsbot	CMaNGOS Commits	0	12.12.2010 11:41
[9953] Remove two not needed cast to Creature* and make code more safe	newsbot	CMaNGOS Commits	0	22.05.2010 13:10

02.04.2011, 18:00	#2
Ambal MaNGOS Dev Регистрация: 22.06.2010 Сообщений: 78 Сказал(а) спасибо: 24 Поблагодарили 71 раз(а) в 25 сообщениях	Кто-то может собрать мангос с '-fast-math' и глянуть не стали ли vmaps лучше себя вести?

03.04.2011, 09:54	#3
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	а конкретнее, какая именно опция нужна? в gcc -ffast-math не работает если используется оптимизация (любой уровень). -ffast-math Sets -fno-math-errno, -funsafe-math-optimizations, -ffinite-math-only, -fno-rounding-math, -fno-signaling-nans and -fcx-limited-range.

04.04.2011, 21:29	#4
Ambal MaNGOS Dev Регистрация: 22.06.2010 Сообщений: 78 Сказал(а) спасибо: 24 Поблагодарили 71 раз(а) в 25 сообщениях	Именно этот флаг и нужен: -ffast-math. Он по-умолчанию не активируется если задействованы флаги оптимизации - включать нужно принудительно.

04.04.2011, 22:43	#5
virusav Администратор Регистрация: 19.02.2010 Сообщений: 492 Сказал(а) спасибо: 55 Поблагодарили 341 раз(а) в 154 сообщениях	Вопрос по VMAP: создал перса, примерно на уровне 10 портанулся в Штормград, пошел к тренеру оружейных навыков и по другим столицам и тренерам - все нормально, перс входит. На 20-22 уровне уже не могу войти в те здания, куда входил на 10-м. Это интеллектуальный учет роста перса со временем?

04.04.2011, 23:05	#6
Ambal MaNGOS Dev Регистрация: 22.06.2010 Сообщений: 78 Сказал(а) спасибо: 24 Поблагодарили 71 раз(а) в 25 сообщениях	virusav, встречный вопрос на засыпку: собирали с -ffast-math? Когда персу подняли лвл до 20-22 логаут делали?

05.04.2011, 10:02	#7
virusav Администратор Регистрация: 19.02.2010 Сообщений: 492 Сказал(а) спасибо: 55 Поблагодарили 341 раз(а) в 154 сообщениях	Компилю в VS2008, никаких опцию не задаю, еще не смотрел, где это можно задать. С 10 по 20-22 уровни была куча логаутов, несколько ребутов сервера.

05.04.2011, 16:29	#10
virusav Администратор Регистрация: 19.02.2010 Сообщений: 492 Сказал(а) спасибо: 55 Поблагодарили 341 раз(а) в 154 сообщениях	Про это думал, но факт остается фактом.

06.04.2011, 07:32	#13
rsa Почетный флудер Старожил Регистрация: 08.03.2010 Адрес: Мурманск, Россия Сообщений: 788 Сказал(а) спасибо: 55 Поблагодарили 333 раз(а) в 151 сообщениях Записей в дневнике: 1	А. Тогда я не помощник. Изменение нагрузки замерить не смогу по причине кучи задач и ее сильной неравномерности. К тому же я сомневаюсь чтобы вносимая -ffast-math ошибка могла вообще как-то повлиять что на скорость что на точность (там минимум 6й знак плавает если вообще плавает). На глаз пока баранка. Посмотрим на стабильность, пока претензий нет...