Board.KolibriOS.org

Официальный форум KolibriOS
Текущее время: Чт ноя 15, 2018 12:03 pm

Часовой пояс: UTC+03:00




Начать новую тему  Эта тема закрыта, вы не можете редактировать и оставлять сообщения в ней.  [ 30 сообщений ]  На страницу 1 2 След.

Оставить как есть или добавить фикс отменяющий 3936 ?
Опрос закончился Ср окт 24, 2018 9:02 pm
Как есть 33%  33%  [ 2 ]
Ревертнуть 67%  67%  [ 4 ]
Всего голосов: 6
Автор Сообщение
 Заголовок сообщения: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 8:56 pm 
Не в сети
Designer
Аватара пользователя

Зарегистрирован: Чт янв 25, 2007 3:33 pm
Сообщения: 4878
ССЫЛКА Mario_r4 писал(а):
0CodErr писал(а):
Тут на картинке показано, как при перемещении окна Calc, не выходящего за пределы окна KPack, также перерисовывается Debugger, окно которого находится ниже окна KPack:
Спойлер: Показать
Изображение
Подобное происходит и с другими приложениями. Так ведь не должно быть?

Исправлено в SVN r. 3936.
Изменение исправило баг, но повлекло за собой:
1. другой баг viewtopic.php?f=36&t=3395&p=72096#p72096
2. существенное уменьшение производительности на 25-40%

Вопрос: оставить как есть или добавить фикс отменяющий 3936 (достаточно закоментировать одну строчку 3843 в kernel.asm) ?


Вложения:
3936.png
3936.png [ 17 КБ | 420 просмотров ]

_________________
Через тернии к звездам
Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 9:08 pm 
Не в сети
Аватара пользователя

Зарегистрирован: Чт янв 19, 2017 9:11 pm
Сообщения: 14
А там нет просто ошибки off-by-one?


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 9:11 pm 
Не в сети
Designer
Аватара пользователя

Зарегистрирован: Чт янв 25, 2007 3:33 pm
Сообщения: 4878
nina писал(а):
А там нет просто ошибки off-by-one?
Где-то есть и ее возможно исправить, а вот производительность нет.

_________________
Через тернии к звездам


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 9:33 pm 
Не в сети
Аватара пользователя

Зарегистрирован: Чт янв 19, 2017 9:11 pm
Сообщения: 14
А, тем более тогда надо откатывать, имхо.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 9:43 pm 
Не в сети

Зарегистрирован: Вс окт 30, 2011 6:43 pm
Сообщения: 1503
Leency писал(а):
уменьшение производительности на 25-40%
Потому что появились дополнительные проверки.
А без них снова будет
0CodErr писал(а):
при перемещении окна Calc, не выходящего за пределы окна KPack, также перерисовывается Debugger, окно которого находится ниже окна KPack
Иными словами, снова будет происходить совершенно ненужная(а как же производительность?) перерисовка.
Ну а MGB — синтетический бенчмарк, другого результата и не могло быть.

Я считаю, что баг исправлять нужно(что и было сделано).
Возможно, есть способ немного оптимизировать код дополнительных проверок.
Тот баг viewtopic.php?f=36&t=3395&p=72096#p72096 исправлять тоже нужно.
Поэтому данный опрос немного странный.
Вам какой баг оставить: тот или этот? Выбирайте. :lol:

Leency, ты исправил SysFn73:Blit viewtopic.php?f=2&t=3427&p=72110#p72110 но баг там появился вследствии "Small optimization", то есть ты ухудшил производительность, но, однако же теперь стало работать правильно.
Быстрый, но работающий неправильно код — это хуже, чем более медленный, но работающий правильно.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 9:56 pm 
Не в сети
Designer
Аватара пользователя

Зарегистрирован: Чт янв 25, 2007 3:33 pm
Сообщения: 4878
Если бы падение производительности было 5-15% одно дело, но 25-40% совершенно другое.

_________________
Через тернии к звездам


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 10:06 pm 
Не в сети

Зарегистрирован: Вс окт 30, 2011 6:43 pm
Сообщения: 1503
Leency писал(а):
5-15% одно дело, но 25-40% совершенно другое.
:lol: Совершенно не имеет значения.
Посмотри-ка, сколько дополнительных проверок происходит до вызова real put_pixel function http://websvn.kolibrios.org/filedetails ... 3#line-865 может и их надо убрать?

Я не голосовал, так как опрос глупый по своей сути.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 10:41 pm 
Не в сети

Зарегистрирован: Сб апр 22, 2017 6:11 pm
Сообщения: 219
Немного демагогии...
Просадка производительности выглядит серьёзно, вероятно имеет место плохая архитектура, поэтому проверки столько съели. Опять же если окон будет много, все они будут перерисовываться, в некотором случае - всё может вывернуться наоборот. И мы получим не прирост, а просадку (после отката). Всё зависит от общей реализации.

По хорошему нужно разобраться в архитектуре, откатить и наложить хороший патч (значительно лучший, чтобы обычные просадки были не больше 3%, а остальное не изменилось). Если желающих нет, то лучше не трогать.

Leency, протестируй с несколькими окнами, если перерисовку (баг, показанный 0CodErr) и падения производительности не видно, то по-моему откат лучшее решение. Это позволит начать движение к переделке.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 11:06 pm 
Не в сети
Designer
Аватара пользователя

Зарегистрирован: Чт янв 25, 2007 3:33 pm
Сообщения: 4878
Ок, мои доводы: падение производительности влияет на все без исключения программы.
А выигрыш, т.е отсутствие перерисовки, будет в очень небольшом проценте случаев, вдумайтесь: когда есть три программы и самая верхняя передвигается в пределах средней. Это не стоит 25-40%.

Кстати, перерисовку MTDBG из первого поста я исправил давным давно.


Вложения:
scr.png
scr.png [ 31.31 КБ | 341 просмотр ]

_________________
Через тернии к звездам


Последний раз редактировалось Leency Ср окт 17, 2018 11:07 pm, всего редактировалось 1 раз.
Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 11:07 pm 
Не в сети

Зарегистрирован: Вт июн 24, 2008 11:12 pm
Сообщения: 62
Фикс 3936 абсолютно к месту. Он решает определённую проблему, которая вполне очевидна. Но надо исправить проблему с 1px. А цифры бенчмарка совершенно логично свидетельствую о том, что добавились доп. проверки. Но из этих цифр не очень понятно на сколько реально замедлилась работа по отрисовке GUI.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 11:14 pm 
Не в сети
Designer
Аватара пользователя

Зарегистрирован: Чт янв 25, 2007 3:33 pm
Сообщения: 4878
Фикс поидее должнен был затронуть только момент после передвижения окна, почему он повлиял на рисование в самом окне не понятно.

_________________
Через тернии к звездам


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Ср окт 17, 2018 11:27 pm 
Не в сети

Зарегистрирован: Вс окт 30, 2011 6:43 pm
Сообщения: 1503
Leency писал(а):
Кстати, перерисовку MTDBG из первого поста я исправил давным давно.
Это вообще не имеет отношения к теме.
Приложение получает сообщение "надо перерисоваться", и затем вызывается процедура перерисовки(BeginDraw, EndDraw, DrawWindow, DrawButton, DrawText, и вызовы других системных функций, если они используются).
Или логика у вас, у художников: если не мерцает, то и ресурсы не расходуются? :mrgreen:
Ресурсы расходуются независимо от того, заметил ты это или нет.
Leency писал(а):
почему он повлиял на рисование в самом окне не понятно
Во время рисования точки проверяется, какому окну она принадлежит(_WinMapAddress).
Возможно, эту проверку можно оптимизировать.
Но это странно, что необходимость этой проверки ещё вызывает у кого-то вопросы.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Чт окт 18, 2018 8:19 pm 
Не в сети
Moderator

Зарегистрирован: Чт апр 08, 2010 8:11 pm
Сообщения: 269
Нужно вернуть производительность и по возможности найти более элегантное решение бага. Тем более, Leency, только ты окна попиксельно двигаешь :).


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Чт окт 18, 2018 9:03 pm 
Не в сети

Зарегистрирован: Вс окт 30, 2011 6:43 pm
Сообщения: 1503
popovpa, "вернуть производительность" и "пофиксить баг" в данном случае взаимоисключающие вещи.

Оба бага — плохо. И тот и другой нужно фиксить.
Потеря производительности — тоже плохо, нужно по возможности оптимизировать дополнительные проверки.

Я не знаю, следишь ли ты за проектом и насколько именно.
Но вот тебе ещё пример http://websvn.kolibrios.org/filedetails ... 4#line-308
Просто твои мысли по этому поводу? Правильная работа или быстрая работа? Что лучше?
Ну этот http://websvn.kolibrios.org/filedetails ... 3#line-865 ты выше видел, наверное.
А ещё есть переключение контекста, однозадачная ОС могла быть ещё быстрее.
Есть вещи, которые так или иначе отъедают ресурсы, но это нужные вещи!

А данный опрос не имеет никакой силы.
Глупо спрашивать, нужно ли было фиксить баг или стоит вернуть баг на место.
Ведь и так понятно, что любые баги нужно фиксить. И совершенно неважно, что там наголосуют художники со школьниками.

Я со всей ответственностью заявляю, что в случае возврата к багу, откачу обратно.
Я не собираюсь учитывать мнение некомпетентных людей.
Вообще я как-то уже предлагал сделать ответственных людей за коммиты, а не коммитить всем подряд, я бы предложил, например, таких людей: CleverMouse, Serge, hidnplayr, dunkaist.

А кое-кого следовало вообще забанить.
Модератор кое-что потёр уже, но archive.is всё помнит http://archive.is/eW7Uq

Ну, в общем, без администрации тут явно не обойтись.
Очень странно, что админы до сих пор не вмешиваются.
Я же со своей стороны будут делать так, как считаю правильным.
Баги в любом случае нужно фиксить.
Уж если админы скажут, что баги это норм, зато быстро — ну ок, продолжайте закапывать проект и дальше, а пока что я спокойно молчать не буду.


Вернуться к началу
 
 Заголовок сообщения: Re: Ревизия 3936
СообщениеДобавлено: Чт окт 18, 2018 9:20 pm 
Не в сети

Зарегистрирован: Вт мар 08, 2016 11:00 pm
Сообщения: 412
0CodErr писал(а):
popovpa, "вернуть производительность" и "пофиксить баг" в данном случае взаимоисключающие вещи.

Оба бага — плохо. И тот и другой нужно фиксить.
Потеря производительности — тоже плохо, нужно по возможности оптимизировать дополнительные проверки.

Я не знаю, следишь ли ты за проектом и насколько именно.
Но вот тебе ещё пример http://websvn.kolibrios.org/filedetails ... 4#line-308
Просто твои мысли по этому поводу? Правильная работа или быстрая работа? Что лучше?
Ну этот http://websvn.kolibrios.org/filedetails ... 3#line-865 ты выше видел, наверное.
А ещё есть переключение контекста, однозадачная ОС могла быть ещё быстрее.
Есть вещи которые так или иначе отъедают ресурсы, но это нужные вещи!

А данный опрос не имеет никакой силы.
Глупо спрашивать, нужно ли было фиксить баг или стоит вернуть баг на место.
Ведь и так понятно, что любые баги нужно фиксить. И совершенно неважно, что там наголосуют художники со школьниками.

Я со всей ответственностью заявляю, что в случае возврата к багу, откачу обратно.
Я не собираюсь учитывать мнение некомпетентных людей.
Вообще я как-то уже предлагал сделать ответственных людей за коммиты, а не коммитить всем подряд, я бы предложил, например, таких людей: CleverMouse, Serge, hidnplayr, dunkaist.

А кое-кого следовало вообще забанить.
Модератор кое-что потёр уже, но archive.is всё помнит http://archive.is/eW7Uq

Ну, в общем, без администрации тут явно не обойтись.
Очень странно, что админы до сих пор не вмешиваются.
Я же со своей стороны будут делать так, как считаю правильным.
Баги в любом случае нужно фиксить.
Уж если админы скажут, что баги это норм, зато быстро — ну ок, продолжайте закапывать проект и дальше, а пока что я спокойно молчать не буду.
Фиксить то можно по разному.
1. Впендюрить первое попавшееся решение. Как раз что ты предлагаешь.
2. Оценить влияние на остальное окружение и действовать дальше:
2.1 Забить
2.2 Сделать воркэраунд
2.3 Задокументировать
2.4. Проработать в рабочем порядке и выбрать оптимальный результат.

Поскольку этот баг никому не мешал, спешка ни к чему.

У меня вот, кнопка закрытия окна, если находится над иконками десктопа,перестают работать. Возможно, это связано


Вернуться к началу
 
Показать сообщения за:  Поле сортировки  
Начать новую тему  Эта тема закрыта, вы не можете редактировать и оставлять сообщения в ней.  [ 30 сообщений ]  На страницу 1 2 След.

Часовой пояс: UTC+03:00


Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и 1 гость


Вы не можете начинать темы
Вы не можете отвечать на сообщения
Вы не можете редактировать свои сообщения
Вы не можете удалять свои сообщения
Вы не можете добавлять вложения

Найти:
Перейти:  
Создано на основе phpBB® Forum Software © phpBB Limited
Русская поддержка phpBB