[devel] Q: libva feedback

[Anton Farygin]

13.10.2017 15:32, Konstantin Lepikhov пишет:
> Hi Anton!
>
> On 10/13/2017, at 02:35:32 PM you wrote:
>
>> Я всегда проверяю libva на железе перед отправкой на связке ffmpeg+vlc.
>>
>> Что там за неправильный merge ?
>>
> <skip>
> http://git.altlinux.org/people/lakostis/packages/?p=libva.git;a=blobdiff;f=va/va.c;h=d4c188f66d4d0494ab84988a4442c0b2dece955d;hp=7f027dd8d812b53e41835fd094a9b903daad0e76;hb=a620ea1ba8a174cbf2c699b763a8ac0bf796c2c4;hpb=39a559e6e117f0d6aae62a0e541544a36a19caed
>
> @@ -648,8 +648,6 @@ VAStatus vaInitialize (
> ...
>
> Насколько я помню из своего патча для /etc/libva.conf, там не должно быть
> повторного вызова getenv, поскольку значение LIBVA_DRIVER_NAME будет null
> если переменнная окружения не задана и это значение перепишет предыдущее
> из /etc/libva.conf
>
> см. va/va.c:
>
>    59 /*
>    60  * read a config "env" for libva.conf or from environment setting
>    61  * libva.conf has higher priority
>    62  * return 0: the "env" is set, and the value is copied into env_value
>    63  *        1: the env is not set
>    64  */
>    65 int va_parseConfig(char *env, char *env_value)
>
Сейчас насколько я вижу это всё работает так, что через переменную 
окружения LIBVA_DRIVER_NAME вообще невозможно переопределить драйвер. 
Только через конфигурационный файл.

Повторного вызова getenv нет, как и первого.
Твой коммит :
http://git.altlinux.org/gears/l/libva.git?p=libva.git;a=commitdiff;h=722acba63ea18ba7daee32fa8af76666b73a83b7

вносит такое поведение и в этом месте никаких изменений больше не было.

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