Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

E2k support #91

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

makise-homura
Copy link

This pull request introduces support for warningless building of ftl-sdk as a requirement of obs-studio on Elbrus hardware platform (which is based on Russian Elbrus CPU family) with its native lcc (eLbrus Compiler Collection) compiler.

As currently Elbrus-based stations are introduced into general puprose applications, including video streaming and recording, it is better to have OBS buildable on this platform directly from upstream without any warnings, rather than being patched with some specific patches prior to building, or producing mass of irrelevant warnings while being built.

This was:

  • Built on Elbrus workstation (e2k-linux-gnu);
  • Built on x86_64 workstation (x86_64-linux-gnu) to be sure that nothing is broken on x86_64.

Details of Elbrus workstation:

  • CPU: Elbrus-8C (formerly Elbrus-8S) running on 1300 MHz
  • Motherboard: E8C-EATX
  • OS: OS Elbrus 5.0-rc2
  • GPU: Advanced Micro Devices, Inc. [AMD/ATI] Vega 10 XL/XT [Radeon RX Vega 56/64] (rev c1)
  • Kernel: 5.4.0-1.1-e8c
  • Compiler: lcc:1.24.09:Feb-11-2020:e2k-v4-linux (gcc (GCC) 7.3.0 compatible)

Details of x86_64 workstation:

  • CPU: AMD Ryzen Threadripper 2990WX 32-Core Processor running at 3.0 GHz
  • Motherboard: MSI X399 SLI PLUS (MS-7B09)
  • GPU: Advanced Micro Devices, Inc. [AMD/ATI] Caicos [Radeon HD 6450/7450/8450 / R5 230 OEM]
  • OS: Ubuntu 18.04.5 LTS
  • Kernel: 5.3.0-28-generic another rtp corner case fixed #30~18.04.1-Ubuntu
  • Compiler: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0

Results:

Successfully performed mkdir build; cd build; cmake -DCMAKE_BUILD_TYPE=Release .. && make -j16 && sudo make install; no suspicious behavior found, no warnings produced on e2k; -Wint-to-pointer-cast produced at libftl/ingest.c:338:10 on x86_64, but this seems to be unavoidable considering return type of _ingest_get_hosts().

@ghost
Copy link

ghost commented Aug 18, 2020

CLA assistant check
All CLA requirements met.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant