* [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; 7+ 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] 7+ 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; 7+ 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] 7+ 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; 7+ 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] 7+ 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 2025-03-29 15:52 ` Zhao Zhili 0 siblings, 1 reply; 7+ 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] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems 2025-03-21 4:11 ` Zhao Zhili @ 2025-03-29 15:52 ` Zhao Zhili 2025-03-29 17:06 ` Andreas Rheinhardt 0 siblings, 1 reply; 7+ messages in thread From: Zhao Zhili @ 2025-03-29 15:52 UTC (permalink / raw) To: FFmpeg development discussions and patches Ping. Fate on powerpc is still broken. > On Mar 21, 2025, at 12:11, Zhao Zhili <quinkblack@foxmail.com> wrote: > >> >> 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 <mailto:ffmpeg-devel@ffmpeg.org> >> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> >> To unsubscribe, visit link above, or email >> ffmpeg-devel-request@ffmpeg.org <mailto: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] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems 2025-03-29 15:52 ` Zhao Zhili @ 2025-03-29 17:06 ` Andreas Rheinhardt 2025-03-29 17:20 ` Sean McGovern 0 siblings, 1 reply; 7+ messages in thread From: Andreas Rheinhardt @ 2025-03-29 17:06 UTC (permalink / raw) To: ffmpeg-devel Zhao Zhili: > Ping. Fate on powerpc is still broken. > I still think that we should strip le/be in ffprobe. The documentation of bitexact requires it for the cases where decoders output native endianness and if we can't distinguish the cases of native endianness vs explicit source endianness (and we can't really), then we have to strip it in all cases. >> On Mar 21, 2025, at 12:11, Zhao Zhili <quinkblack@foxmail.com> wrote: >> >>> >>> 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 <mailto:ffmpeg-devel@ffmpeg.org> >>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel >>> >>> To unsubscribe, visit link above, or email >>> ffmpeg-devel-request@ffmpeg.org <mailto: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". _______________________________________________ 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] 7+ messages in thread
* Re: [FFmpeg-devel] [PATCH] tests: Fix enhanced-flv-hevc-hdr10 on big endian systems 2025-03-29 17:06 ` Andreas Rheinhardt @ 2025-03-29 17:20 ` Sean McGovern 0 siblings, 0 replies; 7+ messages in thread From: Sean McGovern @ 2025-03-29 17:20 UTC (permalink / raw) To: FFmpeg development discussions and patches Hi, On Sat, Mar 29, 2025, 13:07 Andreas Rheinhardt < andreas.rheinhardt@outlook.com> wrote: > Zhao Zhili: > > Ping. Fate on powerpc is still broken. > > > > I still think that we should strip le/be in ffprobe. The documentation > of bitexact requires it for the cases where decoders output native > endianness and if we can't distinguish the cases of native endianness vs > explicit source endianness (and we can't really), then we have to strip > it in all cases. > > >> On Mar 21, 2025, at 12:11, Zhao Zhili <quinkblack@foxmail.com> wrote: > >> > >>> > >>> 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 <mailto:ffmpeg-devel@ffmpeg.org> > >>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > >>> > >>> To unsubscribe, visit link above, or email > >>> ffmpeg-devel-request@ffmpeg.org <mailto: > 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". > > _______________________________________________ > 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". > As an aside, my G5 has been unavailable ever since a package was pushed to Debian sid that broke boot. The machine is headless and nothing I have tried so far will resurrect it. -- 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] 7+ messages in thread
end of thread, other threads:[~2025-03-29 17:20 UTC | newest] Thread overview: 7+ 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 2025-03-29 15:52 ` Zhao Zhili 2025-03-29 17:06 ` Andreas Rheinhardt 2025-03-29 17:20 ` Sean McGovern
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