* [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 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: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 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: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
* 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
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