[devel] Q: turbojpeg on ppc64le

[devel] Q: turbojpeg on ppc64le

[Konstantin Lepikhov]

Привет!

[ppc64le]                  from src/mjpeg_decode.c:17:
[ppc64le] /usr/lib64/gcc/ppc64le-alt-linux/9/include/xmmintrin.h:54:2:
error: #error "Please read comment above.  Use -DNO_WARN_X86_INTRINSICS to
disable this error."
[ppc64le]    54 | #error "Please read comment above.  Use
-DNO_WARN_X86_INTRINSICS to disable this error."

Столкнулся при сборке вот с таким сообщением, мне оно кажется логичным,
что да, какие X86_INTRINSICS на ppc64. А теперь, внимание, вопрос:

А стоит ли вообще тогда собирать пакет libturbojpeg на ppc64, если он там
даже не работает?

-- 
WBR et al.

Re: [devel] Q: turbojpeg on ppc64le

[Alexey Sheplyakov]

On 09.12.2020 22:46, Konstantin Lepikhov wrote:


> [ppc64le]                  from src/mjpeg_decode.c:17:

А из какого района^W пакета этот файл?

> [ppc64le] /usr/lib64/gcc/ppc64le-alt-linux/9/include/xmmintrin.h:54:2:
> error: #error "Please read comment above.  Use -DNO_WARN_X86_INTRINSICS to
> disable this error."
> [ppc64le]    54 | #error "Please read comment above.  Use
> -DNO_WARN_X86_INTRINSICS to disable this error."
> 
> Столкнулся при сборке вот с таким сообщением, мне оно кажется логичным,
> что да, какие X86_INTRINSICS на ppc64
POWER8 VSX ISA (извините за caps, но так уж он называется) очень похож на SSE,
но есть ньюансы.

> А теперь, внимание, вопрос: 
> А стоит ли вообще тогда собирать пакет libturbojpeg на ppc64, если он там
> даже не работает?

А на основании чего сделан вывод о неработоспособности libjpeg-turbo?


 

Re: [devel] Q: turbojpeg on ppc64le

[Konstantin Lepikhov]

Hi Alexey!

On 12/09/2020, at 11:24:35 PM you wrote:

> On 09.12.2020 22:46, Konstantin Lepikhov wrote:
> 
> 
> > [ppc64le]                  from src/mjpeg_decode.c:17:
> 
> А из какого района^W пакета этот файл?
Как это относится к вопросу? В проекте просто есть такой код:

...
#include "turbojpeg.h"
}

#include "decoder.h"

struct MJpegDecoder : Decoder {
    tjhandle tj;
    uint8_t *frameBuf;
    int mSubsamp;

    MJpegDecoder(void) {
        ready = false;
        failed = false;
        alloc_count = 0;

        tj = NULL;
        frameBuf = NULL;
        mSubsamp = 0;
    }

    ~MJpegDecoder(void);
    bool init(void);
    bool decode_video(struct obs_source_frame2*, DataPacket*, bool *got_output);
    bool decode_audio(struct obs_source_audio* a, DataPacket* d, bool *got_output) {
        (void) a; (void) d;
        *got_output = false;
        return false;
    }

    void push_ready_packet(DataPacket*);
};
...

> 
> > [ppc64le] /usr/lib64/gcc/ppc64le-alt-linux/9/include/xmmintrin.h:54:2:
> > error: #error "Please read comment above.  Use -DNO_WARN_X86_INTRINSICS to
> > disable this error."
> > [ppc64le]    54 | #error "Please read comment above.  Use
> > -DNO_WARN_X86_INTRINSICS to disable this error."
> > 
> > Столкнулся при сборке вот с таким сообщением, мне оно кажется логичным,
> > что да, какие X86_INTRINSICS на ppc64
> POWER8 VSX ISA (извините за caps, но так уж он называется) очень похож на SSE,
> но есть ньюансы.
почем купил, за и то и продал. Вам виднее, я просто привел вывод от
gcc.

> 
> > А теперь, внимание, вопрос: 
> > А стоит ли вообще тогда собирать пакет libturbojpeg на ppc64, если он там
> > даже не работает?
> 
> А на основании чего сделан вывод о неработоспособности libjpeg-turbo?
На основании этого сообщения, доступа к железу у меня нет, поэтому 100%
сказать не могу. У меня теперь есть даже сомнения насчет arm, поскольку
там то NEON вряд ли на SSE похож.

-- 
WBR et al.

Re: [devel] Q: turbojpeg on ppc64le

[Alexey Sheplyakov]

On 09.12.2020 23:42, Konstantin Lepikhov wrote:

> On 12/09/2020, at 11:24:35 PM you wrote:
> 
>> On 09.12.2020 22:46, Konstantin Lepikhov wrote:
>>
>>
>>> [ppc64le]                  from src/mjpeg_decode.c:17:
>>
>> А из какого района^W пакета этот файл?
> Как это относится к вопросу?

Непосредственно. Предполагаю, что проблема не в libjpeg-turbo, а в самом коде,
который ее использует. Для того, чтобы обосновать предположение, и указать, где
именно затык, нужно исходник.

> В проекте просто есть такой код:
> 
> ...
> #include "turbojpeg.h"
> }
> 
> #include "decoder.h"
> 
> struct MJpegDecoder : Decoder {
>     tjhandle tj;
>     uint8_t *frameBuf;
>     int mSubsamp;
> 
>     MJpegDecoder(void) {
>         ready = false;
>         failed = false;
>         alloc_count = 0;
> 
>         tj = NULL;
>         frameBuf = NULL;
>         mSubsamp = 0;
>     }
> 
>     ~MJpegDecoder(void);
>     bool init(void);
>     bool decode_video(struct obs_source_frame2*, DataPacket*, bool *got_output);
>     bool decode_audio(struct obs_source_audio* a, DataPacket* d, bool *got_output) {
>         (void) a; (void) d;
>         *got_output = false;
>         return false;
>     }
> 
>     void push_ready_packet(DataPacket*);
> };
> ...


И без многоточий. Потому что в них самое интересное.


>> А на основании чего сделан вывод о неработоспособности libjpeg-turbo?
> На основании этого сообщения,

Это сообщение не дает достаточных оснований считать libjpeg-turbo неработоспособным.
Потому что в libjpeg-turbo нет упоминаний xmmintrin.h:
https://github.com/libjpeg-turbo/libjpeg-turbo/search?q=xmmintrin.h


> У меня теперь есть даже сомнения насчет arm, поскольку
> там то NEON вряд ли на SSE похож.

Авторы libjpeg-turbo вряд ли настолько упороты. Судя по наличию AltiVec кода,
они вполне в курсе про ppc. Скорее авторы проблемного кода (приведенного выше
struct MJpegDecoder) не знают, что бывает что-то кроме x86 и arm. (А Linux
видели только на картинке).

Re: [devel] Q: turbojpeg on ppc64le

[Konstantin Lepikhov]

Hi Alexey!

On 12/10/2020, at 12:26:00 AM you wrote:

<skip>
> > У меня теперь есть даже сомнения насчет arm, поскольку
> > там то NEON вряд ли на SSE похож.
> 
> Авторы libjpeg-turbo вряд ли настолько упороты. Судя по наличию AltiVec кода,
> они вполне в курсе про ppc. Скорее авторы проблемного кода (приведенного выше
> struct MJpegDecoder) не знают, что бывает что-то кроме x86 и arm. (А Linux
> видели только на картинке).
> 
"упороты", "Linux на картинке". Спасибо, я все понял!

-- 
WBR et al.