* [FFmpeg-devel] FATE Errors @ 2022-04-30 18:54 Soft Works 2022-04-30 19:01 ` Felix LeClair 2022-04-30 19:06 ` Andreas Rheinhardt 0 siblings, 2 replies; 24+ messages in thread From: Soft Works @ 2022-04-30 18:54 UTC (permalink / raw) To: FFmpeg development discussions and patches Hi, is it a known issue that the current head of the master branch has FATE errors? I get the same locally as well as on the automated GitHub build. One is this: --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 14:23:44.330424058 +0000 +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 +0000 @@ -1,4 +1,4 @@ -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2-mpeg2-422.err for details. make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 Is anybody seeing the same? Thanks, sw _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 18:54 [FFmpeg-devel] FATE Errors Soft Works @ 2022-04-30 19:01 ` Felix LeClair 2022-04-30 19:07 ` Soft Works 2022-04-30 19:06 ` Andreas Rheinhardt 1 sibling, 1 reply; 24+ messages in thread From: Felix LeClair @ 2022-04-30 19:01 UTC (permalink / raw) To: FFmpeg development discussions and patches I’ve been getting the same locally, assumed an issue had already been opened. -Fclc > On Apr 30, 2022, at 14:54, Soft Works <softworkz@hotmail.com> wrote: > > Hi, > > is it a known issue that the current head of the master branch has > FATE errors? > > I get the same locally as well as on the automated GitHub build. > One is this: > > --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 14:23:44.330424058 +0000 > +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 +0000 > @@ -1,4 +1,4 @@ > -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2-mpeg2-422.err for details. > make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > > > Is anybody seeing the same? > > Thanks, > sw > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:01 ` Felix LeClair @ 2022-04-30 19:07 ` Soft Works 0 siblings, 0 replies; 24+ messages in thread From: Soft Works @ 2022-04-30 19:07 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Felix LeClair > Sent: Saturday, April 30, 2022 9:01 PM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > I’ve been getting the same locally, assumed an issue had already been > opened. The weird thing is that the patchwork builds don't show any error: https://patchwork.ffmpeg.org/project/ffmpeg/list/ ..which would be expected for a specific commit that might fix this, but none of the commits is showing such error. That's why I was wondering whether it's something on my side (2 sides). Thanks for letting me know that you're seeing the same! sw _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 18:54 [FFmpeg-devel] FATE Errors Soft Works 2022-04-30 19:01 ` Felix LeClair @ 2022-04-30 19:06 ` Andreas Rheinhardt 2022-04-30 19:09 ` James Almer 2022-04-30 19:12 ` Soft Works 1 sibling, 2 replies; 24+ messages in thread From: Andreas Rheinhardt @ 2022-04-30 19:06 UTC (permalink / raw) To: ffmpeg-devel Soft Works: > Hi, > > is it a known issue that the current head of the master branch has > FATE errors? > > I get the same locally as well as on the automated GitHub build. > One is this: > > --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 14:23:44.330424058 +0000 > +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 +0000 > @@ -1,4 +1,4 @@ > -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2-mpeg2-422.err for details. > make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > > > Is anybody seeing the same? > > Thanks, > sw http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is fine for me locally. I recently made changes to FATE (namely the test requirements, vcodec.mak (where the vsynth-tests reside) among the files affected), so I am interested in whether the failing tests are concentrated on the files recently changed by me (it would obviously not haved pushed them if I knew them to cause issues; also patchwork was fine). Are these issues reproducible? Can you bisect them? - Andreas _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:06 ` Andreas Rheinhardt @ 2022-04-30 19:09 ` James Almer 2022-04-30 19:19 ` Soft Works 2022-04-30 20:02 ` Andreas Rheinhardt 2022-04-30 19:12 ` Soft Works 1 sibling, 2 replies; 24+ messages in thread From: James Almer @ 2022-04-30 19:09 UTC (permalink / raw) To: ffmpeg-devel On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > Soft Works: >> Hi, >> >> is it a known issue that the current head of the master branch has >> FATE errors? >> >> I get the same locally as well as on the automated GitHub build. >> One is this: >> >> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 14:23:44.330424058 +0000 >> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 +0000 >> @@ -1,4 +1,4 @@ >> -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video >> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >> -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo >> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 >> +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2-422.mpeg2video >> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >> +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo >> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 >> Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2-mpeg2-422.err for details. >> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 >> >> >> Is anybody seeing the same? >> >> Thanks, >> sw > > http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is fine > for me locally. I recently made changes to FATE (namely the test > requirements, vcodec.mak (where the vsynth-tests reside) among the files > affected), so I am interested in whether the failing tests are > concentrated on the files recently changed by me (it would obviously not > haved pushed them if I knew them to cause issues; also patchwork was fine). > Are these issues reproducible? Can you bisect them? > > - Andreas This may be the alignment issue introduced in lavfi in https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00a680c6ebbaea58db95594d13d assuming it was not fixed. I think it only affected targets where av_cpu_max_align() returned 64. _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:09 ` James Almer @ 2022-04-30 19:19 ` Soft Works 2022-04-30 19:22 ` Felix LeClair 2022-04-30 20:02 ` Andreas Rheinhardt 1 sibling, 1 reply; 24+ messages in thread From: Soft Works @ 2022-04-30 19:19 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > James Almer > Sent: Saturday, April 30, 2022 9:10 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > Soft Works: > >> Hi, > >> > >> is it a known issue that the current head of the master branch has > >> FATE errors? > >> > >> I get the same locally as well as on the automated GitHub build. > >> One is this: > >> > >> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > 14:23:44.330424058 +0000 > >> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 > +0000 > >> @@ -1,4 +1,4 @@ > >> -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > >> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >> -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > >> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > >> +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > >> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >> +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > >> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > >> Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2- > mpeg2-422.err for details. > >> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > >> > >> > >> Is anybody seeing the same? > >> > >> Thanks, > >> sw > > > > http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is > fine > > for me locally. I recently made changes to FATE (namely the test > > requirements, vcodec.mak (where the vsynth-tests reside) among the > files > > affected), so I am interested in whether the failing tests are > > concentrated on the files recently changed by me (it would obviously > not > > haved pushed them if I knew them to cause issues; also patchwork was > fine). > > Are these issues reproducible? Can you bisect them? > > > > - Andreas > > This may be the alignment issue introduced in lavfi in > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00a > 680c6ebbaea58db95594d13d > assuming it was not fixed. > I think it only affected targets where av_cpu_max_align() returned 64. My local build is run on Ubuntu in Hyper-V VM on Windows. The GitHub CI build runs on Azure DevOps which is also an Ubuntu VM (not sure about the host). AFAIC, Patchwork runs in a Docker container only. Not sure whether that could be the difference.. softworkz _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:19 ` Soft Works @ 2022-04-30 19:22 ` Felix LeClair 2022-04-30 19:25 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Felix LeClair @ 2022-04-30 19:22 UTC (permalink / raw) To: FFmpeg development discussions and patches Both times I’ve encountered were on PopOS (think Debian/Ubuntu meets arch like timing release) Currently pulling to a macOS machine and seeing if it can be replicated there. -FCLC > On Apr 30, 2022, at 15:19, Soft Works <softworkz@hotmail.com> wrote: > > > >> -----Original Message----- >> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >> James Almer >> Sent: Saturday, April 30, 2022 9:10 PM >> To: ffmpeg-devel@ffmpeg.org >> Subject: Re: [FFmpeg-devel] FATE Errors >> >> >> >>> On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: >>> Soft Works: >>>> Hi, >>>> >>>> is it a known issue that the current head of the master branch has >>>> FATE errors? >>>> >>>> I get the same locally as well as on the automated GitHub build. >>>> One is this: >>>> >>>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 >> 14:23:44.330424058 +0000 >>>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 >> +0000 >>>> @@ -1,4 +1,4 @@ >>>> -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2- >> 422.mpeg2video >>>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>>> -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2- >> 422.out.rawvideo >>>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 >>>> +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2- >> 422.mpeg2video >>>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>>> +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2- >> 422.out.rawvideo >>>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 >>>> Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2- >> mpeg2-422.err for details. >>>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 >>>> >>>> >>>> Is anybody seeing the same? >>>> >>>> Thanks, >>>> sw >>> >>> http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is >> fine >>> for me locally. I recently made changes to FATE (namely the test >>> requirements, vcodec.mak (where the vsynth-tests reside) among the >> files >>> affected), so I am interested in whether the failing tests are >>> concentrated on the files recently changed by me (it would obviously >> not >>> haved pushed them if I knew them to cause issues; also patchwork was >> fine). >>> Are these issues reproducible? Can you bisect them? >>> >>> - Andreas >> >> This may be the alignment issue introduced in lavfi in >> https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00a >> 680c6ebbaea58db95594d13d >> assuming it was not fixed. >> I think it only affected targets where av_cpu_max_align() returned 64. > > My local build is run on Ubuntu in Hyper-V VM on Windows. The GitHub > CI build runs on Azure DevOps which is also an Ubuntu VM (not sure > about the host). > > AFAIC, Patchwork runs in a Docker container only. Not sure whether > that could be the difference.. > > softworkz > > > > > > > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:22 ` Felix LeClair @ 2022-04-30 19:25 ` Soft Works 2022-04-30 19:31 ` Felix LeClair 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-04-30 19:25 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Felix LeClair > Sent: Saturday, April 30, 2022 9:22 PM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > Both times I’ve encountered were on PopOS (think Debian/Ubuntu meets > arch like timing release) VM or bare metal installation? PS: The people here don't like top-posting Thanks, softworkz _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:25 ` Soft Works @ 2022-04-30 19:31 ` Felix LeClair 2022-04-30 19:33 ` James Almer 0 siblings, 1 reply; 24+ messages in thread From: Felix LeClair @ 2022-04-30 19:31 UTC (permalink / raw) To: FFmpeg development discussions and patches > On Apr 30, 2022, at 15:25, Soft Works <softworkz@hotmail.com> wrote: > > > >> -----Original Message----- >> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >> Felix LeClair >> Sent: Saturday, April 30, 2022 9:22 PM >> To: FFmpeg development discussions and patches <ffmpeg- >> devel@ffmpeg.org> >> Subject: Re: [FFmpeg-devel] FATE Errors >> >> Both times I’ve encountered were on PopOS (think Debian/Ubuntu meets >> arch like timing release) > > VM or bare metal installation? > > PS: The people here don't like top-posting > > Thanks, > softworkz > > _______________________________________________ > 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". It’s a bare metal installation, intel x86 12th gen, ecores disabled, AVX512 enabled (instructions include ice lake + FP16), kernel 5.18rc3. I don’t believe it matters in this case, but one was with a bare config, another with a fair few external libs enabled. Both resulted in the same. Re top posting: My bad! Currently on mobile, using an ssh app to get to my Mac. _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:31 ` Felix LeClair @ 2022-04-30 19:33 ` James Almer 2022-04-30 19:39 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: James Almer @ 2022-04-30 19:33 UTC (permalink / raw) To: ffmpeg-devel On 4/30/2022 4:31 PM, Felix LeClair wrote: > > >> On Apr 30, 2022, at 15:25, Soft Works <softworkz@hotmail.com> wrote: >> >> >> >>> -----Original Message----- >>> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of >>> Felix LeClair >>> Sent: Saturday, April 30, 2022 9:22 PM >>> To: FFmpeg development discussions and patches <ffmpeg- >>> devel@ffmpeg.org> >>> Subject: Re: [FFmpeg-devel] FATE Errors >>> >>> Both times I’ve encountered were on PopOS (think Debian/Ubuntu meets >>> arch like timing release) >> >> VM or bare metal installation? >> >> PS: The people here don't like top-posting >> >> Thanks, >> softworkz >> >> _______________________________________________ >> 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". > > It’s a bare metal installation, intel x86 12th gen, ecores disabled, AVX512 enabled (instructions include ice lake + FP16), kernel 5.18rc3. Then i was right. AVX512 support means av_cpu_max_align() returns 64. > > I don’t believe it matters in this case, but one was with a bare config, another with a fair few external libs enabled. > > Both resulted in the same. > > > Re top posting: My bad! Currently on mobile, using an ssh app to get to my Mac. > > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:33 ` James Almer @ 2022-04-30 19:39 ` Soft Works 0 siblings, 0 replies; 24+ messages in thread From: Soft Works @ 2022-04-30 19:39 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > James Almer > Sent: Saturday, April 30, 2022 9:33 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On 4/30/2022 4:31 PM, Felix LeClair wrote: > > > > > >> On Apr 30, 2022, at 15:25, Soft Works <softworkz@hotmail.com> > wrote: > >> > >> > >> > >>> -----Original Message----- > >>> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > >>> Felix LeClair > >>> Sent: Saturday, April 30, 2022 9:22 PM > >>> To: FFmpeg development discussions and patches <ffmpeg- > >>> devel@ffmpeg.org> > >>> Subject: Re: [FFmpeg-devel] FATE Errors > >>> > >>> Both times I’ve encountered were on PopOS (think Debian/Ubuntu > meets > >>> arch like timing release) > >> > >> VM or bare metal installation? > >> > >> PS: The people here don't like top-posting > >> > >> Thanks, > >> softworkz > >> > >> _______________________________________________ > >> 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". > > > > It’s a bare metal installation, intel x86 12th gen, ecores > disabled, AVX512 enabled (instructions include ice lake + FP16), > kernel 5.18rc3. > > Then i was right. AVX512 support means av_cpu_max_align() returns 64. Makes sense. Mine is 11 gen and I looked at the CI build history (https://dev.azure.com/githubsync/ffmpeg/_build?definitionId=9&_a=summary) which shows some runs have passed and some failed during the recent past - probably depending on the host machine type that gets the job. softworkz _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:09 ` James Almer 2022-04-30 19:19 ` Soft Works @ 2022-04-30 20:02 ` Andreas Rheinhardt 2022-05-27 21:10 ` Soft Works 1 sibling, 1 reply; 24+ messages in thread From: Andreas Rheinhardt @ 2022-04-30 20:02 UTC (permalink / raw) To: ffmpeg-devel James Almer: > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: >> Soft Works: >>> Hi, >>> >>> is it a known issue that the current head of the master branch has >>> FATE errors? >>> >>> I get the same locally as well as on the automated GitHub build. >>> One is this: >>> >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 >>> 14:23:44.330424058 +0000 >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 >>> +0000 >>> @@ -1,4 +1,4 @@ >>> -b2fa9b73c3547191ecc01b8163abd4e5 >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>> -704f6a96f93c2409219bd48b74169041 >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 >>> +8f6d565723ccf879ab2b5aa910b7ce21 >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video >>> +0797fddea4835687dedddebbbe98fa8f >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 >>> Test vsynth2-mpeg2-422 failed. Look at >>> tests/data/fate/vsynth2-mpeg2-422.err for details. >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 >>> >>> >>> Is anybody seeing the same? >>> >>> Thanks, >>> sw >> >> http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is fine >> for me locally. I recently made changes to FATE (namely the test >> requirements, vcodec.mak (where the vsynth-tests reside) among the files >> affected), so I am interested in whether the failing tests are >> concentrated on the files recently changed by me (it would obviously not >> haved pushed them if I knew them to cause issues; also patchwork was >> fine). >> Are these issues reproducible? Can you bisect them? >> >> - Andreas > > This may be the alignment issue introduced in lavfi in > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00a680c6ebbaea58db95594d13d > assuming it was not fixed. > I think it only affected targets where av_cpu_max_align() returned 64. You are completely right: Just making av_cpu_max_align return 64 allows to reproduce the issue. And it has nothing to do with my recent FATE patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be the only contender for changes). - Andreas _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 20:02 ` Andreas Rheinhardt @ 2022-05-27 21:10 ` Soft Works 2022-05-28 7:25 ` Paul B Mahol 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-05-27 21:10 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Andreas Rheinhardt > Sent: Saturday, April 30, 2022 10:02 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] FATE Errors > > James Almer: > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > >> Soft Works: > >>> Hi, > >>> > >>> is it a known issue that the current head of the master branch > has > >>> FATE errors? > >>> > >>> I get the same locally as well as on the automated GitHub build. > >>> One is this: > >>> > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > >>> 14:23:44.330424058 +0000 > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > 14:38:41.071678201 > >>> +0000 > >>> @@ -1,4 +1,4 @@ > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >>> -704f6a96f93c2409219bd48b74169041 > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > >>> +0797fddea4835687dedddebbbe98fa8f > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > >>> Test vsynth2-mpeg2-422 failed. Look at > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > >>> > >>> > >>> Is anybody seeing the same? > >>> > >>> Thanks, > >>> sw > >> > >> http://fate.ffmpeg.org/ doesn't show recent regressions and FATE > is fine > >> for me locally. I recently made changes to FATE (namely the test > >> requirements, vcodec.mak (where the vsynth-tests reside) among the > files > >> affected), so I am interested in whether the failing tests are > >> concentrated on the files recently changed by me (it would > obviously not > >> haved pushed them if I knew them to cause issues; also patchwork > was > >> fine). > >> Are these issues reproducible? Can you bisect them? > >> > >> - Andreas > > > > This may be the alignment issue introduced in lavfi in > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > a680c6ebbaea58db95594d13d > > assuming it was not fixed. > > I think it only affected targets where av_cpu_max_align() returned > 64. > > You are completely right: Just making av_cpu_max_align return 64 > allows > to reproduce the issue. And it has nothing to do with my recent FATE > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be the only > contender for changes). > I found that the FATE errors can be avoided by using ./configure --disable-avx512 Yet, I hope it will be fixed at some time.. softworkz _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-27 21:10 ` Soft Works @ 2022-05-28 7:25 ` Paul B Mahol 2022-05-28 7:26 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Paul B Mahol @ 2022-05-28 7:25 UTC (permalink / raw) To: FFmpeg development discussions and patches On Fri, May 27, 2022 at 11:11 PM Soft Works <softworkz@hotmail.com> wrote: > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Andreas Rheinhardt > > Sent: Saturday, April 30, 2022 10:02 PM > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > James Almer: > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > >> Soft Works: > > >>> Hi, > > >>> > > >>> is it a known issue that the current head of the master branch > > has > > >>> FATE errors? > > >>> > > >>> I get the same locally as well as on the automated GitHub build. > > >>> One is this: > > >>> > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > >>> 14:23:44.330424058 +0000 > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > 14:38:41.071678201 > > >>> +0000 > > >>> @@ -1,4 +1,4 @@ > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > >>> -704f6a96f93c2409219bd48b74169041 > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > >>> +0797fddea4835687dedddebbbe98fa8f > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > > >>> Test vsynth2-mpeg2-422 failed. Look at > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > > >>> > > >>> > > >>> Is anybody seeing the same? > > >>> > > >>> Thanks, > > >>> sw > > >> > > >> http://fate.ffmpeg.org/ doesn't show recent regressions and FATE > > is fine > > >> for me locally. I recently made changes to FATE (namely the test > > >> requirements, vcodec.mak (where the vsynth-tests reside) among the > > files > > >> affected), so I am interested in whether the failing tests are > > >> concentrated on the files recently changed by me (it would > > obviously not > > >> haved pushed them if I knew them to cause issues; also patchwork > > was > > >> fine). > > >> Are these issues reproducible? Can you bisect them? > > >> > > >> - Andreas > > > > > > This may be the alignment issue introduced in lavfi in > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > a680c6ebbaea58db95594d13d > > > assuming it was not fixed. > > > I think it only affected targets where av_cpu_max_align() returned > > 64. > > > > You are completely right: Just making av_cpu_max_align return 64 > > allows > > to reproduce the issue. And it has nothing to do with my recent FATE > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be the only > > contender for changes). > > > > > I found that the FATE errors can be avoided by using > > ./configure --disable-avx512 > > Yet, I hope it will be fixed at some time.. > I hope you will post a fix for it soon. > > > softworkz > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 7:25 ` Paul B Mahol @ 2022-05-28 7:26 ` Soft Works 2022-05-28 7:34 ` Paul B Mahol 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-05-28 7:26 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Paul B Mahol > Sent: Saturday, May 28, 2022 9:26 AM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > On Fri, May 27, 2022 at 11:11 PM Soft Works <softworkz@hotmail.com> > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Andreas Rheinhardt > > > Sent: Saturday, April 30, 2022 10:02 PM > > > To: ffmpeg-devel@ffmpeg.org > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > James Almer: > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > >> Soft Works: > > > >>> Hi, > > > >>> > > > >>> is it a known issue that the current head of the master > branch > > > has > > > >>> FATE errors? > > > >>> > > > >>> I get the same locally as well as on the automated GitHub > build. > > > >>> One is this: > > > >>> > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > >>> 14:23:44.330424058 +0000 > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > 14:38:41.071678201 > > > >>> +0000 > > > >>> @@ -1,4 +1,4 @@ > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > >>> -704f6a96f93c2409219bd48b74169041 > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error > 1 > > > >>> > > > >>> > > > >>> Is anybody seeing the same? > > > >>> > > > >>> Thanks, > > > >>> sw > > > >> > > > >> http://fate.ffmpeg.org/ doesn't show recent regressions and > FATE > > > is fine > > > >> for me locally. I recently made changes to FATE (namely the > test > > > >> requirements, vcodec.mak (where the vsynth-tests reside) among > the > > > files > > > >> affected), so I am interested in whether the failing tests are > > > >> concentrated on the files recently changed by me (it would > > > obviously not > > > >> haved pushed them if I knew them to cause issues; also > patchwork > > > was > > > >> fine). > > > >> Are these issues reproducible? Can you bisect them? > > > >> > > > >> - Andreas > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > a680c6ebbaea58db95594d13d > > > > assuming it was not fixed. > > > > I think it only affected targets where av_cpu_max_align() > returned > > > 64. > > > > > > You are completely right: Just making av_cpu_max_align return 64 > > > allows > > > to reproduce the issue. And it has nothing to do with my recent > FATE > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be the > only > > > contender for changes). > > > > > > > > > I found that the FATE errors can be avoided by using > > > > ./configure --disable-avx512 > > > > Yet, I hope it will be fixed at some time.. > > > > I hope you will post a fix for it soon. That depends on whether we want alignment values > 32bit..? _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 7:26 ` Soft Works @ 2022-05-28 7:34 ` Paul B Mahol 2022-05-28 7:47 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Paul B Mahol @ 2022-05-28 7:34 UTC (permalink / raw) To: FFmpeg development discussions and patches On Sat, May 28, 2022 at 9:26 AM Soft Works <softworkz@hotmail.com> wrote: > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Paul B Mahol > > Sent: Saturday, May 28, 2022 9:26 AM > > To: FFmpeg development discussions and patches <ffmpeg- > > devel@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works <softworkz@hotmail.com> > > wrote: > > > > > > > > > > > > -----Original Message----- > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > > Andreas Rheinhardt > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > To: ffmpeg-devel@ffmpeg.org > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > James Almer: > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > >> Soft Works: > > > > >>> Hi, > > > > >>> > > > > >>> is it a known issue that the current head of the master > > branch > > > > has > > > > >>> FATE errors? > > > > >>> > > > > >>> I get the same locally as well as on the automated GitHub > > build. > > > > >>> One is this: > > > > >>> > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > > >>> 14:23:44.330424058 +0000 > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > 14:38:41.071678201 > > > > >>> +0000 > > > > >>> @@ -1,4 +1,4 @@ > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error > > 1 > > > > >>> > > > > >>> > > > > >>> Is anybody seeing the same? > > > > >>> > > > > >>> Thanks, > > > > >>> sw > > > > >> > > > > >> http://fate.ffmpeg.org/ doesn't show recent regressions and > > FATE > > > > is fine > > > > >> for me locally. I recently made changes to FATE (namely the > > test > > > > >> requirements, vcodec.mak (where the vsynth-tests reside) among > > the > > > > files > > > > >> affected), so I am interested in whether the failing tests are > > > > >> concentrated on the files recently changed by me (it would > > > > obviously not > > > > >> haved pushed them if I knew them to cause issues; also > > patchwork > > > > was > > > > >> fine). > > > > >> Are these issues reproducible? Can you bisect them? > > > > >> > > > > >> - Andreas > > > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > a680c6ebbaea58db95594d13d > > > > > assuming it was not fixed. > > > > > I think it only affected targets where av_cpu_max_align() > > returned > > > > 64. > > > > > > > > You are completely right: Just making av_cpu_max_align return 64 > > > > allows > > > > to reproduce the issue. And it has nothing to do with my recent > > FATE > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be the > > only > > > > contender for changes). > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > ./configure --disable-avx512 > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > I hope you will post a fix for it soon. > > That depends on whether we want alignment values > 32bit..? > Nope, its about mpeg video encoder in lavc that does funny things from old days when linesize alignment was hardcoded. > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 7:34 ` Paul B Mahol @ 2022-05-28 7:47 ` Soft Works 2022-05-28 8:04 ` Paul B Mahol 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-05-28 7:47 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Paul B Mahol > Sent: Saturday, May 28, 2022 9:35 AM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > On Sat, May 28, 2022 at 9:26 AM Soft Works <softworkz@hotmail.com> > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Paul B Mahol > > > Sent: Saturday, May 28, 2022 9:26 AM > > > To: FFmpeg development discussions and patches <ffmpeg- > > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > <softworkz@hotmail.com> > > > wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > Behalf Of > > > > > Andreas Rheinhardt > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > >> Soft Works: > > > > > >>> Hi, > > > > > >>> > > > > > >>> is it a known issue that the current head of the master > > > branch > > > > > has > > > > > >>> FATE errors? > > > > > >>> > > > > > >>> I get the same locally as well as on the automated GitHub > > > build. > > > > > >>> One is this: > > > > > >>> > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > > > >>> 14:23:44.330424058 +0000 > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > 14:38:41.071678201 > > > > > >>> +0000 > > > > > >>> @@ -1,4 +1,4 @@ > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ > 7603200 > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ > 7603200 > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] > Error > > > 1 > > > > > >>> > > > > > >>> > > > > > >>> Is anybody seeing the same? > > > > > >>> > > > > > >>> Thanks, > > > > > >>> sw > > > > > >> > > > > > >> http://fate.ffmpeg.org/ doesn't show recent regressions > and > > > FATE > > > > > is fine > > > > > >> for me locally. I recently made changes to FATE (namely > the > > > test > > > > > >> requirements, vcodec.mak (where the vsynth-tests reside) > among > > > the > > > > > files > > > > > >> affected), so I am interested in whether the failing tests > are > > > > > >> concentrated on the files recently changed by me (it would > > > > > obviously not > > > > > >> haved pushed them if I knew them to cause issues; also > > > patchwork > > > > > was > > > > > >> fine). > > > > > >> Are these issues reproducible? Can you bisect them? > > > > > >> > > > > > >> - Andreas > > > > > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > a680c6ebbaea58db95594d13d > > > > > > assuming it was not fixed. > > > > > > I think it only affected targets where av_cpu_max_align() > > > returned > > > > > 64. > > > > > > > > > > You are completely right: Just making av_cpu_max_align return > 64 > > > > > allows > > > > > to reproduce the issue. And it has nothing to do with my > recent > > > FATE > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be > the > > > only > > > > > contender for changes). > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > ./configure --disable-avx512 > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > I hope you will post a fix for it soon. > > > > That depends on whether we want alignment values > 32bit..? > > > > Nope, its about mpeg video encoder in lavc that does funny things > from old > days when linesize alignment was hardcoded. Adding this align = FFMIN(align, 32); to ff_default_get_video_buffer2() resolves the issue. That's why I had written: > That depends on whether we want alignment values > 32bit..? Thanks, sw _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 7:47 ` Soft Works @ 2022-05-28 8:04 ` Paul B Mahol 2022-05-28 8:12 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Paul B Mahol @ 2022-05-28 8:04 UTC (permalink / raw) To: FFmpeg development discussions and patches On Sat, May 28, 2022 at 9:47 AM Soft Works <softworkz@hotmail.com> wrote: > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Paul B Mahol > > Sent: Saturday, May 28, 2022 9:35 AM > > To: FFmpeg development discussions and patches <ffmpeg- > > devel@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works <softworkz@hotmail.com> > > wrote: > > > > > > > > > > > > -----Original Message----- > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > > Paul B Mahol > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > devel@ffmpeg.org> > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > <softworkz@hotmail.com> > > > > wrote: > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > Behalf Of > > > > > > Andreas Rheinhardt > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > >> Soft Works: > > > > > > >>> Hi, > > > > > > >>> > > > > > > >>> is it a known issue that the current head of the master > > > > branch > > > > > > has > > > > > > >>> FATE errors? > > > > > > >>> > > > > > > >>> I get the same locally as well as on the automated GitHub > > > > build. > > > > > > >>> One is this: > > > > > > >>> > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > > 14:38:41.071678201 > > > > > > >>> +0000 > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ > > 7603200 > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ > > 7603200 > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] > > Error > > > > 1 > > > > > > >>> > > > > > > >>> > > > > > > >>> Is anybody seeing the same? > > > > > > >>> > > > > > > >>> Thanks, > > > > > > >>> sw > > > > > > >> > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent regressions > > and > > > > FATE > > > > > > is fine > > > > > > >> for me locally. I recently made changes to FATE (namely > > the > > > > test > > > > > > >> requirements, vcodec.mak (where the vsynth-tests reside) > > among > > > > the > > > > > > files > > > > > > >> affected), so I am interested in whether the failing tests > > are > > > > > > >> concentrated on the files recently changed by me (it would > > > > > > obviously not > > > > > > >> haved pushed them if I knew them to cause issues; also > > > > patchwork > > > > > > was > > > > > > >> fine). > > > > > > >> Are these issues reproducible? Can you bisect them? > > > > > > >> > > > > > > >> - Andreas > > > > > > > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > a680c6ebbaea58db95594d13d > > > > > > > assuming it was not fixed. > > > > > > > I think it only affected targets where av_cpu_max_align() > > > > returned > > > > > > 64. > > > > > > > > > > > > You are completely right: Just making av_cpu_max_align return > > 64 > > > > > > allows > > > > > > to reproduce the issue. And it has nothing to do with my > > recent > > > > FATE > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would be > > the > > > > only > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > Nope, its about mpeg video encoder in lavc that does funny things > > from old > > days when linesize alignment was hardcoded. > > Adding this > > align = FFMIN(align, 32); > > to ff_default_get_video_buffer2() resolves the issue. > Maybe it resolves it for you but that does not mean in any way that that is valid or correct solution. > > That's why I had written: > > > That depends on whether we want alignment values > 32bit..? > 32bit is 4 integers. so I failed to follow whatever you tried to communicate. > > > Thanks, > sw > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 8:04 ` Paul B Mahol @ 2022-05-28 8:12 ` Soft Works 2022-05-28 8:33 ` Paul B Mahol 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-05-28 8:12 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Paul B Mahol > Sent: Saturday, May 28, 2022 10:04 AM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > On Sat, May 28, 2022 at 9:47 AM Soft Works <softworkz@hotmail.com> > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Paul B Mahol > > > Sent: Saturday, May 28, 2022 9:35 AM > > > To: FFmpeg development discussions and patches <ffmpeg- > > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works > <softworkz@hotmail.com> > > > wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > Behalf Of > > > > > Paul B Mahol > > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > devel@ffmpeg.org> > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > > <softworkz@hotmail.com> > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > > Behalf Of > > > > > > > Andreas Rheinhardt > > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > > >> Soft Works: > > > > > > > >>> Hi, > > > > > > > >>> > > > > > > > >>> is it a known issue that the current head of the > master > > > > > branch > > > > > > > has > > > > > > > >>> FATE errors? > > > > > > > >>> > > > > > > > >>> I get the same locally as well as on the automated > GitHub > > > > > build. > > > > > > > >>> One is this: > > > > > > > >>> > > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > > > 14:38:41.071678201 > > > > > > > >>> +0000 > > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ > > > 7603200 > > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ > > > 7603200 > > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2- > 422] > > > Error > > > > > 1 > > > > > > > >>> > > > > > > > >>> > > > > > > > >>> Is anybody seeing the same? > > > > > > > >>> > > > > > > > >>> Thanks, > > > > > > > >>> sw > > > > > > > >> > > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent > regressions > > > and > > > > > FATE > > > > > > > is fine > > > > > > > >> for me locally. I recently made changes to FATE > (namely > > > the > > > > > test > > > > > > > >> requirements, vcodec.mak (where the vsynth-tests > reside) > > > among > > > > > the > > > > > > > files > > > > > > > >> affected), so I am interested in whether the failing > tests > > > are > > > > > > > >> concentrated on the files recently changed by me (it > would > > > > > > > obviously not > > > > > > > >> haved pushed them if I knew them to cause issues; also > > > > > patchwork > > > > > > > was > > > > > > > >> fine). > > > > > > > >> Are these issues reproducible? Can you bisect them? > > > > > > > >> > > > > > > > >> - Andreas > > > > > > > > > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > > a680c6ebbaea58db95594d13d > > > > > > > > assuming it was not fixed. > > > > > > > > I think it only affected targets where > av_cpu_max_align() > > > > > returned > > > > > > > 64. > > > > > > > > > > > > > > You are completely right: Just making av_cpu_max_align > return > > > 64 > > > > > > > allows > > > > > > > to reproduce the issue. And it has nothing to do with my > > > recent > > > > > FATE > > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would > be > > > the > > > > > only > > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > Nope, its about mpeg video encoder in lavc that does funny things > > > from old > > > days when linesize alignment was hardcoded. > > > > Adding this > > > > align = FFMIN(align, 32); > > > > to ff_default_get_video_buffer2() resolves the issue. > > > > Maybe it resolves it for you but that does not mean in any way that > that is > valid or correct solution. > > > > > > That's why I had written: > > > > > That depends on whether we want alignment values > 32bit..? > > > > 32bit is 4 integers. so I failed to follow whatever you tried to > communicate. Sorry for being unclear. What I meant to ask is this: In cases where av_cpu_max_align() returns something > 32 - for example 64, do we have to use that max-align value of 64 for alignment in video.c and framepool,c or could we just limit this to 32? (hence the FFMIN). Limiting the alignment to 32 appears to fix the issue, meaning that the three fate tests mentioned above are producing the expected ref output and don't fail anymore. softworkz _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 8:12 ` Soft Works @ 2022-05-28 8:33 ` Paul B Mahol 2022-05-28 8:41 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Paul B Mahol @ 2022-05-28 8:33 UTC (permalink / raw) To: FFmpeg development discussions and patches On Sat, May 28, 2022 at 10:12 AM Soft Works <softworkz@hotmail.com> wrote: > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Paul B Mahol > > Sent: Saturday, May 28, 2022 10:04 AM > > To: FFmpeg development discussions and patches <ffmpeg- > > devel@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On Sat, May 28, 2022 at 9:47 AM Soft Works <softworkz@hotmail.com> > > wrote: > > > > > > > > > > > > -----Original Message----- > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > > Paul B Mahol > > > > Sent: Saturday, May 28, 2022 9:35 AM > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > devel@ffmpeg.org> > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works > > <softworkz@hotmail.com> > > > > wrote: > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > Behalf Of > > > > > > Paul B Mahol > > > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > > devel@ffmpeg.org> > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > > > <softworkz@hotmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > > > Behalf Of > > > > > > > > Andreas Rheinhardt > > > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > > > >> Soft Works: > > > > > > > > >>> Hi, > > > > > > > > >>> > > > > > > > > >>> is it a known issue that the current head of the > > master > > > > > > branch > > > > > > > > has > > > > > > > > >>> FATE errors? > > > > > > > > >>> > > > > > > > > >>> I get the same locally as well as on the automated > > GitHub > > > > > > build. > > > > > > > > >>> One is this: > > > > > > > > >>> > > > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > > > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > > > > 14:38:41.071678201 > > > > > > > > >>> +0000 > > > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ > > > > 7603200 > > > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ > > > > 7603200 > > > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for details. > > > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2-mpeg2- > > 422] > > > > Error > > > > > > 1 > > > > > > > > >>> > > > > > > > > >>> > > > > > > > > >>> Is anybody seeing the same? > > > > > > > > >>> > > > > > > > > >>> Thanks, > > > > > > > > >>> sw > > > > > > > > >> > > > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent > > regressions > > > > and > > > > > > FATE > > > > > > > > is fine > > > > > > > > >> for me locally. I recently made changes to FATE > > (namely > > > > the > > > > > > test > > > > > > > > >> requirements, vcodec.mak (where the vsynth-tests > > reside) > > > > among > > > > > > the > > > > > > > > files > > > > > > > > >> affected), so I am interested in whether the failing > > tests > > > > are > > > > > > > > >> concentrated on the files recently changed by me (it > > would > > > > > > > > obviously not > > > > > > > > >> haved pushed them if I knew them to cause issues; also > > > > > > patchwork > > > > > > > > was > > > > > > > > >> fine). > > > > > > > > >> Are these issues reproducible? Can you bisect them? > > > > > > > > >> > > > > > > > > >> - Andreas > > > > > > > > > > > > > > > > > > This may be the alignment issue introduced in lavfi in > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > > > a680c6ebbaea58db95594d13d > > > > > > > > > assuming it was not fixed. > > > > > > > > > I think it only affected targets where > > av_cpu_max_align() > > > > > > returned > > > > > > > > 64. > > > > > > > > > > > > > > > > You are completely right: Just making av_cpu_max_align > > return > > > > 64 > > > > > > > > allows > > > > > > > > to reproduce the issue. And it has nothing to do with my > > > > recent > > > > > > FATE > > > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 would > > be > > > > the > > > > > > only > > > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > > > > Nope, its about mpeg video encoder in lavc that does funny things > > > > from old > > > > days when linesize alignment was hardcoded. > > > > > > Adding this > > > > > > align = FFMIN(align, 32); > > > > > > to ff_default_get_video_buffer2() resolves the issue. > > > > > > > Maybe it resolves it for you but that does not mean in any way that > > that is > > valid or correct solution. > > > > > > > > > > That's why I had written: > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > 32bit is 4 integers. so I failed to follow whatever you tried to > > communicate. > > Sorry for being unclear. What I meant to ask is this: > > In cases where av_cpu_max_align() returns something > 32 - for > example 64, do we have to use that max-align value of 64 for alignment > in video.c and framepool,c or could we just limit this to 32? > (hence the FFMIN). > > Limiting the alignment to 32 appears to fix the issue, meaning that > the three fate tests mentioned above are producing the expected ref > output and don't fail anymore. > That is hack. > > softworkz > > > > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 8:33 ` Paul B Mahol @ 2022-05-28 8:41 ` Soft Works 2022-05-28 8:50 ` Paul B Mahol 0 siblings, 1 reply; 24+ messages in thread From: Soft Works @ 2022-05-28 8:41 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Paul B Mahol > Sent: Saturday, May 28, 2022 10:33 AM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > On Sat, May 28, 2022 at 10:12 AM Soft Works <softworkz@hotmail.com> > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Paul B Mahol > > > Sent: Saturday, May 28, 2022 10:04 AM > > > To: FFmpeg development discussions and patches <ffmpeg- > > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > On Sat, May 28, 2022 at 9:47 AM Soft Works > <softworkz@hotmail.com> > > > wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > Behalf Of > > > > > Paul B Mahol > > > > > Sent: Saturday, May 28, 2022 9:35 AM > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > devel@ffmpeg.org> > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works > > > <softworkz@hotmail.com> > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > > Behalf Of > > > > > > > Paul B Mahol > > > > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > > > devel@ffmpeg.org> > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > > > > <softworkz@hotmail.com> > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> > On > > > > > Behalf Of > > > > > > > > > Andreas Rheinhardt > > > > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > > > > >> Soft Works: > > > > > > > > > >>> Hi, > > > > > > > > > >>> > > > > > > > > > >>> is it a known issue that the current head of the > > > master > > > > > > > branch > > > > > > > > > has > > > > > > > > > >>> FATE errors? > > > > > > > > > >>> > > > > > > > > > >>> I get the same locally as well as on the > automated > > > GitHub > > > > > > > build. > > > > > > > > > >>> One is this: > > > > > > > > > >>> > > > > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04- > 30 > > > > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > > > > > 14:38:41.071678201 > > > > > > > > > >>> +0000 > > > > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > > > > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: > 7603200/ > > > > > 7603200 > > > > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > > > > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: > 7603200/ > > > > > 7603200 > > > > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for > details. > > > > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2- > mpeg2- > > > 422] > > > > > Error > > > > > > > 1 > > > > > > > > > >>> > > > > > > > > > >>> > > > > > > > > > >>> Is anybody seeing the same? > > > > > > > > > >>> > > > > > > > > > >>> Thanks, > > > > > > > > > >>> sw > > > > > > > > > >> > > > > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent > > > regressions > > > > > and > > > > > > > FATE > > > > > > > > > is fine > > > > > > > > > >> for me locally. I recently made changes to FATE > > > (namely > > > > > the > > > > > > > test > > > > > > > > > >> requirements, vcodec.mak (where the vsynth-tests > > > reside) > > > > > among > > > > > > > the > > > > > > > > > files > > > > > > > > > >> affected), so I am interested in whether the > failing > > > tests > > > > > are > > > > > > > > > >> concentrated on the files recently changed by me > (it > > > would > > > > > > > > > obviously not > > > > > > > > > >> haved pushed them if I knew them to cause issues; > also > > > > > > > patchwork > > > > > > > > > was > > > > > > > > > >> fine). > > > > > > > > > >> Are these issues reproducible? Can you bisect > them? > > > > > > > > > >> > > > > > > > > > >> - Andreas > > > > > > > > > > > > > > > > > > > > This may be the alignment issue introduced in lavfi > in > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > > > > a680c6ebbaea58db95594d13d > > > > > > > > > > assuming it was not fixed. > > > > > > > > > > I think it only affected targets where > > > av_cpu_max_align() > > > > > > > returned > > > > > > > > > 64. > > > > > > > > > > > > > > > > > > You are completely right: Just making > av_cpu_max_align > > > return > > > > > 64 > > > > > > > > > allows > > > > > > > > > to reproduce the issue. And it has nothing to do with > my > > > > > recent > > > > > > > FATE > > > > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 > would > > > be > > > > > the > > > > > > > only > > > > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > > > > > > > Nope, its about mpeg video encoder in lavc that does funny > things > > > > > from old > > > > > days when linesize alignment was hardcoded. > > > > > > > > Adding this > > > > > > > > align = FFMIN(align, 32); > > > > > > > > to ff_default_get_video_buffer2() resolves the issue. > > > > > > > > > > Maybe it resolves it for you but that does not mean in any way > that > > > that is > > > valid or correct solution. > > > > > > > > > > > > > > That's why I had written: > > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > 32bit is 4 integers. so I failed to follow whatever you tried to > > > communicate. > > > > Sorry for being unclear. What I meant to ask is this: > > > > In cases where av_cpu_max_align() returns something > 32 - for > > example 64, do we have to use that max-align value of 64 for > alignment > > in video.c and framepool,c or could we just limit this to 32? > > (hence the FFMIN). > > > > Limiting the alignment to 32 appears to fix the issue, meaning that > > the three fate tests mentioned above are producing the expected ref > > output and don't fail anymore. > > > > That is hack. Only when 64bit alignment would be required or beneficial in terms of performance (e.g. AVX512 operations). Which is the reason for my question. If yes, then this would be surely a bad idea. sw _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 8:41 ` Soft Works @ 2022-05-28 8:50 ` Paul B Mahol 2022-05-28 8:57 ` Soft Works 0 siblings, 1 reply; 24+ messages in thread From: Paul B Mahol @ 2022-05-28 8:50 UTC (permalink / raw) To: FFmpeg development discussions and patches On Sat, May 28, 2022 at 10:41 AM Soft Works <softworkz@hotmail.com> wrote: > > > > -----Original Message----- > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > Paul B Mahol > > Sent: Saturday, May 28, 2022 10:33 AM > > To: FFmpeg development discussions and patches <ffmpeg- > > devel@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > On Sat, May 28, 2022 at 10:12 AM Soft Works <softworkz@hotmail.com> > > wrote: > > > > > > > > > > > > -----Original Message----- > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > > Paul B Mahol > > > > Sent: Saturday, May 28, 2022 10:04 AM > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > devel@ffmpeg.org> > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > On Sat, May 28, 2022 at 9:47 AM Soft Works > > <softworkz@hotmail.com> > > > > wrote: > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > Behalf Of > > > > > > Paul B Mahol > > > > > > Sent: Saturday, May 28, 2022 9:35 AM > > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > > devel@ffmpeg.org> > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works > > > > <softworkz@hotmail.com> > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > > > Behalf Of > > > > > > > > Paul B Mahol > > > > > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > > > > devel@ffmpeg.org> > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > > > > > <softworkz@hotmail.com> > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> > > On > > > > > > Behalf Of > > > > > > > > > > Andreas Rheinhardt > > > > > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > > > > > >> Soft Works: > > > > > > > > > > >>> Hi, > > > > > > > > > > >>> > > > > > > > > > > >>> is it a known issue that the current head of the > > > > master > > > > > > > > branch > > > > > > > > > > has > > > > > > > > > > >>> FATE errors? > > > > > > > > > > >>> > > > > > > > > > > >>> I get the same locally as well as on the > > automated > > > > GitHub > > > > > > > > build. > > > > > > > > > > >>> One is this: > > > > > > > > > > >>> > > > > > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04- > > 30 > > > > > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 > > > > > > > > > > 14:38:41.071678201 > > > > > > > > > > >>> +0000 > > > > > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2- > > 422.mpeg2video > > > > > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: > > 7603200/ > > > > > > 7603200 > > > > > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2- > > 422.mpeg2video > > > > > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.out.rawvideo > > > > > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: > > 7603200/ > > > > > > 7603200 > > > > > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for > > details. > > > > > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2- > > mpeg2- > > > > 422] > > > > > > Error > > > > > > > > 1 > > > > > > > > > > >>> > > > > > > > > > > >>> > > > > > > > > > > >>> Is anybody seeing the same? > > > > > > > > > > >>> > > > > > > > > > > >>> Thanks, > > > > > > > > > > >>> sw > > > > > > > > > > >> > > > > > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent > > > > regressions > > > > > > and > > > > > > > > FATE > > > > > > > > > > is fine > > > > > > > > > > >> for me locally. I recently made changes to FATE > > > > (namely > > > > > > the > > > > > > > > test > > > > > > > > > > >> requirements, vcodec.mak (where the vsynth-tests > > > > reside) > > > > > > among > > > > > > > > the > > > > > > > > > > files > > > > > > > > > > >> affected), so I am interested in whether the > > failing > > > > tests > > > > > > are > > > > > > > > > > >> concentrated on the files recently changed by me > > (it > > > > would > > > > > > > > > > obviously not > > > > > > > > > > >> haved pushed them if I knew them to cause issues; > > also > > > > > > > > patchwork > > > > > > > > > > was > > > > > > > > > > >> fine). > > > > > > > > > > >> Are these issues reproducible? Can you bisect > > them? > > > > > > > > > > >> > > > > > > > > > > >> - Andreas > > > > > > > > > > > > > > > > > > > > > > This may be the alignment issue introduced in lavfi > > in > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > > > > > a680c6ebbaea58db95594d13d > > > > > > > > > > > assuming it was not fixed. > > > > > > > > > > > I think it only affected targets where > > > > av_cpu_max_align() > > > > > > > > returned > > > > > > > > > > 64. > > > > > > > > > > > > > > > > > > > > You are completely right: Just making > > av_cpu_max_align > > > > return > > > > > > 64 > > > > > > > > > > allows > > > > > > > > > > to reproduce the issue. And it has nothing to do with > > my > > > > > > recent > > > > > > > > FATE > > > > > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 > > would > > > > be > > > > > > the > > > > > > > > only > > > > > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by using > > > > > > > > > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > > > > > > > > > > Nope, its about mpeg video encoder in lavc that does funny > > things > > > > > > from old > > > > > > days when linesize alignment was hardcoded. > > > > > > > > > > Adding this > > > > > > > > > > align = FFMIN(align, 32); > > > > > > > > > > to ff_default_get_video_buffer2() resolves the issue. > > > > > > > > > > > > > Maybe it resolves it for you but that does not mean in any way > > that > > > > that is > > > > valid or correct solution. > > > > > > > > > > > > > > > > > > That's why I had written: > > > > > > > > > > > That depends on whether we want alignment values > 32bit..? > > > > > > > > > > > > > 32bit is 4 integers. so I failed to follow whatever you tried to > > > > communicate. > > > > > > Sorry for being unclear. What I meant to ask is this: > > > > > > In cases where av_cpu_max_align() returns something > 32 - for > > > example 64, do we have to use that max-align value of 64 for > > alignment > > > in video.c and framepool,c or could we just limit this to 32? > > > (hence the FFMIN). > > > > > > Limiting the alignment to 32 appears to fix the issue, meaning that > > > the three fate tests mentioned above are producing the expected ref > > > output and don't fail anymore. > > > > > > > That is hack. > > Only when 64bit alignment would be required or beneficial in terms of > performance (e.g. AVX512 operations). AVX512 operations are not using 64bit alignment. > > Which is the reason for my question. If yes, then this would be > surely a bad idea. > It is not Yes/No answer here. > > sw > _______________________________________________ > 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-05-28 8:50 ` Paul B Mahol @ 2022-05-28 8:57 ` Soft Works 0 siblings, 0 replies; 24+ messages in thread From: Soft Works @ 2022-05-28 8:57 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Paul B Mahol > Sent: Saturday, May 28, 2022 10:50 AM > To: FFmpeg development discussions and patches <ffmpeg- > devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] FATE Errors > > On Sat, May 28, 2022 at 10:41 AM Soft Works <softworkz@hotmail.com> > wrote: > > > > > > > > -----Original Message----- > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > > > Paul B Mahol > > > Sent: Saturday, May 28, 2022 10:33 AM > > > To: FFmpeg development discussions and patches <ffmpeg- > > > devel@ffmpeg.org> > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > On Sat, May 28, 2022 at 10:12 AM Soft Works > <softworkz@hotmail.com> > > > wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > Behalf Of > > > > > Paul B Mahol > > > > > Sent: Saturday, May 28, 2022 10:04 AM > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > devel@ffmpeg.org> > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > On Sat, May 28, 2022 at 9:47 AM Soft Works > > > <softworkz@hotmail.com> > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On > > > Behalf Of > > > > > > > Paul B Mahol > > > > > > > Sent: Saturday, May 28, 2022 9:35 AM > > > > > > > To: FFmpeg development discussions and patches <ffmpeg- > > > > > > > devel@ffmpeg.org> > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > On Sat, May 28, 2022 at 9:26 AM Soft Works > > > > > <softworkz@hotmail.com> > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> > On > > > > > Behalf Of > > > > > > > > > Paul B Mahol > > > > > > > > > Sent: Saturday, May 28, 2022 9:26 AM > > > > > > > > > To: FFmpeg development discussions and patches > <ffmpeg- > > > > > > > > > devel@ffmpeg.org> > > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > > > On Fri, May 27, 2022 at 11:11 PM Soft Works > > > > > > > <softworkz@hotmail.com> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > From: ffmpeg-devel <ffmpeg-devel- > bounces@ffmpeg.org> > > > On > > > > > > > Behalf Of > > > > > > > > > > > Andreas Rheinhardt > > > > > > > > > > > Sent: Saturday, April 30, 2022 10:02 PM > > > > > > > > > > > To: ffmpeg-devel@ffmpeg.org > > > > > > > > > > > Subject: Re: [FFmpeg-devel] FATE Errors > > > > > > > > > > > > > > > > > > > > > > James Almer: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On 4/30/2022 4:06 PM, Andreas Rheinhardt wrote: > > > > > > > > > > > >> Soft Works: > > > > > > > > > > > >>> Hi, > > > > > > > > > > > >>> > > > > > > > > > > > >>> is it a known issue that the current head of > the > > > > > master > > > > > > > > > branch > > > > > > > > > > > has > > > > > > > > > > > >>> FATE errors? > > > > > > > > > > > >>> > > > > > > > > > > > >>> I get the same locally as well as on the > > > automated > > > > > GitHub > > > > > > > > > build. > > > > > > > > > > > >>> One is this: > > > > > > > > > > > >>> > > > > > > > > > > > >>> --- ./tests/ref/vsynth/vsynth2-mpeg2-422 > 2022-04- > > > 30 > > > > > > > > > > > >>> 14:23:44.330424058 +0000 > > > > > > > > > > > >>> +++ tests/data/fate/vsynth2-mpeg2-422 2022- > 04-30 > > > > > > > > > > > 14:38:41.071678201 > > > > > > > > > > > >>> +0000 > > > > > > > > > > > >>> @@ -1,4 +1,4 @@ > > > > > > > > > > > >>> -b2fa9b73c3547191ecc01b8163abd4e5 > > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > > > >>> -379164 tests/data/fate/vsynth2-mpeg2- > > > 422.mpeg2video > > > > > > > > > > > >>> -704f6a96f93c2409219bd48b74169041 > > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > > > > > > > > > > > >>> -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: > > > 7603200/ > > > > > > > 7603200 > > > > > > > > > > > >>> +8f6d565723ccf879ab2b5aa910b7ce21 > > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > > > > > > > > > > >>> +380544 tests/data/fate/vsynth2-mpeg2- > > > 422.mpeg2video > > > > > > > > > > > >>> +0797fddea4835687dedddebbbe98fa8f > > > > > > > > > > > >>> *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > > > > > > > > > > > >>> +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: > > > 7603200/ > > > > > > > 7603200 > > > > > > > > > > > >>> Test vsynth2-mpeg2-422 failed. Look at > > > > > > > > > > > >>> tests/data/fate/vsynth2-mpeg2-422.err for > > > details. > > > > > > > > > > > >>> make: *** [tests/Makefile:277: fate-vsynth2- > > > mpeg2- > > > > > 422] > > > > > > > Error > > > > > > > > > 1 > > > > > > > > > > > >>> > > > > > > > > > > > >>> > > > > > > > > > > > >>> Is anybody seeing the same? > > > > > > > > > > > >>> > > > > > > > > > > > >>> Thanks, > > > > > > > > > > > >>> sw > > > > > > > > > > > >> > > > > > > > > > > > >> http://fate.ffmpeg.org/ doesn't show recent > > > > > regressions > > > > > > > and > > > > > > > > > FATE > > > > > > > > > > > is fine > > > > > > > > > > > >> for me locally. I recently made changes to > FATE > > > > > (namely > > > > > > > the > > > > > > > > > test > > > > > > > > > > > >> requirements, vcodec.mak (where the vsynth- > tests > > > > > reside) > > > > > > > among > > > > > > > > > the > > > > > > > > > > > files > > > > > > > > > > > >> affected), so I am interested in whether the > > > failing > > > > > tests > > > > > > > are > > > > > > > > > > > >> concentrated on the files recently changed by > me > > > (it > > > > > would > > > > > > > > > > > obviously not > > > > > > > > > > > >> haved pushed them if I knew them to cause > issues; > > > also > > > > > > > > > patchwork > > > > > > > > > > > was > > > > > > > > > > > >> fine). > > > > > > > > > > > >> Are these issues reproducible? Can you bisect > > > them? > > > > > > > > > > > >> > > > > > > > > > > > >> - Andreas > > > > > > > > > > > > > > > > > > > > > > > > This may be the alignment issue introduced in > lavfi > > > in > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=17a59a634c39b00 > > > > > > > > > > > a680c6ebbaea58db95594d13d > > > > > > > > > > > > assuming it was not fixed. > > > > > > > > > > > > I think it only affected targets where > > > > > av_cpu_max_align() > > > > > > > > > returned > > > > > > > > > > > 64. > > > > > > > > > > > > > > > > > > > > > > You are completely right: Just making > > > av_cpu_max_align > > > > > return > > > > > > > 64 > > > > > > > > > > > allows > > > > > > > > > > > to reproduce the issue. And it has nothing to do > with > > > my > > > > > > > recent > > > > > > > > > FATE > > > > > > > > > > > patches (545e87f49dc9fa5b880e330fc4e1854df68cc0f1 > > > would > > > > > be > > > > > > > the > > > > > > > > > only > > > > > > > > > > > contender for changes). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I found that the FATE errors can be avoided by > using > > > > > > > > > > > > > > > > > > > > ./configure --disable-avx512 > > > > > > > > > > > > > > > > > > > > Yet, I hope it will be fixed at some time.. > > > > > > > > > > > > > > > > > > > > > > > > > > > > I hope you will post a fix for it soon. > > > > > > > > > > > > > > > > That depends on whether we want alignment values > > 32bit..? > > > > > > > > > > > > > > > > > > > > > > Nope, its about mpeg video encoder in lavc that does > funny > > > things > > > > > > > from old > > > > > > > days when linesize alignment was hardcoded. > > > > > > > > > > > > Adding this > > > > > > > > > > > > align = FFMIN(align, 32); > > > > > > > > > > > > to ff_default_get_video_buffer2() resolves the issue. > > > > > > > > > > > > > > > > Maybe it resolves it for you but that does not mean in any > way > > > that > > > > > that is > > > > > valid or correct solution. > > > > > > > > > > > > > > > > > > > > > > That's why I had written: > > > > > > > > > > > > > That depends on whether we want alignment values > > 32bit..? > > > > > > > > > > > > > > > > 32bit is 4 integers. so I failed to follow whatever you tried > to > > > > > communicate. > > > > > > > > Sorry for being unclear. What I meant to ask is this: > > > > > > > > In cases where av_cpu_max_align() returns something > 32 - for > > > > example 64, do we have to use that max-align value of 64 for > > > alignment > > > > in video.c and framepool,c or could we just limit this to 32? > > > > (hence the FFMIN). > > > > > > > > Limiting the alignment to 32 appears to fix the issue, meaning > that > > > > the three fate tests mentioned above are producing the expected > ref > > > > output and don't fail anymore. > > > > > > > > > > That is hack. > > > > Only when 64bit alignment would be required or beneficial in terms > of > > performance (e.g. AVX512 operations). > > > AVX512 operations are not using 64bit alignment. They don't require, but I thought they would benefit from it. You may know better than I do, I had just read this article: https://www.intel.com/content/www/us/en/developer/articles/technical/data-alignment-to-assist-vectorization.html > > Which is the reason for my question. If yes, then this would be > > surely a bad idea. > > > > It is not Yes/No answer here. Got you, thanks. I have found a very targeted workaround now. Will submit in a moment. sw _______________________________________________ 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] 24+ messages in thread
* Re: [FFmpeg-devel] FATE Errors 2022-04-30 19:06 ` Andreas Rheinhardt 2022-04-30 19:09 ` James Almer @ 2022-04-30 19:12 ` Soft Works 1 sibling, 0 replies; 24+ messages in thread From: Soft Works @ 2022-04-30 19:12 UTC (permalink / raw) To: FFmpeg development discussions and patches > -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of > Andreas Rheinhardt > Sent: Saturday, April 30, 2022 9:07 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] FATE Errors > > Soft Works: > > Hi, > > > > is it a known issue that the current head of the master branch has > > FATE errors? > > > > I get the same locally as well as on the automated GitHub build. > > One is this: > > > > --- ./tests/ref/vsynth/vsynth2-mpeg2-422 2022-04-30 > 14:23:44.330424058 +0000 > > +++ tests/data/fate/vsynth2-mpeg2-422 2022-04-30 14:38:41.071678201 > +0000 > > @@ -1,4 +1,4 @@ > > -b2fa9b73c3547191ecc01b8163abd4e5 *tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > > -379164 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > -704f6a96f93c2409219bd48b74169041 *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > > -stddev: 4.17 PSNR: 35.73 MAXDIFF: 70 bytes: 7603200/ 7603200 > > +8f6d565723ccf879ab2b5aa910b7ce21 *tests/data/fate/vsynth2-mpeg2- > 422.mpeg2video > > +380544 tests/data/fate/vsynth2-mpeg2-422.mpeg2video > > +0797fddea4835687dedddebbbe98fa8f *tests/data/fate/vsynth2-mpeg2- > 422.out.rawvideo > > +stddev: 4.16 PSNR: 35.73 MAXDIFF: 75 bytes: 7603200/ 7603200 > > Test vsynth2-mpeg2-422 failed. Look at tests/data/fate/vsynth2- > mpeg2-422.err for details. > > make: *** [tests/Makefile:277: fate-vsynth2-mpeg2-422] Error 1 > > > > > > Is anybody seeing the same? > > > > Thanks, > > sw > > http://fate.ffmpeg.org/ doesn't show recent regressions and FATE is > fine > for me locally. That's weird. I got the same errors locally and from the GitHub CI build. Here's the output: https://dev.azure.com/githubsync/ffmpeg/_build/results?buildId=10000&view=logs&j=275f1d19-1bd8-5591-b06b-07d489ea915a&t=704ea7c9-8eb0-53b9-6b30-87d5a7befe2c Kind regards, softworkz _______________________________________________ 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] 24+ messages in thread
end of thread, other threads:[~2022-05-28 8:57 UTC | newest] Thread overview: 24+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-04-30 18:54 [FFmpeg-devel] FATE Errors Soft Works 2022-04-30 19:01 ` Felix LeClair 2022-04-30 19:07 ` Soft Works 2022-04-30 19:06 ` Andreas Rheinhardt 2022-04-30 19:09 ` James Almer 2022-04-30 19:19 ` Soft Works 2022-04-30 19:22 ` Felix LeClair 2022-04-30 19:25 ` Soft Works 2022-04-30 19:31 ` Felix LeClair 2022-04-30 19:33 ` James Almer 2022-04-30 19:39 ` Soft Works 2022-04-30 20:02 ` Andreas Rheinhardt 2022-05-27 21:10 ` Soft Works 2022-05-28 7:25 ` Paul B Mahol 2022-05-28 7:26 ` Soft Works 2022-05-28 7:34 ` Paul B Mahol 2022-05-28 7:47 ` Soft Works 2022-05-28 8:04 ` Paul B Mahol 2022-05-28 8:12 ` Soft Works 2022-05-28 8:33 ` Paul B Mahol 2022-05-28 8:41 ` Soft Works 2022-05-28 8:50 ` Paul B Mahol 2022-05-28 8:57 ` Soft Works 2022-04-30 19:12 ` Soft Works
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