Re: [PATCH v1 1/1] erofs-utils: fix endiannes issue

2025-04-29 Thread Alexander Egorenkov
Hi, Gao Xiang writes: > Hi Alexander, > > On Tue, Apr 29, 2025 at 09:30:52AM +0200, Alexander Egorenkov wrote: >> From: Super User > > Thanks for catching this, the "From:" line seems invalid, so > I change it as "From: Alexander Egorenkov " > Thanks! Argh, sorry for the wrong From field. > >

Re: [PATCH v1 1/1] erofs-utils: fix endiannes issue

2025-04-29 Thread Gao Xiang
Hi Alexander, On Tue, Apr 29, 2025 at 09:30:52AM +0200, Alexander Egorenkov wrote: > From: Super User Thanks for catching this, the "From:" line seems invalid, so I change it as "From: Alexander Egorenkov " > > Macros __BYTE_ORDER, __LITTLE_ENDIAN and __BIG_ENDIAN are defined in > user space h

Re: [PATCH v1 1/1] erofs-utils: fix endiannes issue

2025-04-29 Thread Ian Kent
On 29/4/25 16:30, Hongbo Li wrote: On 2025/4/29 15:30, Alexander Egorenkov wrote: From: Super User Macros __BYTE_ORDER, __LITTLE_ENDIAN and __BIG_ENDIAN are defined in user space header 'endian.h'. Not including this header results in the condition #if __BYTE_ORDER == __LITTLE_ENDIAN being a

Re: [PATCH v1 1/1] erofs-utils: fix endiannes issue

2025-04-29 Thread Hongbo Li
On 2025/4/29 15:30, Alexander Egorenkov wrote: From: Super User Macros __BYTE_ORDER, __LITTLE_ENDIAN and __BIG_ENDIAN are defined in user space header 'endian.h'. Not including this header results in the condition #if __BYTE_ORDER == __LITTLE_ENDIAN being always true, even on BE architecture

[PATCH v1 1/1] erofs-utils: fix endiannes issue

2025-04-29 Thread Alexander Egorenkov
From: Super User Macros __BYTE_ORDER, __LITTLE_ENDIAN and __BIG_ENDIAN are defined in user space header 'endian.h'. Not including this header results in the condition #if __BYTE_ORDER == __LITTLE_ENDIAN being always true, even on BE architectures (e.g. s390x). Due to this bug the compressor libra