initial unified impl magma/kuznetchik MGM mode

[yanovich]

Ported commit 214bb5e by igrkir i.kirillov@kryptonite.ru to v3.0

[yanovich]

Поправил #416

[yanovich]

FreeBSD CI совсем болеет

Configuring Term-Table-0.016 ... ! Installing the dependencies failed: Module 'Importer' is not installed
! Bailing out the installation for Term-Table-0.016.
! Installing the dependencies failed: Module 'Scope::Guard' is not installed, Module 'Sub::Info' is not installed, Module 'Module::Pluggable' is not installed, Module 'Term::Table' is not installed, Installed version (1.302175) of Test2::API is not in range '1.302176', Module 'Importer' is not installed
! Bailing out the installation for Test2-Suite-0.000145.
ld-elf.so.1: /usr/local/lib/perl5/5.32/mach/CORE/libperl.so.5.32: Undefined symbol "strerror_l@FBSD_1.6"

[beldmit]

@chipitsine можете посмотреть, как фрю вылечить?

[yanovich]

Динамические NID никогда не удаляются, что определяется как memory leak. Как с этим быть? Убирать их и делать только провайдер?

https://github.com/openssl/openssl/blob/openssl-3.0.5/crypto/objects/obj_dat.c#L202

[beldmit]

Правильный путь - это таки провайдер.
OBJ_free вроде бы что-то высвобождало, кажется?

[yanovich]

Больше не течёт )

[chipitsine]

@chipitsine можете посмотреть, как фрю вылечить?

октябрь был как в тумане.
там простая правка, сейчас залью

[chipitsine]

если хочется, можно долить мастер

[chipitsine]

@yanovich , посмотрите, пожалуйста #153
можно ли в Вашу ветку долить ту правку ?

[igrkir]

@yanovich , посмотрите, пожалуйста #153 можно ли в Вашу ветку долить ту правку ?

текущая реализация 214bb5e, портированная из ветки gost-engine:ossl_patched в ветку #414) уже имеет указанную оптимизацию.

[beldmit]

Какой нынче статус этой ветки? Ресурсов на всеобъемлющее тестирование у меня сейчас нет :(

[yanovich]

Какой нынче статус этой ветки? Ресурсов на всеобъемлющее тестирование у меня сейчас нет :(

Все ошибки во всех тестах, включая ASAN устанены. Ветка готова к мержу в мастер.

[yanovich]

Я только не знаю, как это принято у вас, но у себя я бы всё это одним коммитом заливал, чтобы потом git bisect корректно работал.

[beldmit]

У меня это никак особо специально не принято. Можете сделать squash коммитов? Я не уверен, что GitHub сохранит авторство, если я это сделаю через веб-интерфейс

[yanovich]

@igrkir С Вашего позволения, я объединю всё, что реализует -mgm для engine (код, тесты) в один коммит от Вашего имени с моими правками по утечкам памяти и мои механизмом динамических NID? В итоге будет три коммита:

1. механизм динамических NID @yanovich
2. -mgm для engine'а @igrkir
3. -mgm для provider'а @yanovich

После каждого коммита код будет полностью работоспособным.

[beldmit]

Да, идеальный вариант

[chipitsine]

в настройках уже выставлен squash commit

предыдущие пул реквесты со squash-ем вливались ?

[beldmit]

Обычно нет

[beldmit]

Merged.
Большое спасибо!