* [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems
@ 2025-03-20 13:17 Zhao Zhili
2025-03-20 21:31 ` Sean McGovern
2025-03-20 21:50 ` James Almer
0 siblings, 2 replies; 4+ messages in thread
From: Zhao Zhili @ 2025-03-20 13:17 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Zhao Zhili
From: Zhao Zhili <zhilizhao@tencent.com>
hevc decoder output native endian yuv420p10. yuv420p10 is alias of
yuv420p10le on little endian system, and yuv420p10be on big endian
system.
---
tests/fate/flvenc.mak | 2 +-
tests/ref/fate/enhanced-flv-hevc-hdr10 | 19 -------------------
2 files changed, 1 insertion(+), 20 deletions(-)
diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak
index 471bb0fe72..22208aa64b 100644
--- a/tests/fate/flvenc.mak
+++ b/tests/fate/flvenc.mak
@@ -15,7 +15,7 @@ fate-enhanced-flv-av1: CMD = stream_remux ivf $(TARGET_SAMPLES)/av1/seq_hdr_op_p
FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV HEVC, FLV_DEMUXER HEVC_DECODER HEVC_PARSER) += fate-enhanced-flv-hevc-hdr10
fate-enhanced-flv-hevc-hdr10: CMD = stream_remux hevc $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc "-c:v hevc" \
- flv "-c copy" "-c:v hevc" "-c copy" "-show_frames"
+ flv "-c copy" "-c:v hevc" "-c copy" "-show_frames -show_entries frame=media_type,stream_index,key_frame,pts,width,height,pict_type,color_range,color_space,color_primaries,color_transfer,chroma_location,side_data"
FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV, FLV_DEMUXER AAC_PARSER AC3_PARSER OPUS_PARSER FLAC_PARSER VP9_PARSER AV1_PARSER HEVC_PARSER H264_PARSER) += fate-enhanced-flv-multitrack
fate-enhanced-flv-multitrack: CMD = stream_remux flv $(TARGET_SAMPLES)/flv/multitrack.flv "" flv "-map 0" "" "-c copy -map 0" \
diff --git a/tests/ref/fate/enhanced-flv-hevc-hdr10 b/tests/ref/fate/enhanced-flv-hevc-hdr10
index cb7acea328..8495572705 100644
--- a/tests/ref/fate/enhanced-flv-hevc-hdr10
+++ b/tests/ref/fate/enhanced-flv-hevc-hdr10
@@ -10,28 +10,9 @@ media_type=video
stream_index=0
key_frame=1
pts=0
-pts_time=0.000000
-pkt_dts=0
-pkt_dts_time=0.000000
-best_effort_timestamp=0
-best_effort_timestamp_time=0.000000
-duration=N/A
-duration_time=N/A
-pkt_pos=459
-pkt_size=77718
width=1280
height=720
-crop_top=0
-crop_bottom=0
-crop_left=0
-crop_right=0
-pix_fmt=yuv420p10le
-sample_aspect_ratio=N/A
pict_type=I
-interlaced_frame=0
-top_field_first=0
-lossless=0
-repeat_pict=0
color_range=tv
color_space=bt2020nc
color_primaries=bt2020
--
2.46.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems
2025-03-20 13:17 [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems Zhao Zhili
@ 2025-03-20 21:31 ` Sean McGovern
2025-03-20 21:50 ` James Almer
1 sibling, 0 replies; 4+ messages in thread
From: Sean McGovern @ 2025-03-20 21:31 UTC (permalink / raw)
To: FFmpeg development discussions and patches; +Cc: Zhao Zhili
Hi,
On Thu, Mar 20, 2025 at 9:17 AM Zhao Zhili
<quinkblack-at-foxmail.com@ffmpeg.org> wrote:
>
> From: Zhao Zhili <zhilizhao@tencent.com>
>
> hevc decoder output native endian yuv420p10. yuv420p10 is alias of
> yuv420p10le on little endian system, and yuv420p10be on big endian
> system.
> ---
> tests/fate/flvenc.mak | 2 +-
> tests/ref/fate/enhanced-flv-hevc-hdr10 | 19 -------------------
> 2 files changed, 1 insertion(+), 20 deletions(-)
>
> diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak
> index 471bb0fe72..22208aa64b 100644
> --- a/tests/fate/flvenc.mak
> +++ b/tests/fate/flvenc.mak
> @@ -15,7 +15,7 @@ fate-enhanced-flv-av1: CMD = stream_remux ivf $(TARGET_SAMPLES)/av1/seq_hdr_op_p
>
> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV HEVC, FLV_DEMUXER HEVC_DECODER HEVC_PARSER) += fate-enhanced-flv-hevc-hdr10
> fate-enhanced-flv-hevc-hdr10: CMD = stream_remux hevc $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc "-c:v hevc" \
> - flv "-c copy" "-c:v hevc" "-c copy" "-show_frames"
> + flv "-c copy" "-c:v hevc" "-c copy" "-show_frames -show_entries frame=media_type,stream_index,key_frame,pts,width,height,pict_type,color_range,color_space,color_primaries,color_transfer,chroma_location,side_data"
>
> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV, FLV_DEMUXER AAC_PARSER AC3_PARSER OPUS_PARSER FLAC_PARSER VP9_PARSER AV1_PARSER HEVC_PARSER H264_PARSER) += fate-enhanced-flv-multitrack
> fate-enhanced-flv-multitrack: CMD = stream_remux flv $(TARGET_SAMPLES)/flv/multitrack.flv "" flv "-map 0" "" "-c copy -map 0" \
> diff --git a/tests/ref/fate/enhanced-flv-hevc-hdr10 b/tests/ref/fate/enhanced-flv-hevc-hdr10
> index cb7acea328..8495572705 100644
> --- a/tests/ref/fate/enhanced-flv-hevc-hdr10
> +++ b/tests/ref/fate/enhanced-flv-hevc-hdr10
> @@ -10,28 +10,9 @@ media_type=video
> stream_index=0
> key_frame=1
> pts=0
> -pts_time=0.000000
> -pkt_dts=0
> -pkt_dts_time=0.000000
> -best_effort_timestamp=0
> -best_effort_timestamp_time=0.000000
> -duration=N/A
> -duration_time=N/A
> -pkt_pos=459
> -pkt_size=77718
> width=1280
> height=720
> -crop_top=0
> -crop_bottom=0
> -crop_left=0
> -crop_right=0
> -pix_fmt=yuv420p10le
> -sample_aspect_ratio=N/A
> pict_type=I
> -interlaced_frame=0
> -top_field_first=0
> -lossless=0
> -repeat_pict=0
> color_range=tv
> color_space=bt2020nc
> color_primaries=bt2020
> --
> 2.46.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
OK, and I wonder if the other tests that vary between endianness
should do likewise if they don't really care about the pixel format.
-- Sean McGovern
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems
2025-03-20 13:17 [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems Zhao Zhili
2025-03-20 21:31 ` Sean McGovern
@ 2025-03-20 21:50 ` James Almer
2025-03-21 4:11 ` Zhao Zhili
1 sibling, 1 reply; 4+ messages in thread
From: James Almer @ 2025-03-20 21:50 UTC (permalink / raw)
To: ffmpeg-devel
[-- Attachment #1.1.1: Type: text/plain, Size: 2556 bytes --]
On 3/20/2025 10:17 AM, Zhao Zhili wrote:
> From: Zhao Zhili <zhilizhao@tencent.com>
>
> hevc decoder output native endian yuv420p10. yuv420p10 is alias of
> yuv420p10le on little endian system, and yuv420p10be on big endian
> system.
> ---
> tests/fate/flvenc.mak | 2 +-
> tests/ref/fate/enhanced-flv-hevc-hdr10 | 19 -------------------
> 2 files changed, 1 insertion(+), 20 deletions(-)
>
> diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak
> index 471bb0fe72..22208aa64b 100644
> --- a/tests/fate/flvenc.mak
> +++ b/tests/fate/flvenc.mak
> @@ -15,7 +15,7 @@ fate-enhanced-flv-av1: CMD = stream_remux ivf $(TARGET_SAMPLES)/av1/seq_hdr_op_p
>
> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV HEVC, FLV_DEMUXER HEVC_DECODER HEVC_PARSER) += fate-enhanced-flv-hevc-hdr10
> fate-enhanced-flv-hevc-hdr10: CMD = stream_remux hevc $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc "-c:v hevc" \
> - flv "-c copy" "-c:v hevc" "-c copy" "-show_frames"
> + flv "-c copy" "-c:v hevc" "-c copy" "-show_frames -show_entries frame=media_type,stream_index,key_frame,pts,width,height,pict_type,color_range,color_space,color_primaries,color_transfer,chroma_location,side_data"
>
> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV, FLV_DEMUXER AAC_PARSER AC3_PARSER OPUS_PARSER FLAC_PARSER VP9_PARSER AV1_PARSER HEVC_PARSER H264_PARSER) += fate-enhanced-flv-multitrack
> fate-enhanced-flv-multitrack: CMD = stream_remux flv $(TARGET_SAMPLES)/flv/multitrack.flv "" flv "-map 0" "" "-c copy -map 0" \
> diff --git a/tests/ref/fate/enhanced-flv-hevc-hdr10 b/tests/ref/fate/enhanced-flv-hevc-hdr10
> index cb7acea328..8495572705 100644
> --- a/tests/ref/fate/enhanced-flv-hevc-hdr10
> +++ b/tests/ref/fate/enhanced-flv-hevc-hdr10
> @@ -10,28 +10,9 @@ media_type=video
> stream_index=0
> key_frame=1
> pts=0
> -pts_time=0.000000
> -pkt_dts=0
> -pkt_dts_time=0.000000
> -best_effort_timestamp=0
> -best_effort_timestamp_time=0.000000
> -duration=N/A
> -duration_time=N/A
> -pkt_pos=459
> -pkt_size=77718
> width=1280
> height=720
> -crop_top=0
> -crop_bottom=0
> -crop_left=0
> -crop_right=0
> -pix_fmt=yuv420p10le
> -sample_aspect_ratio=N/A
> pict_type=I
> -interlaced_frame=0
> -top_field_first=0
> -lossless=0
> -repeat_pict=0
> color_range=tv
> color_space=bt2020nc
> color_primaries=bt2020
Did you check the test i mentioned that solved this with a sed string
replace? I'd rather do that than print less information.
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems
2025-03-20 21:50 ` James Almer
@ 2025-03-21 4:11 ` Zhao Zhili
0 siblings, 0 replies; 4+ messages in thread
From: Zhao Zhili @ 2025-03-21 4:11 UTC (permalink / raw)
To: FFmpeg development discussions and patches
> On Mar 21, 2025, at 05:50, James Almer <jamrial@gmail.com> wrote:
>
> On 3/20/2025 10:17 AM, Zhao Zhili wrote:
>> From: Zhao Zhili <zhilizhao@tencent.com>
>> hevc decoder output native endian yuv420p10. yuv420p10 is alias of
>> yuv420p10le on little endian system, and yuv420p10be on big endian
>> system.
>> ---
>> tests/fate/flvenc.mak | 2 +-
>> tests/ref/fate/enhanced-flv-hevc-hdr10 | 19 -------------------
>> 2 files changed, 1 insertion(+), 20 deletions(-)
>> diff --git a/tests/fate/flvenc.mak b/tests/fate/flvenc.mak
>> index 471bb0fe72..22208aa64b 100644
>> --- a/tests/fate/flvenc.mak
>> +++ b/tests/fate/flvenc.mak
>> @@ -15,7 +15,7 @@ fate-enhanced-flv-av1: CMD = stream_remux ivf $(TARGET_SAMPLES)/av1/seq_hdr_op_p
>> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV HEVC, FLV_DEMUXER HEVC_DECODER HEVC_PARSER) += fate-enhanced-flv-hevc-hdr10
>> fate-enhanced-flv-hevc-hdr10: CMD = stream_remux hevc $(TARGET_SAMPLES)/hevc/hdr10_plus_h265_sample.hevc "-c:v hevc" \
>> - flv "-c copy" "-c:v hevc" "-c copy" "-show_frames"
>> + flv "-c copy" "-c:v hevc" "-c copy" "-show_frames -show_entries frame=media_type,stream_index,key_frame,pts,width,height,pict_type,color_range,color_space,color_primaries,color_transfer,chroma_location,side_data"
>> FATE_ENHANCED_FLVENC_FFMPEG_FFPROBE-$(call REMUX, FLV, FLV_DEMUXER AAC_PARSER AC3_PARSER OPUS_PARSER FLAC_PARSER VP9_PARSER AV1_PARSER HEVC_PARSER H264_PARSER) += fate-enhanced-flv-multitrack
>> fate-enhanced-flv-multitrack: CMD = stream_remux flv $(TARGET_SAMPLES)/flv/multitrack.flv "" flv "-map 0" "" "-c copy -map 0" \
>> diff --git a/tests/ref/fate/enhanced-flv-hevc-hdr10 b/tests/ref/fate/enhanced-flv-hevc-hdr10
>> index cb7acea328..8495572705 100644
>> --- a/tests/ref/fate/enhanced-flv-hevc-hdr10
>> +++ b/tests/ref/fate/enhanced-flv-hevc-hdr10
>> @@ -10,28 +10,9 @@ media_type=video
>> stream_index=0
>> key_frame=1
>> pts=0
>> -pts_time=0.000000
>> -pkt_dts=0
>> -pkt_dts_time=0.000000
>> -best_effort_timestamp=0
>> -best_effort_timestamp_time=0.000000
>> -duration=N/A
>> -duration_time=N/A
>> -pkt_pos=459
>> -pkt_size=77718
>> width=1280
>> height=720
>> -crop_top=0
>> -crop_bottom=0
>> -crop_left=0
>> -crop_right=0
>> -pix_fmt=yuv420p10le
>> -sample_aspect_ratio=N/A
>> pict_type=I
>> -interlaced_frame=0
>> -top_field_first=0
>> -lossless=0
>> -repeat_pict=0
>> color_range=tv
>> color_space=bt2020nc
>> color_primaries=bt2020
>
> Did you check the test i mentioned that solved this with a sed string replace? I'd rather do that than print less information.
The test first remux hevc to flv, then probe and check frame colorspace and side data information. How to use sed together with stream_remux?
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-03-21 4:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-03-20 13:17 [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems Zhao Zhili
2025-03-20 21:31 ` Sean McGovern
2025-03-20 21:50 ` James Almer
2025-03-21 4:11 ` Zhao Zhili
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git