From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <rider@basealt.ru>
To: devel@lists.altlinux.org
References: <20171013093613.GA5570@lks.home>
 <8b319066-178a-af81-e6a2-0e42cf48fb16@basealt.ru>
 <20171013123233.GA30649@lks.home>
From: Anton Farygin <rider@basealt.ru>
Organization: BaseALT
Message-ID: <3e08395d-aa71-de93-654c-6c5715e8580d@basealt.ru>
Date: Fri, 13 Oct 2017 15:46:25 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <20171013123233.GA30649@lks.home>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
Subject: Re: [devel] Q: libva feedback
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Fri, 13 Oct 2017 12:46:26 -0000
Archived-At: <http://lore.altlinux.org/devel/3e08395d-aa71-de93-654c-6c5715e8580d@basealt.ru/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

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

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

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